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

Unified Diff: chrome/browser/ui/views/frame/browser_frame.h

Issue 1637203002: [MD] Don't use new OTR native-theming when using a custom browser theme. (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: todo done Created 4 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: chrome/browser/ui/views/frame/browser_frame.h
diff --git a/chrome/browser/ui/views/frame/browser_frame.h b/chrome/browser/ui/views/frame/browser_frame.h
index 4497095b3b13948b1deb022646b3ea224aee0e09..b62cd21532115dce14ca85b8c70c2ca5df4110ec 100644
--- a/chrome/browser/ui/views/frame/browser_frame.h
+++ b/chrome/browser/ui/views/frame/browser_frame.h
@@ -19,6 +19,7 @@ class BrowserView;
class NativeBrowserFrame;
class NonClientFrameView;
class SystemMenuModelBuilder;
+class ThemeService;
namespace gfx {
class FontList;
@@ -28,7 +29,6 @@ class Rect;
namespace ui {
class EventHandler;
class MenuModel;
-class ThemeProvider;
}
namespace views {
@@ -102,6 +102,7 @@ class BrowserFrame
bool GetAccelerator(int command_id,
ui::Accelerator* accelerator) const override;
const ui::ThemeProvider* GetThemeProvider() const override;
+ const ui::NativeTheme* GetNativeTheme() const override;
void SchedulePaintInRect(const gfx::Rect& rect) override;
void OnNativeWidgetActivationChanged(bool active) override;
@@ -140,7 +141,7 @@ class BrowserFrame
// NativeBrowserFrame::UsesNativeSystemMenu() returns false.
scoped_ptr<views::MenuRunner> menu_runner_;
- const ui::ThemeProvider* theme_provider_;
+ const ThemeService* theme_service_;
scoped_ptr<ui::EventHandler> browser_command_handler_;

Powered by Google App Engine
This is Rietveld 408576698