Change PluginInstallImpl to use base::WindowImpl instead of CWindowImpl to reduce a dependency on ATL.

BUG=5023
TEST=Uninstall flash.  Visit hulu.com and install the flash plugin.
Review URL: http://codereview.chromium.org/165469

git-svn-id: svn://svn.chromium.org/chrome/trunk/src@23406 0039d316-1c4b-4281-b951-d872f2087c98

2 files changed, 24 insertions, 28 deletions

diff --git a/base/window_impl.cc b/base/window_impl.cc
index 4a37f7b..c3b3ad4 100644
--- a/base/window_impl.cc
+++ b/base/window_impl.cc
@@ -96,7 +96,7 @@ class ClassRegistrar {
   typedef std::list<RegisteredClass> RegisteredClasses;
   RegisteredClasses registered_classes_;
 
-  // Counter of how many classes have ben registered so far.
+  // Counter of how many classes have been registered so far.
   int registered_count_;
 
   DISALLOW_COPY_AND_ASSIGN(ClassRegistrar);
@@ -136,7 +136,7 @@ void WindowImpl::Init(HWND parent, const gfx::Rect& bounds) {
     height = bounds.height();
   }
 
-  hwnd_ = CreateWindowEx(window_ex_style_, GetWindowClassName().c_str(), L"",
+  hwnd_ = CreateWindowEx(window_ex_style_, GetWindowClassName().c_str(), NULL,
                          window_style_, x, y, width, height,
                          parent, NULL, NULL, this);
   DCHECK(hwnd_);
@@ -145,27 +145,17 @@ void WindowImpl::Init(HWND parent, const gfx::Rect& bounds) {
   DCHECK(win_util::GetWindowUserData(hwnd_) == this);
 }
 
-gfx::NativeView WindowImpl::GetNativeView() const {
-  return hwnd_;
-}
-
 HICON WindowImpl::GetDefaultWindowIcon() const {
   return NULL;
 }
 
-BOOL WindowImpl::DestroyWindow() {
-  DCHECK(::IsWindow(GetNativeView()));
-  return ::DestroyWindow(GetNativeView());
-}
-
 LRESULT WindowImpl::OnWndProc(UINT message, WPARAM w_param, LPARAM l_param) {
-  HWND window = GetNativeView();
   LRESULT result = 0;
 
   // Handle the message if it's in our message map; otherwise, let the system
   // handle it.
-  if (!ProcessWindowMessage(window, message, w_param, l_param, result))
-    result = DefWindowProc(window, message, w_param, l_param);
+  if (!ProcessWindowMessage(hwnd_, message, w_param, l_param, result))
+    result = DefWindowProc(hwnd_, message, w_param, l_param);
 
   return result;
 }
diff --git a/base/window_impl.h b/base/window_impl.h
index 30c6298..2941c3d9 100644
--- a/base/window_impl.h
+++ b/base/window_impl.h
@@ -18,6 +18,19 @@
 
 namespace base {
 
+// An interface implemented by classes that use message maps.
+// ProcessWindowMessage is implemented by the BEGIN_MESSAGE_MAP_EX macro.
+class MessageMapInterface {
+ public:
+  // Processes one message from the window's message queue.
+  virtual BOOL ProcessWindowMessage(HWND window,
+                                    UINT message,
+                                    WPARAM w_param,
+                                    LPARAM l_param,
+                                    LRESULT& result,
+                                    DWORD msg_mad_id = 0) = 0;
+};
+
 ///////////////////////////////////////////////////////////////////////////////
 //
 // WindowImpl
@@ -26,24 +39,20 @@ namespace base {
 //  Windows.
 //
 ///////////////////////////////////////////////////////////////////////////////
-class WindowImpl {
+class WindowImpl : public MessageMapInterface {
  public:
   WindowImpl();
   virtual ~WindowImpl();
 
-  BEGIN_MSG_MAP_EX(WindowImpl)
-    // No messages to handle
-  END_MSG_MAP()
-
-  // Initialize the Window with a parent and an initial desired size.
-  virtual void Init(HWND parent, const gfx::Rect& bounds);
-
-  // Returns the gfx::NativeView associated with this Window.
-  virtual gfx::NativeView GetNativeView() const;
+  // Initializes the Window with a parent and an initial desired size.
+  void Init(HWND parent, const gfx::Rect& bounds);
 
   // Retrieves the default window icon to use for windows if none is specified.
   virtual HICON GetDefaultWindowIcon() const;
 
+  // Returns the HWND associated with this Window.
+  HWND hwnd() const { return hwnd_; }
+
   // Sets the window styles. This is ONLY used when the window is created.
   // In other words, if you invoke this after invoking Init, nothing happens.
   void set_window_style(DWORD style) { window_style_ = style; }
@@ -62,16 +71,13 @@ class WindowImpl {
   UINT initial_class_style() { return class_style_; }
 
  protected:
-  // Call close instead of this to Destroy the window.
-  BOOL DestroyWindow();
-
   // Handles the WndProc callback for this object.
   virtual LRESULT OnWndProc(UINT message, WPARAM w_param, LPARAM l_param);
 
  private:
   friend class ClassRegistrar;
 
-  // The windows procedure used by all Windows.
+  // The window procedure used by all Windows.
   static LRESULT CALLBACK WndProc(HWND window,
                                   UINT message,
                                   WPARAM w_param,