Native themes: Add separator color

ui/views/controls/separator.h

 // Copyright (c) 2011 The Chromium Authors. All rights reserved.
 // Use of this source code is governed by a BSD-style license that can be
 // found in the LICENSE file.
 
 #ifndef UI_VIEWS_CONTROLS_SEPARATOR_H_
 #define UI_VIEWS_CONTROLS_SEPARATOR_H_
 
 #include <string>
 
 #include "base/macros.h"
@@ skipping 22 matching lines @@
 
   int size() const { return size_; }
   // Preferred size of one axis: height for horizontal separator
   // and width for vertical separator
   void SetPreferredSize(int size);
 
   // Overridden from View:
   gfx::Size GetPreferredSize() const override;
   void GetAccessibleNodeData(ui::AXNodeData* node_data) override;
   void OnPaint(gfx::Canvas* canvas) override;
+  void OnNativeThemeChanged(const ui::NativeTheme* theme) override;
   const char* GetClassName() const override;
 
  private:
+  void SetColorFromNativeTheme();
+
   const Orientation orientation_;
   SkColor color_;

[Evan Stade, 2017/01/10 16:58:07]

if you make this base::Optional<SkColor> you don't need the new member

[Tom (Use chromium acct), 2017/01/10 20:59:10]

But then color() would be more complicated and it would have to be renamed to
GetColor() :(

[Evan Stade, 2017/01/11 00:32:04]

good thing it's never actually called then

+  bool color_overridden_;
   int size_;
 
   DISALLOW_COPY_AND_ASSIGN(Separator);
 };
 
 }  // namespace views
 
 #endif  // UI_VIEWS_CONTROLS_SEPARATOR_H_