Material Design: Fix MdLabelButton not updating text colors on theme changes

ui/views/controls/button/label_button.cc

Index: ui/views/controls/button/label_button.cc
diff --git a/ui/views/controls/button/label_button.cc b/ui/views/controls/button/label_button.cc
index 21891fca927332c33e9336ab28c08d6d08eabedd..d61d44d9946f98d8e2273071540bafabc5b890cf 100644
--- a/ui/views/controls/button/label_button.cc
+++ b/ui/views/controls/button/label_button.cc
@@ -377,6 +377,11 @@ void LabelButton::SetBorder(std::unique_ptr<Border> border) {
   ResetCachedPreferredSize();
 }
 
+void LabelButton::ResetExplicitnessOfEnabledTextColors() {
+  for (ButtonState state : {STATE_NORMAL, STATE_HOVERED, STATE_PRESSED})

[Evan Stade, 2017/01/07 01:31:40]

hmm, this is not quite as I imagined, because STATE_DISABLED is missing. that is
what you want that for the MdTextButton case but perhaps not for the general
case. It would be nice to reuse this at least in two places (MdTextButton and
below in ResetColorsFromNativeTheme).

Options that come to mind:
a) create a SetTextColorInternal that does everything except flip the explicit
bit, or
b) create a |setting_text_colors_from_theme_| boolean which, when true, disables
flipping the explicit bit
c) expose explicitly_set_colors() and set_explicitly_set_colors(), then do:

auto old_explicitly_set_colors = explicitly_set_colors();
SetEnabledTextColors(...);
set_explicitly_set_colors(old_explicitly_set_colors);

[Tom (Use chromium acct), 2017/01/07 02:35:11]

Done.  I liked (c) the most so I went with that

+    explicitly_set_colors_[state] = false;
+}
+
 gfx::Rect LabelButton::GetChildAreaBounds() {
   return GetLocalBounds();
 }