[NTP::PhysicalWeb] Enable by default and depend on physical web flag.

[NTP::PhysicalWeb] Enable by default and depend on physical web flag.

Make Physical Web page provider depend on #enable-physical-web flag.

Reason: M57 feature freeze is close and we want this provider in M57.
We depend on their flag, so that in case they unlaunch their feature, we
do the same. We add our own flag in case we decide to unlaunch our own
feature, while Physical Web in general should works.

BUG=666647
CQ_INCLUDE_TRYBOTS=master.tryserver.chromium.linux:closure_compilation

Review-Url: https://codereview.chromium.org/2614733002
Cr-Commit-Position: refs/heads/master@{#442210}
Committed: https://chromium.googlesource.com/chromium/src/+/e6dc58fc9701e2f1bec9c49f1375e871c8b0eb51

[vitaliii, 2017-01-04 16:55:21]

Description was changed from

==========
[NTP::PhysicalWeb] Enable by default and depend on physical web flag.

Make Physical Web page provider depend on #enable-physical-web flag.

Reason: M57 feature freeze is close and we want this provider in M57.
We depend on their flag, so that in case they unlaunch their feature, we
do the same. We add our own flag in case we decide to unlaunch our own
feature, while Physical Web in general should works.

BUG=666647
==========

to

==========
[NTP::PhysicalWeb] Enable by default and depend on physical web flag.

Make Physical Web page provider depend on #enable-physical-web flag.

Reason: M57 feature freeze is close and we want this provider in M57.
We depend on their flag, so that in case they unlaunch their feature, we
do the same. We add our own flag in case we decide to unlaunch our own
feature, while Physical Web in general should works.

BUG=666647
CQ_INCLUDE_TRYBOTS=master.tryserver.chromium.linux:closure_compilation
==========

[vitaliii, 2017-01-04 16:56:00]

The CQ bit was checked by vitaliii@chromium.org to run a CQ dry run

[commit-bot: I haz the power, 2017-01-04 16:56:11]

Dry run: CQ is trying da patch. Follow status at
 
https://chromium-cq-status.appspot.com/v2/patch-status/codereview.chromium.or...

[vitaliii, 2017-01-04 16:56:52]

vitaliii@chromium.org changed reviewers:
+ jkrcal@chromium.org

[vitaliii, 2017-01-04 16:56:52]

[small CL]

Hi jkrcal@,

Please take a look at my components/ntp_snippets change.

[vitaliii, 2017-01-04 16:57:27]

vitaliii@chromium.org changed reviewers:
+ bauerb@chromium.org

[vitaliii, 2017-01-04 16:57:28]

[small CL]

Hi bauerb@,

Please take a look at my snippets_internals_.* change.

[Bernhard Bauer, 2017-01-04 16:59:53]

lgtm

[commit-bot: I haz the power, 2017-01-04 18:52:18]

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

[commit-bot: I haz the power, 2017-01-04 18:52:19]

Dry run: Try jobs failed on following builders:
  linux_android_rel_ng on master.tryserver.chromium.android (JOB_FAILED,
https://build.chromium.org/p/tryserver.chromium.android/builders/linux_androi...)

[vitaliii, 2017-01-05 08:01:21]

The CQ bit was checked by vitaliii@chromium.org to run a CQ dry run

[commit-bot: I haz the power, 2017-01-05 08:01:35]

Dry run: CQ is trying da patch. Follow status at
 
https://chromium-cq-status.appspot.com/v2/patch-status/codereview.chromium.or...

[commit-bot: I haz the power, 2017-01-05 09:42:34]

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

[commit-bot: I haz the power, 2017-01-05 09:42:34]

Dry run: Try jobs failed on following builders:
  linux_android_rel_ng on master.tryserver.chromium.android (JOB_FAILED,
https://build.chromium.org/p/tryserver.chromium.android/builders/linux_androi...)

[jkrcal, 2017-01-05 10:23:51]

On 2017/01/04 16:59:53, Bernhard Bauer wrote:
> lgtm

lgtm

[vitaliii, 2017-01-05 10:35:42]

The CQ bit was checked by vitaliii@chromium.org to run a CQ dry run

[commit-bot: I haz the power, 2017-01-05 10:35:56]

Dry run: CQ is trying da patch. Follow status at
 
https://chromium-cq-status.appspot.com/v2/patch-status/codereview.chromium.or...

[commit-bot: I haz the power, 2017-01-05 12:01:10]

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

[commit-bot: I haz the power, 2017-01-05 12:01:11]

Dry run: Try jobs failed on following builders:
  linux_android_rel_ng on master.tryserver.chromium.android (JOB_FAILED,
https://build.chromium.org/p/tryserver.chromium.android/builders/linux_androi...)

[vitaliii, 2017-01-05 12:05:46]

The CQ bit was checked by vitaliii@chromium.org to run a CQ dry run

[commit-bot: I haz the power, 2017-01-05 12:05:59]

Dry run: CQ is trying da patch. Follow status at
 
https://chromium-cq-status.appspot.com/v2/patch-status/codereview.chromium.or...

[commit-bot: I haz the power, 2017-01-05 13:35:13]

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

[commit-bot: I haz the power, 2017-01-05 13:35:14]

Dry run: Try jobs failed on following builders:
  linux_android_rel_ng on master.tryserver.chromium.android (JOB_FAILED,
https://build.chromium.org/p/tryserver.chromium.android/builders/linux_androi...)

[vitaliii, 2017-01-05 14:47:15]

The CQ bit was checked by vitaliii@chromium.org to run a CQ dry run

[commit-bot: I haz the power, 2017-01-05 14:47:30]

Dry run: CQ is trying da patch. Follow status at
 
https://chromium-cq-status.appspot.com/v2/patch-status/codereview.chromium.or...

[vitaliii, 2017-01-05 15:51:59]

The CQ bit was checked by vitaliii@chromium.org

[vitaliii, 2017-01-05 15:51:59]

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

[commit-bot: I haz the power, 2017-01-05 15:52:14]

CQ is trying da patch. Follow status at
 
https://chromium-cq-status.appspot.com/v2/patch-status/codereview.chromium.or...

[commit-bot: I haz the power, 2017-01-05 17:21:38]

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

[commit-bot: I haz the power, 2017-01-05 17:21:40]

Try jobs failed on following builders:
  linux_android_rel_ng on master.tryserver.chromium.android (JOB_FAILED,
https://build.chromium.org/p/tryserver.chromium.android/builders/linux_androi...)

[vitaliii, 2017-01-06 10:20:19]

The CQ bit was checked by vitaliii@chromium.org to run a CQ dry run

[commit-bot: I haz the power, 2017-01-06 10:20:27]

Dry run: CQ is trying da patch. Follow status at
 
https://chromium-cq-status.appspot.com/v2/patch-status/codereview.chromium.or...

[commit-bot: I haz the power, 2017-01-06 11:52:23]

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

[commit-bot: I haz the power, 2017-01-06 11:52:24]

Dry run: Try jobs failed on following builders:
  linux_android_rel_ng on master.tryserver.chromium.android (JOB_FAILED,
https://build.chromium.org/p/tryserver.chromium.android/builders/linux_androi...)

[vitaliii, 2017-01-09 07:46:03]

The CQ bit was checked by vitaliii@chromium.org to run a CQ dry run

[commit-bot: I haz the power, 2017-01-09 07:46:10]

Dry run: CQ is trying da patch. Follow status at
 
https://chromium-cq-status.appspot.com/v2/patch-status/codereview.chromium.or...

[vitaliii, 2017-01-09 08:21:19]

The CQ bit was unchecked by vitaliii@chromium.org

[vitaliii, 2017-01-09 08:21:24]

The CQ bit was checked by vitaliii@chromium.org

[vitaliii, 2017-01-09 08:21:25]

The patchset sent to the CQ was uploaded after l-g-t-m from bauerb@chromium.org,
jkrcal@chromium.org
Link to the patchset: https://codereview.chromium.org/2614733002/#ps60001
(title: "clean rebase.")

[commit-bot: I haz the power, 2017-01-09 08:21:28]

CQ is trying da patch. Follow status at
 
https://chromium-cq-status.appspot.com/v2/patch-status/codereview.chromium.or...

[commit-bot: I haz the power, 2017-01-09 08:45:35]

CQ is committing da patch.

Bot data: {"patchset_id": 60001, "attempt_start_ts": 1483950084520010,
"parent_rev": "4c28543e8b6e6897f0f0dbb66cb9035a1872b422", "commit_rev":
"e6dc58fc9701e2f1bec9c49f1375e871c8b0eb51"}

[commit-bot: I haz the power, 2017-01-09 08:46:20]

Description was changed from

==========
[NTP::PhysicalWeb] Enable by default and depend on physical web flag.

Make Physical Web page provider depend on #enable-physical-web flag.

Reason: M57 feature freeze is close and we want this provider in M57.
We depend on their flag, so that in case they unlaunch their feature, we
do the same. We add our own flag in case we decide to unlaunch our own
feature, while Physical Web in general should works.

BUG=666647
CQ_INCLUDE_TRYBOTS=master.tryserver.chromium.linux:closure_compilation
==========

to

==========
[NTP::PhysicalWeb] Enable by default and depend on physical web flag.

Make Physical Web page provider depend on #enable-physical-web flag.

Reason: M57 feature freeze is close and we want this provider in M57.
We depend on their flag, so that in case they unlaunch their feature, we
do the same. We add our own flag in case we decide to unlaunch our own
feature, while Physical Web in general should works.

BUG=666647
CQ_INCLUDE_TRYBOTS=master.tryserver.chromium.linux:closure_compilation

Review-Url: https://codereview.chromium.org/2614733002
Cr-Commit-Position: refs/heads/master@{#442210}
Committed:
https://chromium.googlesource.com/chromium/src/+/e6dc58fc9701e2f1bec9c49f1375...
==========

[commit-bot: I haz the power, 2017-01-09 08:46:21]

Committed patchset #4 (id:60001) as
https://chromium.googlesource.com/chromium/src/+/e6dc58fc9701e2f1bec9c49f1375...