New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Remove appcache from shared workers #2384

Merged
merged 1 commit into from Feb 24, 2017

Conversation

2 participants
@domenic
Member

domenic commented Feb 23, 2017

This was never implemented anywhere, and since appcache is deprecated,
it never will be.

I just noticed this while scrolling through the spec. I guess we'll want tests of the -historical.html variety.

I didn't bother getting rid of the "cache host" term even though now it just means "Document", as that seemed like a lot of work for marginal gain.

Remove appcache from shared workers
This was never implemented anywhere, and since appcache is deprecated,
it never will be.

domenic added a commit to web-platform-tests/wpt that referenced this pull request Feb 23, 2017

@domenic

This comment has been minimized.

Member

domenic commented Feb 23, 2017

@domenic domenic removed the needs tests label Feb 23, 2017

@foolip

foolip approved these changes Feb 24, 2017

foolip added a commit to web-platform-tests/wpt that referenced this pull request Feb 24, 2017

@foolip foolip merged commit 3f1c84d into master Feb 24, 2017

2 checks passed

continuous-integration/travis-ci/pr The Travis CI build passed
Details
continuous-integration/travis-ci/push The Travis CI build passed
Details

@foolip foolip deleted the rm-appcache-sharedworker branch Feb 24, 2017

zcorpan added a commit to web-platform-tests/wpt that referenced this pull request Feb 24, 2017

foolip added a commit to web-platform-tests/wpt that referenced this pull request Feb 24, 2017

@whatwg whatwg deleted a comment Aug 23, 2017

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment