Guard some extensions build rules with enable_extensions.

Guard some extensions build rules with enable_extensions.

Review-Url: https://codereview.chromium.org/2905153003
Cr-Commit-Position: refs/heads/master@{#478122}
Committed: https://chromium.googlesource.com/chromium/src/+/344fe3f3a731f0580069ea7e76afe0567bc22cfc

[Lei Zhang, 2017-05-26 09:22:58]

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

[commit-bot: I haz the power, 2017-05-26 09:23:03]

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-05-26 10:29:03]

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

[commit-bot: I haz the power, 2017-05-26 10:29:04]

Dry run: This issue passed the CQ dry run.

[Lei Zhang, 2017-05-26 17:20:57]

thestig@chromium.org changed reviewers:
+ dpranke@chromium.org

[Dirk Pranke, 2017-06-07 19:59:59]

lgtm, although I feel like I reviewed another CL that did something along these
lines recently, so there might be a merge conflict?

[Lei Zhang, 2017-06-07 20:16:28]

On 2017/06/07 19:59:59, Dirk Pranke (slow) wrote:
> lgtm, although I feel like I reviewed another CL that did something along
these
> lines recently, so there might be a merge conflict?

I'll check when I rebase.

[Lei Zhang, 2017-06-08 20:49:40]

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

[commit-bot: I haz the power, 2017-06-08 20:50:13]

Dry run: CQ is trying da patch.

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

[Lei Zhang, 2017-06-08 21:11:17]

On 2017/06/07 20:16:28, Lei Zhang wrote:
> On 2017/06/07 19:59:59, Dirk Pranke (slow) wrote:
> > lgtm, although I feel like I reviewed another CL that did something along
> these
> > lines recently, so there might be a merge conflict?
> 
> I'll check when I rebase.

Only conflicted in chrome/test/BUILD.gn due to test files being added/removed.

[commit-bot: I haz the power, 2017-06-08 22:44:17]

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

[commit-bot: I haz the power, 2017-06-08 22:44:18]

Dry run: This issue passed the CQ dry run.

[Lei Zhang, 2017-06-08 23:36:25]

The CQ bit was checked by thestig@chromium.org

[Lei Zhang, 2017-06-08 23:36:25]

The patchset sent to the CQ was uploaded after l-g-t-m from dpranke@chromium.org
Link to the patchset: https://codereview.chromium.org/2905153003/#ps20001
(title: "rebase")

[commit-bot: I haz the power, 2017-06-08 23:37:05]

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-06-08 23:42:16]

CQ is committing da patch.

Bot data: {"patchset_id": 20001, "attempt_start_ts": 1496964985275570,
"parent_rev": "1733d120eb79e2d8d49b18892e60ee1297f646f7", "commit_rev":
"344fe3f3a731f0580069ea7e76afe0567bc22cfc"}

[commit-bot: I haz the power, 2017-06-08 23:42:26]

Description was changed from

==========
Guard some extensions build rules with enable_extensions.
==========

to

==========
Guard some extensions build rules with enable_extensions.

Review-Url: https://codereview.chromium.org/2905153003
Cr-Commit-Position: refs/heads/master@{#478122}
Committed:
https://chromium.googlesource.com/chromium/src/+/344fe3f3a731f0580069ea7e76af...
==========

[commit-bot: I haz the power, 2017-06-08 23:42:27]

Committed patchset #2 (id:20001) as
https://chromium.googlesource.com/chromium/src/+/344fe3f3a731f0580069ea7e76af...

[michaelpg, 2017-06-09 16:46:36]

On 2017/06/08 21:11:17, Lei Zhang wrote:
> On 2017/06/07 20:16:28, Lei Zhang wrote:
> > On 2017/06/07 19:59:59, Dirk Pranke (slow) wrote:
> > > lgtm, although I feel like I reviewed another CL that did something along
> > these
> > > lines recently, so there might be a merge conflict?
> > 
> > I'll check when I rebase.
> 
> Only conflicted in chrome/test/BUILD.gn due to test files being added/removed.

There's also https://codereview.chromium.org/2904443004 (which hasn't landed
yet)