ServiceWorker: Enable WindowClient.navigate() method by default.

ServiceWorker: Enable WindowClient.navigate() method by default.

Enable WindowClient.navigate() method by default.

Intent to ship:
  http://groups.google.com/a/chromium.org/forum/#!topic/blink-dev/21YdGHt2ogM

Spec:
  http://slightlyoff.github.io/ServiceWorker/spec/service_worker/#client-navigate

BUG=500911
Committed: https://crrev.com/ee1a4f22a370d13d746af1c78aecde6f67f16988
Cr-Commit-Position: refs/heads/master@{#363473}

[zino, 2015-11-26 11:15:26]

Description was changed from

==========
ServiceWorker: Enable WindowClient.navigate() method by default.

Enable WindowClient.navigate() method by default.

Intent to ship:
  http://groups.google.com/a/chromium.org/forum/#!topic/blink-dev/21YdGHt2ogM

Intent to implement:
  http://groups.google.com/a/chromium.org/forum/#!topic/blink-dev/KETlHSSc878

Spec:
 
http://slightlyoff.github.io/ServiceWorker/spec/service_worker/#client-navigate

BUG=500911
==========

to

==========
ServiceWorker: Enable WindowClient.navigate() method by default.

Enable WindowClient.navigate() method by default.

Intent to ship:
  http://groups.google.com/a/chromium.org/forum/#!topic/blink-dev/21YdGHt2ogM

Intent to implement:
  http://groups.google.com/a/chromium.org/forum/#!topic/blink-dev/KETlHSSc878

Spec:
 
http://slightlyoff.github.io/ServiceWorker/spec/service_worker/#client-navigate

BUG=500911
==========

[zino, 2015-11-27 06:10:27]

Patchset #2 (id:20001) has been deleted

[zino, 2015-12-04 19:42:16]

jinho.bang@samsung.com changed reviewers:
+ falken@chromium.org, jungkee.song@samsung.com, nhiroki@chromium.org

[zino, 2015-12-04 19:42:51]

Description was changed from

==========
ServiceWorker: Enable WindowClient.navigate() method by default.

Enable WindowClient.navigate() method by default.

Intent to ship:
  http://groups.google.com/a/chromium.org/forum/#!topic/blink-dev/21YdGHt2ogM

Intent to implement:
  http://groups.google.com/a/chromium.org/forum/#!topic/blink-dev/KETlHSSc878

Spec:
 
http://slightlyoff.github.io/ServiceWorker/spec/service_worker/#client-navigate

BUG=500911
==========

to

==========
ServiceWorker: Enable WindowClient.navigate() method by default.

Enable WindowClient.navigate() method by default.

Intent to ship:
  http://groups.google.com/a/chromium.org/forum/#!topic/blink-dev/21YdGHt2ogM

Spec:
 
http://slightlyoff.github.io/ServiceWorker/spec/service_worker/#client-navigate

BUG=500911
==========

[zino, 2015-12-04 19:48:00]

jinho.bang@samsung.com changed reviewers:
+ tkent@chromium.org

[zino, 2015-12-04 19:48:49]

Please take a look.

Thank you.

[tkent, 2015-12-06 22:50:21]

lgtm

[nhiroki, 2015-12-07 00:29:29]

LGTM after the other CL[1] is landed.

(Could you add a test case for in-scope-but-not-controlled-client[2] in
windowclient-navigate.html? A separate CL is OK.)

[1] https://codereview.chromium.org/1481523006/
[2]
https://github.com/slightlyoff/ServiceWorker/issues/752#issuecomment-161877793

[zino, 2015-12-07 10:54:07]

On 2015/12/07 00:29:29, nhiroki wrote:
> LGTM after the other CL[1] is landed.
> 
> (Could you add a test case for in-scope-but-not-controlled-client[2] in
> windowclient-navigate.html? A separate CL is OK.)
> 
> [1] https://codereview.chromium.org/1481523006/
> [2]
> https://github.com/slightlyoff/ServiceWorker/issues/752#issuecomment-161877793

The CL[1] has been landed.

I'll work on the test[2] in separate CL.

Thank you.

[zino, 2015-12-07 10:54:24]

The CQ bit was checked by jinho.bang@samsung.com

[commit-bot: I haz the power, 2015-12-07 10:55:44]

CQ is trying da patch. Follow status at
 https://chromium-cq-status.appspot.com/patch-status/1481753002/1
View timeline at
 https://chromium-cq-status.appspot.com/patch-timeline/1481753002/1

[commit-bot: I haz the power, 2015-12-07 12:00:38]

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

[commit-bot: I haz the power, 2015-12-07 12:00:38]

Try jobs failed on following builders:
  win_chromium_rel_ng on tryserver.chromium.win (JOB_FAILED,
http://build.chromium.org/p/tryserver.chromium.win/builders/win_chromium_rel_...)

[zino, 2015-12-07 12:31:40]

The CQ bit was checked by jinho.bang@samsung.com

[zino, 2015-12-07 12:31:40]

The patchset sent to the CQ was uploaded after l-g-t-m from tkent@chromium.org,
nhiroki@chromium.org
Link to the patchset: https://codereview.chromium.org/1481753002/#ps40001
(title: " ")

[commit-bot: I haz the power, 2015-12-07 12:33:43]

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

[commit-bot: I haz the power, 2015-12-07 14:13:28]

Description was changed from

==========
ServiceWorker: Enable WindowClient.navigate() method by default.

Enable WindowClient.navigate() method by default.

Intent to ship:
  http://groups.google.com/a/chromium.org/forum/#!topic/blink-dev/21YdGHt2ogM

Spec:
 
http://slightlyoff.github.io/ServiceWorker/spec/service_worker/#client-navigate

BUG=500911
==========

to

==========
ServiceWorker: Enable WindowClient.navigate() method by default.

Enable WindowClient.navigate() method by default.

Intent to ship:
  http://groups.google.com/a/chromium.org/forum/#!topic/blink-dev/21YdGHt2ogM

Spec:
 
http://slightlyoff.github.io/ServiceWorker/spec/service_worker/#client-navigate

BUG=500911
==========

[commit-bot: I haz the power, 2015-12-07 14:13:28]

Committed patchset #2 (id:40001)

[commit-bot: I haz the power, 2015-12-07 14:14:17]

Description was changed from

==========
ServiceWorker: Enable WindowClient.navigate() method by default.

Enable WindowClient.navigate() method by default.

Intent to ship:
  http://groups.google.com/a/chromium.org/forum/#!topic/blink-dev/21YdGHt2ogM

Spec:
 
http://slightlyoff.github.io/ServiceWorker/spec/service_worker/#client-navigate

BUG=500911
==========

to

==========
ServiceWorker: Enable WindowClient.navigate() method by default.

Enable WindowClient.navigate() method by default.

Intent to ship:
  http://groups.google.com/a/chromium.org/forum/#!topic/blink-dev/21YdGHt2ogM

Spec:
 
http://slightlyoff.github.io/ServiceWorker/spec/service_worker/#client-navigate

BUG=500911

Committed: https://crrev.com/ee1a4f22a370d13d746af1c78aecde6f67f16988
Cr-Commit-Position: refs/heads/master@{#363473}
==========

[commit-bot: I haz the power, 2015-12-07 14:14:18]

Patchset 2 (id:??) landed as
https://crrev.com/ee1a4f22a370d13d746af1c78aecde6f67f16988
Cr-Commit-Position: refs/heads/master@{#363473}