Implement ShellWindow in Views.

chrome/browser/ui/views/extensions/shell_window_views.h

Index: chrome/browser/ui/views/extensions/shell_window_views.h
diff --git a/chrome/browser/ui/views/extensions/shell_window_views.h b/chrome/browser/ui/views/extensions/shell_window_views.h
new file mode 100644
index 0000000000000000000000000000000000000000..472a89d7806a347dbd55905bc1ea6e39f52332f4
--- /dev/null
+++ b/chrome/browser/ui/views/extensions/shell_window_views.h
@@ -0,0 +1,38 @@
+// Copyright (c) 2011 The Chromium Authors. All rights reserved.

[Mihai Parparita -not on Chrome, 2012/01/12 02:09:09]

Presubmit check will most likely yell about non-2012 copyright dates.

[jeremya, 2012/01/12 18:28:46]

My local one didn't... but done anyway :)

+// Use of this source code is governed by a BSD-style license that can be
+// found in the LICENSE file.
+
+#ifndef CHROME_BROWSER_UI_VIEWS_EXTENSIONS_SHELL_WINDOW_VIEWS_H_
+#define CHROME_BROWSER_UI_VIEWS_EXTENSIONS_SHELL_WINDOW_VIEWS_H_
+#pragma once
+
+#include "chrome/browser/ui/extensions/shell_window.h"
+#include "chrome/browser/ui/views/extensions/extension_view.h"
+#include "ui/views/widget/widget_delegate.h"
+
+class ExtensionHost;
+
+class ShellWindowViews : public ShellWindow,
+                         public ExtensionView::Container,
+                         public views::WidgetDelegate {
+ public:
+  explicit ShellWindowViews(ExtensionHost* host);
+
+  // ShellWindow implementation.
+  virtual void Close() OVERRIDE;
+
+  // WidgetDelegate implementation.
+  views::View* GetContentsView() OVERRIDE;
+  bool CanResize() const OVERRIDE;
+  views::Widget* GetWidget() OVERRIDE;
+  const views::Widget* GetWidget() const OVERRIDE;
+  string16 GetWindowTitle() const OVERRIDE;
+  void DeleteDelegate() OVERRIDE;
+
+ private:
+  virtual ~ShellWindowViews();
+
+  views::Widget* window_;

[Mihai Parparita -not on Chrome, 2012/01/12 02:09:09]

Seems like this should be a scoped_ptr

[jeremya, 2012/01/12 18:28:46]

It shouldn't, because the default ownership model of widgets is
NATIVE_WIDGET_OWNS_WIDGET, so the native widget should destroy it when it goes.

+};
+
+#endif  // CHROME_BROWSER_UI_VIEWS_EXTENSIONS_SHELL_WINDOW_VIEWS_H_