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

Side by Side Diff: chrome/browser/ui/views/frame/opaque_browser_frame_view.h

Issue 1624773002: Clean up frame code a bit: (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: Resync, fix compile 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 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 #ifndef CHROME_BROWSER_UI_VIEWS_FRAME_OPAQUE_BROWSER_FRAME_VIEW_H_ 5 #ifndef CHROME_BROWSER_UI_VIEWS_FRAME_OPAQUE_BROWSER_FRAME_VIEW_H_
6 #define CHROME_BROWSER_UI_VIEWS_FRAME_OPAQUE_BROWSER_FRAME_VIEW_H_ 6 #define CHROME_BROWSER_UI_VIEWS_FRAME_OPAQUE_BROWSER_FRAME_VIEW_H_
7 7
8 #include "base/macros.h" 8 #include "base/macros.h"
9 #include "base/memory/scoped_ptr.h" 9 #include "base/memory/scoped_ptr.h"
10 #include "chrome/browser/ui/view_ids.h" 10 #include "chrome/browser/ui/view_ids.h"
(...skipping 122 matching lines...) Expand 10 before | Expand all | Expand 10 after
133 // Returns the bounds of the titlebar icon (or where the icon would be if 133 // Returns the bounds of the titlebar icon (or where the icon would be if
134 // there was one). 134 // there was one).
135 gfx::Rect IconBounds() const; 135 gfx::Rect IconBounds() const;
136 136
137 // Returns true if the view should draw its own custom title bar. 137 // Returns true if the view should draw its own custom title bar.
138 bool ShouldShowWindowTitleBar() const; 138 bool ShouldShowWindowTitleBar() const;
139 139
140 // Paint various sub-components of this view. The *FrameBorder() functions 140 // Paint various sub-components of this view. The *FrameBorder() functions
141 // also paint the background of the titlebar area, since the top frame border 141 // also paint the background of the titlebar area, since the top frame border
142 // and titlebar background are a contiguous component. 142 // and titlebar background are a contiguous component.
143 void PaintRestoredFrameBorder(gfx::Canvas* canvas); 143 void PaintRestoredFrameBorder(gfx::Canvas* canvas) const;
144 void PaintMaximizedFrameBorder(gfx::Canvas* canvas); 144 void PaintMaximizedFrameBorder(gfx::Canvas* canvas) const;
145 void PaintToolbarBackground(gfx::Canvas* canvas); 145 void PaintToolbarBackground(gfx::Canvas* canvas) const;
146 void PaintClientEdge(gfx::Canvas* canvas); 146 void PaintClientEdge(gfx::Canvas* canvas) const;
147 void FillClientEdgeRects(int x, 147 void FillClientEdgeRects(int x,
148 int y, 148 int y,
149 int right, 149 int right,
150 int bottom, 150 int bottom,
151 SkColor color, 151 SkColor color,
152 gfx::Canvas* canvas) const; 152 gfx::Canvas* canvas) const;
153 153
154 // Our layout manager also calculates various bounds. 154 // Our layout manager also calculates various bounds.
155 OpaqueBrowserFrameViewLayout* layout_; 155 OpaqueBrowserFrameViewLayout* layout_;
156 156
(...skipping 10 matching lines...) Expand all
167 // Background painter for the window frame. 167 // Background painter for the window frame.
168 scoped_ptr<views::FrameBackground> frame_background_; 168 scoped_ptr<views::FrameBackground> frame_background_;
169 169
170 // Observer that handles platform dependent configuration. 170 // Observer that handles platform dependent configuration.
171 scoped_ptr<OpaqueBrowserFrameViewPlatformSpecific> platform_observer_; 171 scoped_ptr<OpaqueBrowserFrameViewPlatformSpecific> platform_observer_;
172 172
173 DISALLOW_COPY_AND_ASSIGN(OpaqueBrowserFrameView); 173 DISALLOW_COPY_AND_ASSIGN(OpaqueBrowserFrameView);
174 }; 174 };
175 175
176 #endif // CHROME_BROWSER_UI_VIEWS_FRAME_OPAQUE_BROWSER_FRAME_VIEW_H_ 176 #endif // CHROME_BROWSER_UI_VIEWS_FRAME_OPAQUE_BROWSER_FRAME_VIEW_H_
OLDNEW
« no previous file with comments | « chrome/browser/ui/views/frame/glass_browser_frame_view.cc ('k') | chrome/browser/ui/views/frame/opaque_browser_frame_view.cc » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698