MD Settings: Should not show "Import and Bind" button on non-CrOS.

MD Settings: Should not show "Import and Bind" button on non-CrOS.

BUG=600468
R=stevenjb@chromium.org
CQ_INCLUDE_TRYBOTS=master.tryserver.chromium.linux:closure_compilation

Review-Url: https://codereview.chromium.org/2704713002
Cr-Commit-Position: refs/heads/master@{#451407}
Committed: https://chromium.googlesource.com/chromium/src/+/6bdd26b43756cb3fb97568d7f126359aff6a5904

[dpapad, 2017-02-17 19:39:22]

Description was changed from

==========
MD Settings: Should not show "Import and Bind" button on non-CrOS.

BUG=600468
R=stevenjb@chromium.org
==========

to

==========
MD Settings: Should not show "Import and Bind" button on non-CrOS.

BUG=600468
R=stevenjb@chromium.org
CQ_INCLUDE_TRYBOTS=master.tryserver.chromium.linux:closure_compilation
==========

[dpapad, 2017-02-17 19:39:42]

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

[commit-bot: I haz the power, 2017-02-17 19:40:51]

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

[stevenjb, 2017-02-17 20:06:58]

doh. lgtm.

[dpapad, 2017-02-17 20:22:37]

https://codereview.chromium.org/2704713002/diff/20001/chrome/browser/resource...
File chrome/browser/resources/settings/settings_resources.grd (right):

https://codereview.chromium.org/2704713002/diff/20001/chrome/browser/resource...
chrome/browser/resources/settings/settings_resources.grd:281: preprocess="true"
Actually this file also needs to be preprocessed.

[stevenjb, 2017-02-17 20:34:26]

Would be nice if this were automated somehow... still lgtm.

[dpapad, 2017-02-17 20:39:45]

The CQ bit was checked by dpapad@chromium.org

[commit-bot: I haz the power, 2017-02-17 20:40:34]

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

[dpapad, 2017-02-17 20:41:21]

On 2017/02/17 at 20:34:26, stevenjb wrote:
> Would be nice if this were automated somehow... still lgtm.

This would have caused a runtime parse error until very recently, where we put
them behind a comment "//" (for other good reasons). Automated check SG.

[commit-bot: I haz the power, 2017-02-18 00:31:07]

CQ is committing da patch.

Bot data: {"patchset_id": 20001, "attempt_start_ts": 1487363985882750,
"parent_rev": "afd9060c2b7facfeb9a8ebef831557e4b4c531dd", "commit_rev":
"6bdd26b43756cb3fb97568d7f126359aff6a5904"}

[commit-bot: I haz the power, 2017-02-18 00:31:43]

Description was changed from

==========
MD Settings: Should not show "Import and Bind" button on non-CrOS.

BUG=600468
R=stevenjb@chromium.org
CQ_INCLUDE_TRYBOTS=master.tryserver.chromium.linux:closure_compilation
==========

to

==========
MD Settings: Should not show "Import and Bind" button on non-CrOS.

BUG=600468
R=stevenjb@chromium.org
CQ_INCLUDE_TRYBOTS=master.tryserver.chromium.linux:closure_compilation

Review-Url: https://codereview.chromium.org/2704713002
Cr-Commit-Position: refs/heads/master@{#451407}
Committed:
https://chromium.googlesource.com/chromium/src/+/6bdd26b43756cb3fb97568d7f126...
==========

[commit-bot: I haz the power, 2017-02-18 00:31:45]

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