Fix MdTextButton colors in incognito windows.

Fix MdTextButton colors in incognito windows.

BUG=598541
Committed: https://crrev.com/1e11d95dbbbf692ac14617db7572aa0f87029f03
Cr-Commit-Position: refs/heads/master@{#384662}

[Evan Stade, 2016-03-30 23:50:39]

estade@chromium.org changed reviewers:
+ sky@chromium.org

[sky, 2016-03-31 01:11:12]

LGTM - is it worth a test to ensure that when new values added the switch is
updated?

[Evan Stade, 2016-03-31 02:14:53]

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

[Evan Stade, 2016-03-31 02:15:21]

On 2016/03/31 01:11:12, sky wrote:
> LGTM - is it worth a test to ensure that when new values added the switch is
> updated?

would you rather see a test or removal of the default case?

[commit-bot: I haz the power, 2016-03-31 02:15:23]

Dry run: CQ is trying da patch. Follow status at
 https://chromium-cq-status.appspot.com/patch-status/1844003003/1
View timeline at
 https://chromium-cq-status.appspot.com/patch-timeline/1844003003/1

[sky, 2016-03-31 02:29:46]

Removal of default case gets the test for free. So, I vote that.

On Wed, Mar 30, 2016 at 7:15 PM,  <estade@chromium.org> wrote:
> On 2016/03/31 01:11:12, sky wrote:
>> LGTM - is it worth a test to ensure that when new values added the switch
>> is
>> updated?
>
> would you rather see a test or removal of the default case?
>
> https://codereview.chromium.org/1844003003/

-- 
You received this message because you are subscribed to the Google Groups
"Chromium-reviews" group.
To unsubscribe from this group and stop receiving emails from it, send an email
to chromium-reviews+unsubscribe@chromium.org.

[commit-bot: I haz the power, 2016-03-31 02:38:05]

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

[commit-bot: I haz the power, 2016-03-31 02:38:06]

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

[Evan Stade, 2016-03-31 19:23:07]

On 2016/03/31 02:29:46, sky wrote:
> Removal of default case gets the test for free. So, I vote that.
> 
> On Wed, Mar 30, 2016 at 7:15 PM,  <mailto:estade@chromium.org> wrote:
> > On 2016/03/31 01:11:12, sky wrote:
> >> LGTM - is it worth a test to ensure that when new values added the switch
> >> is
> >> updated?
> >
> > would you rather see a test or removal of the default case?
> >
> > https://codereview.chromium.org/1844003003/
> 
> -- 
> You received this message because you are subscribed to the Google Groups
> "Chromium-reviews" group.
> To unsubscribe from this group and stop receiving emails from it, send an
email
> to mailto:chromium-reviews+unsubscribe@chromium.org.
> 

done. Sticking in CQ.

[Evan Stade, 2016-03-31 19:23:34]

The CQ bit was checked by estade@chromium.org

[Evan Stade, 2016-03-31 19:23:34]

The patchset sent to the CQ was uploaded after l-g-t-m from sky@chromium.org
Link to the patchset: https://codereview.chromium.org/1844003003/#ps20001
(title: "enumerate all colors")

[commit-bot: I haz the power, 2016-03-31 19:24:00]

CQ is trying da patch. Follow status at
 https://chromium-cq-status.appspot.com/patch-status/1844003003/20001
View timeline at
 https://chromium-cq-status.appspot.com/patch-timeline/1844003003/20001

[commit-bot: I haz the power, 2016-03-31 19:47:33]

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

[commit-bot: I haz the power, 2016-03-31 19:47:34]

Try jobs failed on following builders:
  chromeos_amd64-generic_chromium_compile_only_ng on tryserver.chromium.linux
(JOB_FAILED,
http://build.chromium.org/p/tryserver.chromium.linux/builders/chromeos_amd64-...)
  chromeos_daisy_chromium_compile_only_ng on tryserver.chromium.linux
(JOB_FAILED,
http://build.chromium.org/p/tryserver.chromium.linux/builders/chromeos_daisy_...)

[Evan Stade, 2016-04-01 17:52:33]

The CQ bit was checked by estade@chromium.org

[Evan Stade, 2016-04-01 17:52:33]

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

[commit-bot: I haz the power, 2016-04-01 17:53:04]

CQ is trying da patch. Follow status at
 https://chromium-cq-status.appspot.com/patch-status/1844003003/40001
View timeline at
 https://chromium-cq-status.appspot.com/patch-timeline/1844003003/40001

[commit-bot: I haz the power, 2016-04-01 19:37:42]

Committed patchset #3 (id:40001)

[commit-bot: I haz the power, 2016-04-01 19:39:52]

Description was changed from

==========
Fix MdTextButton colors in incognito windows.

BUG=598541
==========

to

==========
Fix MdTextButton colors in incognito windows.

BUG=598541

Committed: https://crrev.com/1e11d95dbbbf692ac14617db7572aa0f87029f03
Cr-Commit-Position: refs/heads/master@{#384662}
==========

[commit-bot: I haz the power, 2016-04-01 19:39:53]

Patchset 3 (id:??) landed as
https://crrev.com/1e11d95dbbbf692ac14617db7572aa0f87029f03
Cr-Commit-Position: refs/heads/master@{#384662}