Reland of Clean up some NativeTheme code.

Reland of Clean up some NativeTheme code.
 ( https://codereview.chromium.org/2480993002/ )

Now with appropriate use of break in switch statement.

Original issue's description:
> Clean up some NativeTheme code.
>
> - improve instantiation
> - allow to specify separate instances for web and native
> - two NativeThemeAura instances coexist for w/wo overlay scrollbars
>
> BUG=307091
Committed: https://crrev.com/93ed99a0238f25fe391874d3f6dfa02c01acb781
Cr-Commit-Position: refs/heads/master@{#430608}

[Evan Stade, 2016-11-08 01:41:42]

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

[commit-bot: I haz the power, 2016-11-08 01:42:22]

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

[Evan Stade, 2016-11-08 01:42:29]

https://codereview.chromium.org/2485013002/diff/1/ui/native_theme/native_them...
File ui/native_theme/native_theme_aura.cc (right):

https://codereview.chromium.org/2485013002/diff/1/ui/native_theme/native_them...
ui/native_theme/native_theme_aura.cc:205: break;
these breaks were missing from the first patch.

[Evan Stade, 2016-11-08 01:42:32]

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

[commit-bot: I haz the power, 2016-11-08 02:32:50]

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

[commit-bot: I haz the power, 2016-11-08 02:32:51]

Dry run: This issue passed the CQ dry run.

[bokan, 2016-11-08 03:29:28]

lgtm

[sky, 2016-11-08 05:06:29]

LGTM

[Evan Stade, 2016-11-08 14:25:27]

The CQ bit was checked by estade@chromium.org

[commit-bot: I haz the power, 2016-11-08 14:25:48]

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-08 14:33:25]

Committed patchset #1 (id:1)

[commit-bot: I haz the power, 2016-11-08 14:36:06]

Description was changed from

==========
Reland of Clean up some NativeTheme code.
 ( https://codereview.chromium.org/2480993002/ )

Now with appropriate use of break in switch statement.

Original issue's description:
> Clean up some NativeTheme code.
>
> - improve instantiation
> - allow to specify separate instances for web and native
> - two NativeThemeAura instances coexist for w/wo overlay scrollbars
>
> BUG=307091
==========

to

==========
Reland of Clean up some NativeTheme code.
 ( https://codereview.chromium.org/2480993002/ )

Now with appropriate use of break in switch statement.

Original issue's description:
> Clean up some NativeTheme code.
>
> - improve instantiation
> - allow to specify separate instances for web and native
> - two NativeThemeAura instances coexist for w/wo overlay scrollbars
>
> BUG=307091

Committed: https://crrev.com/b2fd274ec5b20ace720420106a13b8c90b61bed1
Cr-Commit-Position: refs/heads/master@{#430607}
==========

[commit-bot: I haz the power, 2016-11-08 14:36:07]

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

[commit-bot: I haz the power, 2016-11-08 14:36:08]

Description was changed from

==========
Reland of Clean up some NativeTheme code.
 ( https://codereview.chromium.org/2480993002/ )

Now with appropriate use of break in switch statement.

Original issue's description:
> Clean up some NativeTheme code.
>
> - improve instantiation
> - allow to specify separate instances for web and native
> - two NativeThemeAura instances coexist for w/wo overlay scrollbars
>
> BUG=307091

Committed: https://crrev.com/b2fd274ec5b20ace720420106a13b8c90b61bed1
Cr-Commit-Position: refs/heads/master@{#430607}
==========

to

==========
Reland of Clean up some NativeTheme code.
 ( https://codereview.chromium.org/2480993002/ )

Now with appropriate use of break in switch statement.

Original issue's description:
> Clean up some NativeTheme code.
>
> - improve instantiation
> - allow to specify separate instances for web and native
> - two NativeThemeAura instances coexist for w/wo overlay scrollbars
>
> BUG=307091

Committed: https://crrev.com/93ed99a0238f25fe391874d3f6dfa02c01acb781
Cr-Commit-Position: refs/heads/master@{#430608}
==========

[commit-bot: I haz the power, 2016-11-08 14:36:09]

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

[findit-for-me, 2016-11-08 14:47:20]

FYI: Findit identified this CL at revision 430608 as the culprit for
failures in the build cycles as shown on:
https://findit-for-me.appspot.com/waterfall/culprit?key=ag9zfmZpbmRpdC1mb3Itb...

[xlai (Olivia), 2016-11-08 14:48:54]

A revert of this CL (patchset #1 id:1) has been created in
https://codereview.chromium.org/2484973003/ by xlai@chromium.org.

The reason for reverting is: Apparently CQ landed this CL twice. Tried reverting
it once to see what happens..