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

Unified Diff: chrome/browser/ui/views/constrained_window_frame_simple.h

Issue 11192014: Revert 162252 - Make Web Intents picker in Views conform to latest mocks (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src/
Patch Set: Created 8 years, 2 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/constrained_window_frame_simple.h
===================================================================
--- chrome/browser/ui/views/constrained_window_frame_simple.h (revision 162255)
+++ chrome/browser/ui/views/constrained_window_frame_simple.h (working copy)
@@ -6,13 +6,15 @@
#define CHROME_BROWSER_UI_VIEWS_CONSTRAINED_WINDOW_FRAME_SIMPLE_H_
#include "base/memory/scoped_ptr.h"
-#include "chrome/browser/ui/views/constrained_window_views.h"
#include "ui/views/controls/button/button.h"
#include "ui/views/window/non_client_view.h"
+class ConstrainedWindowViews;
+
namespace views {
class ImageButton;
class Label;
+class LayoutManager;
}
////////////////////////////////////////////////////////////////////////////////
@@ -22,11 +24,24 @@
class ConstrainedWindowFrameSimple : public views::NonClientFrameView,
public views::ButtonListener {
public:
- explicit ConstrainedWindowFrameSimple(
- ConstrainedWindowViews* container,
- ConstrainedWindowViews::ChromeStyleClientInsets client_insets);
+ // Contains references to relevant views in the header. The header
+ // must be non-NULL.
+ struct HeaderViews {
+ HeaderViews(views::View* header,
+ views::Label* title_label,
+ views::Button* close_button);
+
+ views::View* header;
+ views::Label* title_label;
+ views::Button* close_button;
+ };
+
+ explicit ConstrainedWindowFrameSimple(ConstrainedWindowViews* container);
virtual ~ConstrainedWindowFrameSimple();
+ // SetHeaderView assumes ownership of the passed parameter.
+ void SetHeaderView(HeaderViews* header_views);
+
private:
// Overridden from views::NonClientFrameView:
virtual gfx::Rect GetBoundsForClientView() const OVERRIDE;
@@ -46,10 +61,16 @@
virtual void ButtonPressed(views::Button* sender,
const ui::Event& event) OVERRIDE;
+ HeaderViews* CreateDefaultHeaderView();
+
+ views::ImageButton* CreateCloseButton();
+
ConstrainedWindowViews* container_;
- views::Label* title_label_;
- views::ImageButton* close_button_;
+ views::LayoutManager* layout_;
+
+ scoped_ptr<HeaderViews> header_views_;
+
DISALLOW_COPY_AND_ASSIGN(ConstrainedWindowFrameSimple);
};
#endif // CHROME_BROWSER_UI_VIEWS_CONSTRAINED_WINDOW_FRAME_SIMPLE_H_

Powered by Google App Engine
This is Rietveld 408576698