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

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

Issue 6904160: Implement new gray mock. (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src
Patch Set: Fixed power test Created 9 years, 8 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/opaque_browser_frame_view.h
diff --git a/chrome/browser/ui/views/frame/opaque_browser_frame_view.h b/chrome/browser/ui/views/frame/opaque_browser_frame_view.h
index 7305a061d4417772abb654f7c088a1de00557484..8c56b83a43080c317089be06ed853b320059cf16 100644
--- a/chrome/browser/ui/views/frame/opaque_browser_frame_view.h
+++ b/chrome/browser/ui/views/frame/opaque_browser_frame_view.h
@@ -75,6 +75,23 @@ class OpaqueBrowserFrameView : public BrowserNonClientFrameView,
virtual bool ShouldTabIconViewAnimate() const OVERRIDE;
virtual SkBitmap GetFaviconForTabIconView() OVERRIDE;
+ // Returns the height of the entire nonclient top border, including the window
sky 2011/05/02 14:26:27 Most these before other overrides in the section.
+ // frame, any title area, and any connected client edge. If |restored| is
+ // true, acts as if the window is restored regardless of the real mode. If
+ // |ignore_vertical_tabs| is true, acts as if vertical tabs are off regardless
+ // of the real state.
+ int NonClientTopBorderHeight(bool restored, bool ignore_vertical_tabs) const;
+
+ // Allows a subclass to tweak the frame. Chromeos uses this to support
+ // drawing themes correctly even with extra padding above the tabs as
+ // well as drawing custom corners on the frame.
+ virtual void ModifyMaximizedFramePainting(
+ int* theme_offset, SkBitmap** left_corner, SkBitmap** right_corner);
sky 2011/05/02 14:26:27 each param on its own line. Also, document what th
+
+ // Expose these to subclasses.
+ BrowserFrame* frame() { return frame_; }
+ BrowserView* browser_view() { return browser_view_; }
+
private:
// Returns the thickness of the border that makes up the window frame edges.
// This does not include any client edge. If |restored| is true, acts as if
@@ -89,13 +106,6 @@ class OpaqueBrowserFrameView : public BrowserNonClientFrameView,
// borders, including both the window frame and any client edge.
int NonClientBorderThickness() const;
- // Returns the height of the entire nonclient top border, including the window
- // frame, any title area, and any connected client edge. If |restored| is
- // true, acts as if the window is restored regardless of the real mode. If
- // |ignore_vertical_tabs| is true, acts as if vertical tabs are off regardless
- // of the real state.
- int NonClientTopBorderHeight(bool restored, bool ignore_vertical_tabs) const;
sky 2011/05/02 14:26:27 move implementation to match new position in heade
-
// Returns the y-coordinate of the caption buttons. If |restored| is true,
// acts as if the window is restored regardless of the real mode.
int CaptionButtonY(bool restored) const;

Powered by Google App Engine
This is Rietveld 408576698