Add in DOMBrowserView and Frame related classes

chrome/browser/chromeos/webui/login/browser/dom_browser_view.cc

Index: chrome/browser/chromeos/webui/login/browser/dom_browser_view.cc
diff --git a/chrome/browser/chromeos/webui/login/browser/dom_browser_view.cc b/chrome/browser/chromeos/webui/login/browser/dom_browser_view.cc
new file mode 100644
index 0000000000000000000000000000000000000000..8dc43f4c7a175611a069781e29c1740cf86e6896
--- /dev/null
+++ b/chrome/browser/chromeos/webui/login/browser/dom_browser_view.cc
@@ -0,0 +1,128 @@
+// 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.
+
+#include "chrome/browser/chromeos/webui/login/browser/dom_browser_view.h"
+
+#include "base/command_line.h"
+#include "chrome/browser/chromeos/status/status_area_view.h"
+#include "chrome/browser/chromeos/view_ids.h"
+#include "chrome/browser/chromeos/webui/login/browser/dom_browser_view_layout.h"
+#include "chrome/browser/chromeos/webui/login/frame/dom_browser_frame.h"
+#include "chrome/browser/chromeos/wm_ipc.h"
+#include "chrome/browser/profiles/profile.h"
+#include "chrome/browser/sidebar/sidebar_manager.h"
+#include "chrome/browser/themes/browser_theme_provider.h"
+#include "chrome/browser/ui/views/browser_dialogs.h"
+#include "chrome/browser/ui/views/frame/browser_frame.h"
+#include "chrome/browser/ui/views/frame/browser_view.h"
+#include "chrome/browser/ui/views/frame/contents_container.h"
+#include "chrome/browser/ui/views/status_bubble_views.h"
+#include "chrome/browser/ui/views/toolbar_view.h"
+#include "chrome/common/chrome_switches.h"
+#include "grit/chromium_strings.h"
+#include "grit/generated_resources.h"
+#include "ui/base/l10n/l10n_util.h"
+#include "ui/base/theme_provider.h"
+#include "ui/views/layout/layout_manager.h"

[oshima, 2011/03/16 21:10:25]

I believe some of these includes are not necessary (like browser_dialog,
status_bubble_view, toolbar_view). Can you check and eliminate them?

[rharrison, 2011/03/17 17:00:50]

Done.

+
+namespace chromeos {
+
+// DOMBrowserView, public

[oshima, 2011/03/16 21:10:25]

// DOMBrowserView, public ------------------

and for the rest (if any)

[rharrison, 2011/03/17 17:00:50]

Done.

+
+DOMBrowserView::DOMBrowserView(Browser* browser)
+    : chromeos::BrowserView(browser) {}
+
+DOMBrowserView::~DOMBrowserView() {}
+
+// static
+BrowserWindow* DOMBrowserView::CreateDOMWindow(Browser* browser) {
+  // Create a browser view for chromeos.

[Nikita (slow), 2011/03/16 21:33:45]

Move this comment to .h file and make more specific.

[rharrison, 2011/03/17 17:00:50]

Done.

+  DOMBrowserView* view = new DOMBrowserView(browser);
+  DOMBrowserFrame::Create(view, browser->profile());
+  return view;
+}
+
+void DOMBrowserView::WindowMoveOrResizeStarted() {}
+
+gfx::Rect DOMBrowserView::GetToolbarBounds() const {
+  return gfx::Rect(0, 0, 0, 0);

[oshima, 2011/03/16 21:10:25]

just gfx::Rect()

[rharrison, 2011/03/17 17:00:50]

Done.

+}
+
+int DOMBrowserView::GetTabStripHeight() const {
+  return 0;
+}
+
+bool DOMBrowserView::IsTabStripVisible() const {
+  return false;
+}
+
+bool DOMBrowserView::AcceleratorPressed(const views::Accelerator& accelerator) {
+  return false;
+}
+
+void DOMBrowserView::SetStarredState(bool is_starred) {}
+
+LocationBar* DOMBrowserView::GetLocationBar() const {
+  return NULL;
+}
+
+void DOMBrowserView::SetFocusToLocationBar(bool select_all) {}
+
+void DOMBrowserView::UpdateReloadStopState(bool is_loading, bool force) {}
+
+void DOMBrowserView::UpdateToolbar(TabContentsWrapper* contents,
+                                bool should_restore_state) {}
+
+void DOMBrowserView::FocusToolbar() {}
+
+void DOMBrowserView::FocusAppMenu() {}
+
+void DOMBrowserView::ShowBookmarkBubble(const GURL& url,
+                                        bool already_bookmarked) {}
+
+void DOMBrowserView::ShowAppMenu() {}
+
+LocationBarView* DOMBrowserView::GetLocationBarView() const {
+  return NULL;
+}
+
+ToolbarView* DOMBrowserView::GetToolbarView() const {
+  return NULL;
+}
+
+bool DOMBrowserView::ShouldShowOffTheRecordAvatar() const {
+  return false;
+}
+
+bool DOMBrowserView::GetAcceleratorForCommandId(int command_id,
+                                                ui::Accelerator* accelerator) {
+  return GetWidget()->GetAccelerator(command_id, accelerator);
+}
+
+bool DOMBrowserView::IsToolbarVisible() const {
+  return false;
+}
+
+// BrowserView, protected

[Nikita (slow), 2011/03/16 21:33:45]

Please follow suggested comment style
// ... --------

[Nikita (slow), 2011/03/16 21:33:45]

BrowserView > DOMBrowserView

[rharrison, 2011/03/17 17:00:50]

Done.

[rharrison, 2011/03/17 17:00:50]

Done.

+
+void DOMBrowserView::GetAccessiblePanes(
+    std::vector<AccessiblePaneView*>* panes) {}
+
+void DOMBrowserView::PaintChildren(gfx::Canvas* canvas) {
+  views::ClientView::PaintChildren(canvas);
+}
+
+void DOMBrowserView::InitTabStrip(TabStripModel* model) {}
+
+views::LayoutManager* DOMBrowserView::CreateLayoutManager() const {
+  return new DOMBrowserViewLayout;
+}
+
+ToolbarView* DOMBrowserView::CreateToolbar() const {
+  return NULL;
+}
+
+void DOMBrowserView::LoadingAnimationCallback() {}
+
+}  // namespace chromeos