[ServiceWorker] Update cache.match/matchAll tests.

[ServiceWorker] Update cache.match/matchAll tests.

Updates include:
   * Test that fragment part of URLs are ignored.
   * Add Cache.match tests for scenarios that were only being tested
     using Cache.matchAll. Currently all Cache.matchAll tests are
     failing due to the method not being implemented.
   * Add tests for URLs that contain embedded credentials.

BUG=374802, 428970
R=jsbell, bkelly@mozilla.com

Committed: https://src.chromium.org/viewvc/blink?view=rev&revision=185044

[asanka, 2014-11-06 00:47:33]

asanka@chromium.org changed required reviewers:
+ jsbell@chromium.org

[asanka, 2014-11-06 00:50:05]

Updates to Cache.add/put/delete for checking if fragments are ignored is
forthcoming.

[asanka, 2014-11-06 18:58:15]

The CQ bit was checked by asanka@chromium.org

[asanka, 2014-11-06 18:59:04]

The CQ bit was unchecked by asanka@chromium.org

[asanka, 2014-11-06 18:59:26]

Patchset #2 (id:20001) has been deleted

[jsbell, 2014-11-07 01:00:56]

one glitch, otherwise lgtm

I didn't rigorously compare against the spec, though. I trust bkelly@ and future
implementers to nitpick the specific assertions as necessary. :)

https://codereview.chromium.org/704043002/diff/1/LayoutTests/http/tests/servi...
File LayoutTests/http/tests/serviceworker/resources/cache-match-worker.js
(right):

https://codereview.chromium.org/704043002/diff/1/LayoutTests/http/tests/servi...
LayoutTests/http/tests/serviceworker/resources/cache-match-worker.js:104: return
cacheAll.match('not-present-in-the-cache')
cacheAll.match -> cache.matchAll ?

[bkelly, 2014-11-07 16:03:19]

Thanks for the patch!  FYI, I plan to review and test this later today.

[asanka, 2014-11-07 18:55:40]

https://codereview.chromium.org/704043002/diff/1/LayoutTests/http/tests/servi...
File LayoutTests/http/tests/serviceworker/resources/cache-match-worker.js
(right):

https://codereview.chromium.org/704043002/diff/1/LayoutTests/http/tests/servi...
LayoutTests/http/tests/serviceworker/resources/cache-match-worker.js:104: return
cacheAll.match('not-present-in-the-cache')
On 2014/11/07 01:00:56, jsbell wrote:
> cacheAll.match -> cache.matchAll ?

Oops. Done.

[bkelly, 2014-11-07 19:17:12]

These changes look good to me.  I ran the tests against my gecko implementation
and they all pass now.  I had one failure, but it was a gecko bug.

I also had to work around an issue with the test prepopulation code.  Right now
it is putting the same Request/Response objects into Cache objects repeatedly. 
I've already adapted our code to check/set the body used flag, so this was
causing failures.  I assume this will be fixed as part of issue #430962, though.

Thanks!

[asanka, 2014-11-10 16:04:10]

The CQ bit was checked by asanka@chromium.org

[commit-bot: I haz the power, 2014-11-10 16:04:59]

CQ is trying da patch. Follow status at
 https://chromium-cq-status.appspot.com/patch-status/704043002/40001

[commit-bot: I haz the power, 2014-11-10 17:05:58]

Committed patchset #2 (id:40001) as 185044