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

Side by Side Diff: chrome/browser/views/frame/standard_extender.cc

Issue 329009: CompactNavigationBar for toolkit views build, with some design change per Cole's request. (Closed)
Patch Set: updates per review Created 11 years, 1 month 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
« no previous file with comments | « chrome/browser/views/frame/browser_view.cc ('k') | chrome/browser/views/location_bar_view.h » ('j') | no next file with comments »
Toggle Intra-line Diffs ('i') | Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
OLDNEW
1 // Copyright (c) 2009 The Chromium Authors. All rights reserved. 1 // Copyright (c) 2009 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/views/frame/browser_extender.h" 5 #include "chrome/browser/views/frame/browser_extender.h"
6 6
7 namespace { 7 namespace {
8 8
9 // StandardExtender for non ChromeOS build. This currently adds/does nothing. 9 // StandardExtender for non ChromeOS build. This currently adds/does nothing.
10 // TODO(oshima): Add MainMenu support with a command line flag. 10 // TODO(oshima): Add MainMenu support with a command line flag.
11 class StandardExtender : public BrowserExtender { 11 class StandardExtender : public BrowserExtender {
12 public: 12 public:
13 explicit StandardExtender(BrowserView* browser_view) 13 explicit StandardExtender(BrowserView* browser_view)
14 : BrowserExtender(browser_view) { 14 : BrowserExtender(browser_view) {
15 } 15 }
16 virtual ~StandardExtender() {} 16 virtual ~StandardExtender() {}
17 17
18 private: 18 private:
19 // BrowserExtender overrides. 19 // BrowserExtender overrides.
20 virtual void Init() {} 20 virtual void Init() {}
21 virtual gfx::Rect Layout(const gfx::Rect& bounds) { return bounds; }
21 virtual bool NonClientHitTest(const gfx::Point& point) { return false; } 22 virtual bool NonClientHitTest(const gfx::Point& point) { return false; }
22 virtual void Show() {} 23 virtual void Show() {}
23 virtual void Close() {} 24 virtual void Close() {}
24 virtual void UpdateTitleBar() {} 25 virtual void UpdateTitleBar() {}
25 virtual void ActivationChanged() {} 26 virtual void ActivationChanged() {}
27 virtual bool ShouldForceHideToolbar() { return false; }
28 virtual void ToggleCompactNavigationBar() {}
26 29
27 DISALLOW_COPY_AND_ASSIGN(StandardExtender); 30 DISALLOW_COPY_AND_ASSIGN(StandardExtender);
28 }; 31 };
29 32
30 } // namespace 33 } // namespace
31 34
32 //////////////////////////////////////////////////////////////////////////////// 35 ////////////////////////////////////////////////////////////////////////////////
33 // BrowserExtender, public: 36 // BrowserExtender, public:
34 37
35 // static 38 // static
36 BrowserExtender* BrowserExtender::Create(BrowserView* browser_view) { 39 BrowserExtender* BrowserExtender::Create(BrowserView* browser_view) {
37 return new StandardExtender(browser_view); 40 return new StandardExtender(browser_view);
38 } 41 }
OLDNEW
« no previous file with comments | « chrome/browser/views/frame/browser_view.cc ('k') | chrome/browser/views/location_bar_view.h » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698