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

Unified Diff: chrome/browser/chromeos/webui/login/browser/dom_browser_view_layout.h

Issue 6692001: Add in DOMBrowserView and Frame related classes (Closed) Base URL: http://git.chromium.org/git/chromium.git@trunk
Patch Set: Updated CL in response to comments by oshima and rjkroege Created 9 years, 9 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/chromeos/webui/login/browser/dom_browser_view_layout.h
diff --git a/chrome/browser/chromeos/webui/login/browser/dom_browser_view_layout.h b/chrome/browser/chromeos/webui/login/browser/dom_browser_view_layout.h
new file mode 100644
index 0000000000000000000000000000000000000000..60885989e842ff53185fb9907a53061ddc9acc48
--- /dev/null
+++ b/chrome/browser/chromeos/webui/login/browser/dom_browser_view_layout.h
@@ -0,0 +1,48 @@
+// Copyright (c) 2011 The Chromium Authors. All rights reserved.
+// Use of this source code is governed by a BSD-style license that can be
+// found in the LICENSE file.
+
+#ifndef CHROME_BROWSER_CHROMEOS_WEBUI_LOGIN_BROWSER_DOM_BROWSER_VIEW_LAYOUT_H_
+#define CHROME_BROWSER_CHROMEOS_WEBUI_LOGIN_BROWSER_DOM_BROWSER_VIEW_LAYOUT_H_
+#pragma once
+
+#include "base/compiler_specific.h"
+#include "chrome/browser/chromeos/webui/login/browser/dom_browser_view.h"
+#include "chrome/browser/chromeos/status/status_area_host.h"
oshima 2011/03/16 21:10:25 These two includes can be replaced with forward de
rharrison 2011/03/17 17:00:50 Done.
+#include "chrome/browser/ui/views/frame/browser_view_layout.h"
+
+namespace chromeos {
+
+// LayoutManager for DOMBrowserView, which layouts extra components such as
+// the status views
Nikita (slow) 2011/03/16 21:33:45 nit: dot at the end.
rharrison 2011/03/17 17:00:50 Done.
+class DOMBrowserViewLayout : public ::BrowserViewLayout {
+ public:
+ DOMBrowserViewLayout() : ::BrowserViewLayout() {}
Nikita (slow) 2011/03/16 21:33:45 Move ctor body into .cc file.
rharrison 2011/03/17 17:00:50 Done.
+ virtual ~DOMBrowserViewLayout() {}
Nikita (slow) 2011/03/16 21:33:45 Move body to .cc file.
rharrison 2011/03/17 17:00:50 Done.
+
+ // BrowserViewLayout overrides:
+ virtual void Installed(views::View* host) OVERRIDE;
+ virtual void ViewAdded(views::View* host,
+ views::View* view) OVERRIDE;
+ virtual bool IsPositionInWindowCaption(const gfx::Point& point) OVERRIDE;
+ virtual int NonClientHitTest(const gfx::Point& point) OVERRIDE;
+
+ protected:
+ // BrowserViewLayout overrides:
+ virtual int LayoutTabStrip() OVERRIDE;
+ virtual int LayoutToolbar(int top) OVERRIDE;
+ virtual int LayoutBookmarkAndInfoBars(int top) OVERRIDE;
+
+ private:
+ DOMBrowserView* GetDOMBrowserView();
+ chromeos::StatusAreaView* status_area_;
+
+ bool IsPointInViewsInTitleArea(const gfx::Point& point) const;
+ int LayoutTitlebarComponents(const gfx::Rect& bounds);
+
+ DISALLOW_COPY_AND_ASSIGN(DOMBrowserViewLayout);
+};
+
+} // namespace chromeos
+
+#endif // CHROME_BROWSER_CHROMEOS_WEBUI_LOGIN_BROWSER_DOM_BROWSER_VIEW_LAYOUT_H_

Powered by Google App Engine
This is Rietveld 408576698