Implement the palette help and options buttons.

Implement the palette help and options buttons.

BUG=625251
Committed: https://crrev.com/2c49f1d783a00f8a7dcb31b36d845f150ffb5d80
Cr-Commit-Position: refs/heads/master@{#411856}

[jdufault, 2016-08-05 00:15:05]

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

[commit-bot: I haz the power, 2016-08-05 00:15:37]

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-08-05 00:25:49]

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

[commit-bot: I haz the power, 2016-08-05 00:25:50]

Dry run: Try jobs failed on following builders:
  linux_chromium_chromeos_rel_ng on master.tryserver.chromium.linux (JOB_FAILED,
http://build.chromium.org/p/tryserver.chromium.linux/builders/linux_chromium_...)
  linux_chromium_rel_ng on master.tryserver.chromium.linux (JOB_FAILED,
http://build.chromium.org/p/tryserver.chromium.linux/builders/linux_chromium_...)

[jdufault, 2016-08-05 21:39:04]

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

[jdufault, 2016-08-05 21:39:29]

Patchset #1 (id:1) has been deleted

[commit-bot: I haz the power, 2016-08-05 21:39:39]

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-08-05 22:16:24]

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

[commit-bot: I haz the power, 2016-08-05 22:16:25]

Dry run: This issue passed the CQ dry run.

[jdufault, 2016-08-08 20:40:44]

Description was changed from

==========
Implement the palette help and options buttons.

BUG=625251
==========

to

==========
Implement the palette help and options buttons.

BUG=625251
==========

[jdufault, 2016-08-08 20:40:45]

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

[jdufault, 2016-08-08 20:40:55]

Oshima, PTAL. Thanks!

[oshima, 2016-08-10 20:22:44]

lgtm with a nit

https://codereview.chromium.org/2216933003/diff/20001/ash/common/system/chrom...
File ash/common/system/chromeos/palette/palette_tray.cc (right):

https://codereview.chromium.org/2216933003/diff/20001/ash/common/system/chrom...
ash/common/system/chromeos/palette/palette_tray.cc:60: TitleView(PaletteTray*
palette_tray) : palette_tray_(palette_tray) {
explicit

[jdufault, 2016-08-11 18:55:19]

Thanks!

https://codereview.chromium.org/2216933003/diff/20001/ash/common/system/chrom...
File ash/common/system/chromeos/palette/palette_tray.cc (right):

https://codereview.chromium.org/2216933003/diff/20001/ash/common/system/chrom...
ash/common/system/chromeos/palette/palette_tray.cc:60: TitleView(PaletteTray*
palette_tray) : palette_tray_(palette_tray) {
On 2016/08/10 20:22:44, oshima wrote:
> explicit

Done.

[jdufault, 2016-08-11 18:55:29]

The CQ bit was checked by jdufault@chromium.org

[jdufault, 2016-08-11 18:55:29]

The patchset sent to the CQ was uploaded after l-g-t-m from oshima@chromium.org
Link to the patchset: https://codereview.chromium.org/2216933003/#ps40001
(title: "Address nit")

[commit-bot: I haz the power, 2016-08-11 18:56:09]

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-08-11 19:07:04]

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

[commit-bot: I haz the power, 2016-08-11 19:07:05]

Try jobs failed on following builders:
  chromium_presubmit on master.tryserver.chromium.linux (JOB_FAILED,
http://build.chromium.org/p/tryserver.chromium.linux/builders/chromium_presub...)

[jdufault, 2016-08-11 19:39:57]

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

[jdufault, 2016-08-11 19:40:20]

holte@ PTAL at tools/metrics/actions/actions.xml. Thanks!

[Steven Holte, 2016-08-13 00:39:08]

On 2016/08/11 19:40:20, jdufault wrote:
> holte@ PTAL at tools/metrics/actions/actions.xml. Thanks!

actions lgtm

[jdufault, 2016-08-13 00:40:04]

The CQ bit was checked by jdufault@chromium.org

[commit-bot: I haz the power, 2016-08-13 00:40:32]

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-08-13 02:13:20]

Description was changed from

==========
Implement the palette help and options buttons.

BUG=625251
==========

to

==========
Implement the palette help and options buttons.

BUG=625251
==========

[commit-bot: I haz the power, 2016-08-13 02:13:21]

Committed patchset #2 (id:40001)

[commit-bot: I haz the power, 2016-08-13 02:15:31]

Description was changed from

==========
Implement the palette help and options buttons.

BUG=625251
==========

to

==========
Implement the palette help and options buttons.

BUG=625251

Committed: https://crrev.com/2c49f1d783a00f8a7dcb31b36d845f150ffb5d80
Cr-Commit-Position: refs/heads/master@{#411856}
==========

[commit-bot: I haz the power, 2016-08-13 02:15:32]

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