Make sure stylus tools are always hidden if disabled by the user.

Make sure stylus tools are always hidden if disabled by the user.

BUG=666450
Committed: https://crrev.com/04d83e71145866d8d7cea925f4ff2c57c0c9bd03
Cr-Commit-Position: refs/heads/master@{#434717}

[jdufault, 2016-11-28 17:42:54]

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

[commit-bot: I haz the power, 2016-11-28 17:43:28]

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, 2016-11-28 18:21:13]

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

[commit-bot: I haz the power, 2016-11-28 18:21:13]

Dry run: This issue passed the CQ dry run.

[jdufault, 2016-11-28 18:25:56]

Description was changed from

==========
Make sure stylus tools are always hidden if disabled by the user.

BUG=666450
==========

to

==========
Make sure stylus tools are always hidden if disabled by the user.

BUG=666450
==========

[jdufault, 2016-11-28 18:25:57]

jdufault@chromium.org changed reviewers:
+ stevenjb@chromium.org

[jdufault, 2016-11-28 18:26:07]

stevenjb@ PTAL. Thanks!

[stevenjb, 2016-11-28 18:47:26]

https://codereview.chromium.org/2535773004/diff/1/ash/common/system/chromeos/...
File ash/common/system/chromeos/palette/palette_tray.h (right):

https://codereview.chromium.org/2535773004/diff/1/ash/common/system/chromeos/...
ash/common/system/chromeos/palette/palette_tray.h:127: // True if the user has
disabled the palette.
s/True/False/ ?

[jdufault, 2016-11-28 19:07:17]

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

[jdufault, 2016-11-28 19:07:48]

https://codereview.chromium.org/2535773004/diff/1/ash/common/system/chromeos/...
File ash/common/system/chromeos/palette/palette_tray.h (right):

https://codereview.chromium.org/2535773004/diff/1/ash/common/system/chromeos/...
ash/common/system/chromeos/palette/palette_tray.h:127: // True if the user has
disabled the palette.
On 2016/11/28 18:47:26, stevenjb wrote:
> s/True/False/ ?
> 

Done.

[commit-bot: I haz the power, 2016-11-28 19:08:33]

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

[stevenjb, 2016-11-28 19:08:38]

lgtm

[jdufault, 2016-11-28 19:11:25]

The CQ bit was unchecked by jdufault@chromium.org

[jdufault, 2016-11-28 19:11:25]

The CQ bit was checked by jdufault@chromium.org

[commit-bot: I haz the power, 2016-11-28 19:12:08]

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, 2016-11-28 21:05:06]

CQ is committing da patch.

Bot data: {"patchset_id": 20001, "attempt_start_ts": 1480360285564870,
"parent_rev": "daa7baa388290a838e63543f4875ba9ad242f1bb", "commit_rev":
"a05b5ef8c89aa47a184dfa5ba51269c3f9fb2b05"}

[commit-bot: I haz the power, 2016-11-28 21:05:31]

Description was changed from

==========
Make sure stylus tools are always hidden if disabled by the user.

BUG=666450
==========

to

==========
Make sure stylus tools are always hidden if disabled by the user.

BUG=666450
==========

[commit-bot: I haz the power, 2016-11-28 21:05:32]

Committed patchset #2 (id:20001)

[commit-bot: I haz the power, 2016-11-28 21:07:33]

Description was changed from

==========
Make sure stylus tools are always hidden if disabled by the user.

BUG=666450
==========

to

==========
Make sure stylus tools are always hidden if disabled by the user.

BUG=666450

Committed: https://crrev.com/04d83e71145866d8d7cea925f4ff2c57c0c9bd03
Cr-Commit-Position: refs/heads/master@{#434717}
==========

[commit-bot: I haz the power, 2016-11-28 21:07:34]

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