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

Side by Side Diff: chrome/browser/ui/views/frame/browser_frame.cc

Issue 1492423003: Rejigger ThemeService: move exposure of ThemeProvider interface to a (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: fix that unittest Created 5 years 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 unified diff | Download patch
OLDNEW
1 // Copyright (c) 2012 The Chromium Authors. All rights reserved. 1 // Copyright (c) 2012 The Chromium Authors. All rights reserved.
2 // Use of this source code is governed by a BSD-style license that can be 2 // Use of this source code is governed by a BSD-style license that can be
3 // found in the LICENSE file. 3 // found in the LICENSE file.
4 4
5 #include "chrome/browser/ui/views/frame/browser_frame.h" 5 #include "chrome/browser/ui/views/frame/browser_frame.h"
6 6
7 #include "base/debug/leak_annotations.h" 7 #include "base/debug/leak_annotations.h"
8 #include "base/i18n/rtl.h" 8 #include "base/i18n/rtl.h"
9 #include "chrome/browser/app_mode/app_mode_utils.h" 9 #include "chrome/browser/app_mode/app_mode_utils.h"
10 #include "chrome/browser/themes/theme_service.h" 10 #include "chrome/browser/themes/theme_service.h"
(...skipping 32 matching lines...) Expand 10 before | Expand all | Expand 10 after
43 #endif 43 #endif
44 44
45 //////////////////////////////////////////////////////////////////////////////// 45 ////////////////////////////////////////////////////////////////////////////////
46 // BrowserFrame, public: 46 // BrowserFrame, public:
47 47
48 BrowserFrame::BrowserFrame(BrowserView* browser_view) 48 BrowserFrame::BrowserFrame(BrowserView* browser_view)
49 : native_browser_frame_(nullptr), 49 : native_browser_frame_(nullptr),
50 root_view_(nullptr), 50 root_view_(nullptr),
51 browser_frame_view_(nullptr), 51 browser_frame_view_(nullptr),
52 browser_view_(browser_view), 52 browser_view_(browser_view),
53 theme_provider_(ThemeServiceFactory::GetForProfile( 53 theme_provider_(
54 browser_view_->browser()->profile())) { 54 &ThemeService::GetThemeProviderForProfile(browser_view_->browser()
55 ->profile())) {
55 browser_view_->set_frame(this); 56 browser_view_->set_frame(this);
56 set_is_secondary_widget(false); 57 set_is_secondary_widget(false);
57 // Don't focus anything on creation, selecting a tab will set the focus. 58 // Don't focus anything on creation, selecting a tab will set the focus.
58 set_focus_on_creation(false); 59 set_focus_on_creation(false);
59 } 60 }
60 61
61 BrowserFrame::~BrowserFrame() { 62 BrowserFrame::~BrowserFrame() {
62 } 63 }
63 64
64 // static 65 // static
(...skipping 192 matching lines...) Expand 10 before | Expand all | Expand 10 after
257 258
258 AvatarMenuButton* BrowserFrame::GetAvatarMenuButton() { 259 AvatarMenuButton* BrowserFrame::GetAvatarMenuButton() {
259 return browser_frame_view_->avatar_button(); 260 return browser_frame_view_->avatar_button();
260 } 261 }
261 262
262 #if defined(FRAME_AVATAR_BUTTON) 263 #if defined(FRAME_AVATAR_BUTTON)
263 NewAvatarButton* BrowserFrame::GetNewAvatarMenuButton() { 264 NewAvatarButton* BrowserFrame::GetNewAvatarMenuButton() {
264 return browser_frame_view_->new_avatar_button(); 265 return browser_frame_view_->new_avatar_button();
265 } 266 }
266 #endif 267 #endif
OLDNEW
« no previous file with comments | « chrome/browser/ui/views/frame/browser_frame.h ('k') | chrome/browser/ui/views/frame/browser_view.cc » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698