Native themes: Add menu separator part

ui/native_theme/native_theme_win.h

 // Copyright (c) 2012 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_NATIVE_THEME_NATIVE_THEME_WIN_H_
 #define UI_NATIVE_THEME_NATIVE_THEME_WIN_H_
 
 // A wrapper class for working with custom XP/Vista themes provided in
 // uxtheme.dll.  This is a singleton class that can be grabbed using
 // NativeThemeWin::instance().
@@ skipping 110 matching lines @@
   bool IsUsingHighContrastThemeInternal();
   void CloseHandlesInternal();
 
   // gfx::SysColorChangeListener implementation:
   void OnSysColorChange() override;
 
   // Update the locally cached set of system colors.
   void UpdateSystemColors();
 
   // Painting functions that paint to SkCanvas.
-  void PaintMenuSeparator(SkCanvas* canvas, const gfx::Rect& rect) const;
+  void PaintMenuSeparator(SkCanvas* canvas,
+                          const gfx::Rect& rect,
+                          const MenuSeparatorExtraParams& extra) const;

[Tom (Use chromium acct), 2017/01/25 22:25:44]

sky: Do you think I should get rid of the extra args I added here and just pass
in extra.paint_rect as rect?

[sky, 2017/01/25 22:40:10]

Yes, I think that would be less confusing.

[Tom (Use chromium acct), 2017/01/25 22:49:43]

Done.

   void PaintMenuGutter(SkCanvas* canvas, const gfx::Rect& rect) const;
   void PaintMenuBackground(SkCanvas* canvas, const gfx::Rect& rect) const;
 
   // Paint directly to canvas' HDC.
   void PaintDirect(SkCanvas* destination_canvas,
                    HDC hdc,
                    Part part,
                    State state,
                    const gfx::Rect& rect,
                    const ExtraParams& extra) const;
@@ skipping 16 matching lines @@
                            SIZE* size) const;
 
   HRESULT PaintButton(HDC hdc,
                       State state,
                       const ButtonExtraParams& extra,
                       int part_id,
                       int state_id,
                       RECT* rect) const;
 
   HRESULT PaintMenuSeparator(HDC hdc,
-                             const gfx::Rect& rect) const;
+                             const gfx::Rect& rect,
+                             const MenuSeparatorExtraParams& extra) const;
 
   HRESULT PaintMenuGutter(HDC hdc, const gfx::Rect& rect) const;
 
   // |arrow_direction| determines whether the arrow is pointing to the left or
   // to the right. In RTL locales, sub-menus open from right to left and
   // therefore the menu arrow should point to the left and not to the right.
   HRESULT PaintMenuArrow(HDC hdc,
                          State state,
                          const gfx::Rect& rect,
                          const MenuArrowExtraParams& extra) const;
@@ skipping 192 matching lines @@
 
   // Is |is_using_high_contrast_| valid?
   mutable bool is_using_high_contrast_valid_;
 
   DISALLOW_COPY_AND_ASSIGN(NativeThemeWin);
 };
 
 }  // namespace ui
 
 #endif  // UI_NATIVE_THEME_NATIVE_THEME_WIN_H_