Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(487)

Unified Diff: ui/native_theme/native_theme_win.h

Issue 2655553003: Native themes: Add menu separator part (Closed)
Patch Set: Created 3 years, 11 months ago
Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.
Jump to:
View side-by-side diff with in-line comments
Download patch
Index: ui/native_theme/native_theme_win.h
diff --git a/ui/native_theme/native_theme_win.h b/ui/native_theme/native_theme_win.h
index 52b2aebb13ad19ce4b1ed8b90a3ad2e5fac1b8b4..78f11a13c78fe90eb072c6781180ff917c940400 100644
--- a/ui/native_theme/native_theme_win.h
+++ b/ui/native_theme/native_theme_win.h
@@ -128,7 +128,9 @@ class NATIVE_THEME_EXPORT NativeThemeWin : public NativeTheme,
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 ar
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;
@@ -165,7 +167,8 @@ class NATIVE_THEME_EXPORT NativeThemeWin : public NativeTheme,
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;

Powered by Google App Engine
This is Rietveld 408576698