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

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

Issue 12036075: alternate ntp: fix website page jankiness when suggestions show and top bars are hidden (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src
Patch Set: remove redundant param Created 7 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 | Annotate | Revision Log
OLDNEW
1 // Copyright 2012 The Chromium Authors. All rights reserved. 1 // Copyright 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_CONTENTS_CONTAINER_H_ 5 #ifndef CHROME_BROWSER_UI_VIEWS_FRAME_CONTENTS_CONTAINER_H_
6 #define CHROME_BROWSER_UI_VIEWS_FRAME_CONTENTS_CONTAINER_H_ 6 #define CHROME_BROWSER_UI_VIEWS_FRAME_CONTENTS_CONTAINER_H_
7 7
8 #include <string> 8 #include <string>
9 9
10 #include "base/basictypes.h" 10 #include "base/basictypes.h"
(...skipping 38 matching lines...) Expand 10 before | Expand all | Expand 10 after
49 bool draw_drop_shadow); 49 bool draw_drop_shadow);
50 50
51 // When the active content is reset and we have a visible preview, 51 // When the active content is reset and we have a visible preview,
52 // the preview must be stacked back at top. 52 // the preview must be stacked back at top.
53 void MaybeStackPreviewAtTop(); 53 void MaybeStackPreviewAtTop();
54 54
55 content::WebContents* preview_web_contents() const { 55 content::WebContents* preview_web_contents() const {
56 return preview_web_contents_; 56 return preview_web_contents_;
57 } 57 }
58 58
59 int preview_height() const;
dhollowa 2013/01/25 21:24:08 nit: since this is snake case (trivial) better to
Peter Kasting 2013/01/28 21:51:00 Anything named unix_hacker()-style should be inlin
kuan 2013/01/29 00:17:08 Done.
kuan 2013/01/29 00:17:08 Done.
60
59 // Sets the active top margin. 61 // Sets the active top margin.
60 void SetActiveTopMargin(int margin); 62 void SetActiveTopMargin(int margin);
61 63
62 // Returns the bounds the preview would be shown at. 64 // Returns the bounds the preview would be shown at.
63 gfx::Rect GetPreviewBounds() const; 65 gfx::Rect GetPreviewBounds() const;
64 66
65 // Returns true if preview occupies full height of content page. 67 // Returns true if preview occupies full height of content page.
66 bool IsPreviewFullHeight(int preview_height, 68 bool IsPreviewFullHeight(int preview_height,
67 InstantSizeUnits preview_height_units) const; 69 InstantSizeUnits preview_height_units) const;
68 70
(...skipping 20 matching lines...) Expand all
89 int active_top_margin_; 91 int active_top_margin_;
90 92
91 // The desired height of the preview and units. 93 // The desired height of the preview and units.
92 int preview_height_; 94 int preview_height_;
93 InstantSizeUnits preview_height_units_; 95 InstantSizeUnits preview_height_units_;
94 96
95 DISALLOW_COPY_AND_ASSIGN(ContentsContainer); 97 DISALLOW_COPY_AND_ASSIGN(ContentsContainer);
96 }; 98 };
97 99
98 #endif // CHROME_BROWSER_UI_VIEWS_FRAME_CONTENTS_CONTAINER_H_ 100 #endif // CHROME_BROWSER_UI_VIEWS_FRAME_CONTENTS_CONTAINER_H_
OLDNEW

Powered by Google App Engine
This is Rietveld 408576698