ServiceWorker: Strip fragment from scope and script URLs

ServiceWorker: Strip fragment identifier from scope and script URLs

Since the fragment is not sent to the server, it can't be matched
by the scope anyway.

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

[jsbell, 2014-05-20 20:18:54]

Anyone?

Not urgent, but trivial

[jsbell, 2014-05-20 23:58:50]

Setting an actual reviewer field.

Anyone - R? CQ?

[kinuko, 2014-05-21 03:36:46]

lgtm

[kinuko, 2014-05-21 03:37:15]

https://codereview.chromium.org/291263002/diff/20001/LayoutTests/http/tests/s...
File LayoutTests/http/tests/serviceworker/serviceworkerobject-scope.html
(right):

https://codereview.chromium.org/291263002/diff/20001/LayoutTests/http/tests/s...
LayoutTests/http/tests/serviceworker/serviceworkerobject-scope.html:20:
expectedURL = expectedURL.replace(/#.*$/, '');
It might be good to include this in normalizeURL helper function

[kinuko, 2014-05-21 03:37:27]

The CQ bit was checked by kinuko@chromium.org

[commit-bot: I haz the power, 2014-05-21 03:38:15]

CQ is trying da patch. Follow status at
 https://chromium-status.appspot.com/cq/jsbell@chromium.org/291263002/20001

[commit-bot: I haz the power, 2014-05-21 09:39:41]

FYI, CQ is re-trying this CL (attempt #1).
Please consider checking whether the failures are real,
and report flakes to chrome-troopers@google.com.
The failing builders are:
  mac_blink_rel on tryserver.blink
(http://build.chromium.org/p/tryserver.blink/builders/mac_blink_rel/builds/8390)
  win_blink_rel on tryserver.blink
(http://build.chromium.org/p/tryserver.blink/builders/win_blink_rel/builds/8489)

[commit-bot: I haz the power, 2014-05-22 05:57:07]

FYI, CQ is re-trying this CL (attempt #1).
Please consider checking whether the failures are real,
and report flakes to chrome-troopers@google.com.
The failing builders are:
  linux_blink_dbg on tryserver.blink
(http://build.chromium.org/p/tryserver.blink/builders/linux_blink_dbg/builds/9178)
  linux_blink_rel on tryserver.blink
(http://build.chromium.org/p/tryserver.blink/builders/linux_blink_rel/builds/8988)
  mac_blink_rel on tryserver.blink
(http://build.chromium.org/p/tryserver.blink/builders/mac_blink_rel/builds/8545)
  win_blink_rel on tryserver.blink
(http://build.chromium.org/p/tryserver.blink/builders/win_blink_rel/builds/8640)

[commit-bot: I haz the power, 2014-05-22 06:40:21]

The CQ bit was unchecked by commit-bot@chromium.org

[commit-bot: I haz the power, 2014-05-22 06:40:21]

Try jobs failed on following builders:
  mac_blink_rel on tryserver.blink
(http://build.chromium.org/p/tryserver.blink/builders/mac_blink_rel/builds/8570)

[jsbell, 2014-05-22 16:49:05]

https://codereview.chromium.org/291263002/diff/20001/LayoutTests/http/tests/s...
File LayoutTests/http/tests/serviceworker/serviceworkerobject-scope.html
(right):

https://codereview.chromium.org/291263002/diff/20001/LayoutTests/http/tests/s...
LayoutTests/http/tests/serviceworker/serviceworkerobject-scope.html:20:
expectedURL = expectedURL.replace(/#.*$/, '');
On 2014/05/21 03:37:15, kinuko wrote:
> It might be good to include this in normalizeURL helper function

Done.

[jsbell, 2014-05-22 16:49:49]

The CQ bit was checked by jsbell@chromium.org

[commit-bot: I haz the power, 2014-05-22 16:50:13]

CQ is trying da patch. Follow status at
 https://chromium-status.appspot.com/cq/jsbell@chromium.org/291263002/60001

[commit-bot: I haz the power, 2014-05-23 00:26:30]

Change committed as 174610