ExternalTabContainer should subclass WidgetWin rather than Widget and ATL CWi...

chrome/browser/external_tab_container.cc

 // Copyright (c) 2006-2008 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/external_tab_container.h"
 
 #include "app/win_util.h"
 #include "base/logging.h"
 #include "base/win_util.h"
 #include "chrome/browser/automation/automation_provider.h"
 #include "chrome/browser/browser.h"
 #include "chrome/browser/extensions/extension_function_dispatcher.h"
 #include "chrome/browser/load_notification_details.h"
 #include "chrome/browser/profile.h"
 #include "chrome/browser/tab_contents/provisional_load_details.h"
 #include "chrome/browser/tab_contents/tab_contents.h"
 #include "chrome/browser/views/tab_contents/tab_contents_container.h"
 #include "chrome/common/chrome_constants.h"
 #include "chrome/common/notification_service.h"
 #include "chrome/test/automation/automation_messages.h"
-// Included for SetRootViewForHWND.
-#include "views/widget/widget_win.h"
 
 static const wchar_t kWindowObjectKey[] = L"ChromeWindowObject";
 
 // TODO(sanjeevr): The external_accel_table_ and external_accel_entry_count_
 // member variables are now obsolete and we don't use them.
 // We need to remove them.
 ExternalTabContainer::ExternalTabContainer(
     AutomationProvider* automation)
     : automation_(automation),
-      root_view_(this),
       tab_contents_(NULL),
       external_accel_table_(NULL),
       external_accel_entry_count_(0),
       tab_contents_container_(NULL),
       tab_handle_(0),
       ignore_next_load_notification_(false) {
 }
 
 ExternalTabContainer::~ExternalTabContainer() {
-  Uninitialize(m_hWnd);
+  Uninitialize(GetNativeView());
 }
 
-bool ExternalTabContainer::Init(Profile* profile, HWND parent,
-                                const gfx::Rect& dimensions,
-                                unsigned int style) {
+bool ExternalTabContainer::Init(Profile* profile,
+                                HWND parent,
+                                const gfx::Rect& bounds,
+                                DWORD style) {
   if (IsWindow()) {
     NOTREACHED();
     return false;
   }
 
-  // First create the container window
-  if (!Create(NULL, dimensions.ToRECT())) {
-    NOTREACHED();
-    return false;
-  }
+  set_window_style(WS_POPUP);
+  views::WidgetWin::Init(parent, bounds, true);

[tommi (sloooow) - chröme, 2009/05/27 20:50:29]

this changes the semantics from before where NULL was the initial parent.  Also,
there's no error handling.

 
   // We don't ever remove the prop because the lifetime of this object
   // is the same as the lifetime of the window
-  SetProp(*this, kWindowObjectKey, this);
+  SetProp(GetNativeView(), kWindowObjectKey, this);
 
-  views::SetRootViewForHWND(m_hWnd, &root_view_);
-  // CreateFocusManager will subclass this window and delete the FocusManager
-  // instance when this window goes away.
   views::FocusManager* focus_manager =
-      views::FocusManager::CreateFocusManager(m_hWnd, GetRootView());
-
-  DCHECK(focus_manager);
+      views::FocusManager::GetFocusManager(GetNativeView());
   focus_manager->AddKeystrokeListener(this);
 
   tab_contents_ = new TabContents(profile, NULL, MSG_ROUTING_NONE, NULL);
   tab_contents_->set_delegate(this);
   tab_contents_->render_view_host()->AllowExternalHostBindings();
 
   // Create a TabContentsContainer to handle focus cycling using Tab and
   // Shift-Tab.
   tab_contents_container_ = new TabContentsContainer;
-  root_view_.AddChildView(tab_contents_container_);
+  SetContentsView(tab_contents_container_);
+
   // Note that SetTabContents must be called after AddChildView is called
   tab_contents_container_->ChangeTabContents(tab_contents_);
 
   NavigationController* controller = &tab_contents_->controller();
   registrar_.Add(this, NotificationType::NAV_ENTRY_COMMITTED,
                  Source<NavigationController>(controller));
   registrar_.Add(this, NotificationType::FAIL_PROVISIONAL_LOAD_WITH_ERROR,
                  Source<NavigationController>(controller));
   registrar_.Add(this, NotificationType::LOAD_STOP,
                  Source<NavigationController>(controller));
   NotificationService::current()->Notify(
       NotificationType::EXTERNAL_TAB_CREATED,
       Source<NavigationController>(controller),
       NotificationService::NoDetails());
 
   // We need WS_POPUP to be on the window during initialization, but
   // once initialized we apply the requested style which may or may not
   // include the popup bit.
   // Note that it's important to do this before we call SetParent since
   // during the SetParent call we will otherwise get a WA_ACTIVATE call
   // that causes us to steal the current focus.
-  ModifyStyle(WS_POPUP, style, 0);
+  SetWindowLong(GWL_STYLE, (GetWindowLong(GWL_STYLE) & ~WS_POPUP) | style);

[ananta, 2009/05/27 13:46:24]

Should this be GetWindowLong(GWL_STYLE) | WS_POPUP | style?

[tommi (sloooow) - chröme, 2009/05/27 20:50:29]

I think this is how it should be.  We want to remove the WS_POPUP style at this
point.

 
   // Now apply the parenting and style
   if (parent)
-    SetParent(parent);
+    SetParent(GetNativeView(), parent);
 
   ::ShowWindow(tab_contents_->GetNativeView(), SW_SHOWNA);
   return true;
 }
 
-bool ExternalTabContainer::Uninitialize(HWND window) {
-  if (::IsWindow(window)) {
-    views::FocusManager* focus_manager =
-        views::FocusManager::GetFocusManager(window);
-    if (focus_manager) {
-      focus_manager->RemoveKeystrokeListener(this);
-    }
-  }
-
-  root_view_.RemoveAllChildViews(true);
-  if (tab_contents_) {
-    NotificationService::current()->Notify(
-        NotificationType::EXTERNAL_TAB_CLOSED,
-        Source<NavigationController>(&tab_contents_->controller()),
-        Details<ExternalTabContainer>(this));
-
-    delete tab_contents_;
-    tab_contents_ = NULL;
-  }
-
-  return true;
+void ExternalTabContainer::SetAccelerators(HACCEL accel_table,
+                                           int accel_table_entry_count) {
+  external_accel_table_ = accel_table;
+  external_accel_entry_count_ = accel_table_entry_count;
 }
 
-void ExternalTabContainer::OnFinalMessage(HWND window) {
-  delete this;
+void ExternalTabContainer::ProcessUnhandledAccelerator(const MSG& msg) {
+  // We just received an accelerator key that we had sent to external host
+  // back. Since the external host was not interested in handling this, we
+  // need to dispatch this message as if we had just peeked this out. (we
+  // also need to call TranslateMessage to generate a WM_CHAR if needed).
+  TranslateMessage(&msg);
+  DispatchMessage(&msg);
 }
 
-LRESULT ExternalTabContainer::OnSize(UINT, WPARAM, LPARAM, BOOL& handled) {
+void ExternalTabContainer::SetInitialFocus(bool reverse) {
+  DCHECK(tab_contents_);
   if (tab_contents_) {
-    RECT client_rect = {0};
-    GetClientRect(&client_rect);
-    ::SetWindowPos(tab_contents_->GetNativeView(), NULL, client_rect.left,
-                   client_rect.top, client_rect.right - client_rect.left,
-                   client_rect.bottom - client_rect.top, SWP_NOZORDER);
+    static_cast<TabContents*>(tab_contents_)->Focus();
+    static_cast<TabContents*>(tab_contents_)->SetInitialFocus(reverse);
   }
-  return 0;
 }
 
-LRESULT ExternalTabContainer::OnDestroy(UINT, WPARAM, LPARAM, BOOL& handled) {
-  Uninitialize(m_hWnd);
-  return 0;
+// static
+bool ExternalTabContainer::IsExternalTabContainer(HWND window) {
+  std::wstring class_name = win_util::GetClassName(window);
+  return _wcsicmp(class_name.c_str(), chrome::kExternalTabWindowClass) == 0;

[tommi (sloooow) - chröme, 2009/05/27 20:50:29]

maybe
return class_name.compare(chrome::kExternalTabWindowClass) == 0;

I think it's safe to be case sensitive here.

 }
 
+// static
+ExternalTabContainer* ExternalTabContainer::GetContainerForTab(
+    HWND tab_window) {
+  HWND parent_window = ::GetParent(tab_window);
+  if (!::IsWindow(parent_window)) {
+    return NULL;
+  }
+  if (!IsExternalTabContainer(parent_window)) {
+    return NULL;
+  }
+  ExternalTabContainer* container = reinterpret_cast<ExternalTabContainer*>(
+      GetProp(parent_window, kWindowObjectKey));
+  return container;
+}
+
+////////////////////////////////////////////////////////////////////////////////
+// ExternalTabContainer, TabContentsDelegate implementation:
+
 void ExternalTabContainer::OpenURLFromTab(TabContents* source,
                            const GURL& url,
                            const GURL& referrer,
                            WindowOpenDisposition disposition,
                            PageTransition::Type transition) {
   switch (disposition) {
     case CURRENT_TAB:
     case SINGLETON_TAB:
     case NEW_FOREGROUND_TAB:
     case NEW_BACKGROUND_TAB:
@@ skipping 70 matching lines @@
 void ExternalTabContainer::ForwardMessageToExternalHost(
     const std::string& message, const std::string& origin,
     const std::string& target) {
   if(automation_) {
     automation_->Send(
         new AutomationMsg_ForwardMessageToExternalHost(0, tab_handle_,
             message, origin, target));
   }
 }
 
+ExtensionFunctionDispatcher* ExternalTabContainer::
+    CreateExtensionFunctionDispatcher(RenderViewHost* render_view_host,
+                                      const std::string& extension_id) {
+  return new ExtensionFunctionDispatcher(render_view_host, NULL, extension_id);
+}
+
 bool ExternalTabContainer::TakeFocus(bool reverse) {
   if (automation_) {
     views::FocusManager* focus_manager =
         views::FocusManager::GetFocusManager(GetNativeView());
     DCHECK(focus_manager);
     if (focus_manager) {
       focus_manager->ClearFocus();
       automation_->Send(new AutomationMsg_TabbedOut(0, tab_handle_,
           win_util::IsShiftPressed()));
     }
   }
 
   return true;
 }
 
+////////////////////////////////////////////////////////////////////////////////
+// ExternalTabContainer, NotificationObserver implementation:
+
 void ExternalTabContainer::Observe(NotificationType type,
                                    const NotificationSource& source,
                                    const NotificationDetails& details) {
   if (!automation_)
     return;
 
   static const int kHttpClientErrorStart = 400;
   static const int kHttpServerErrorEnd = 510;
 
   switch (type.value) {
@@ skipping 40 matching lines @@
           0, tab_handle_, load_details->error_code(), load_details->url()));
 
       ignore_next_load_notification_ = true;
       break;
     }
     default:
       NOTREACHED();
   }
 }
 
-void ExternalTabContainer::GetBounds(gfx::Rect* out,
-                                     bool including_frame) const {
-  CRect crect;
-  GetWindowRect(&crect);
-  *out = gfx::Rect(crect);
+////////////////////////////////////////////////////////////////////////////////
+// ExternalTabContainer, views::WidgetWin overrides:
+
+void ExternalTabContainer::OnDestroy() {
+  Uninitialize(GetNativeView());
+  WidgetWin::OnDestroy();
 }
 
-void ExternalTabContainer::SetBounds(const gfx::Rect& bounds) {
-  SetBounds(bounds, NULL);
-}
-
-void ExternalTabContainer::SetBounds(const gfx::Rect& bounds,
-                                     gfx::NativeView other_window) {
-  NOTIMPLEMENTED();
-}
-
-void ExternalTabContainer::Close() {
-  NOTIMPLEMENTED();
-}
-
-void ExternalTabContainer::CloseNow() {
-  NOTIMPLEMENTED();
-}
-
-void ExternalTabContainer::Show() {
-  NOTIMPLEMENTED();
-}
-
-void ExternalTabContainer::Hide() {
-  NOTIMPLEMENTED();
-}
-
-gfx::NativeView ExternalTabContainer::GetNativeView() const {
-  return m_hWnd;
-}
-
-void ExternalTabContainer::PaintNow(const gfx::Rect& update_rect) {
-  RECT native_update_rect = update_rect.ToRECT();
-  RedrawWindow(&native_update_rect,
-               NULL,
-               RDW_INVALIDATE | RDW_ALLCHILDREN | RDW_NOERASE);
-}
-
-void ExternalTabContainer::SetOpacity(unsigned char opacity) {
-  NOTIMPLEMENTED();
-}
-
-views::RootView* ExternalTabContainer::GetRootView() {
-  return const_cast<views::RootView*>(&root_view_);
-}
-
-bool ExternalTabContainer::IsVisible() const {
-  return !!::IsWindowVisible(*this);
-}
-
-bool ExternalTabContainer::IsActive() const {
-  return win_util::IsWindowActive(*this);
-}
+////////////////////////////////////////////////////////////////////////////////
+// ExternalTabContainer, views::KeystrokeListener implementation:
 
 bool ExternalTabContainer::ProcessKeyStroke(HWND window, UINT message,
                                             WPARAM wparam, LPARAM lparam) {
   if (!automation_) {
     return false;
   }
   if ((wparam == VK_TAB) && !win_util::IsCtrlPressed()) {
     // Tabs are handled separately (except if this is Ctrl-Tab or
     // Ctrl-Shift-Tab)
     return false;
@@ skipping 18 matching lines @@
     msg.message = message;
     msg.wParam = wparam;
     msg.lParam = lparam;
     automation_->Send(new AutomationMsg_HandleAccelerator(0, tab_handle_, msg));
     return true;
   }
 
   return false;
 }
 
-void ExternalTabContainer::SetAccelerators(HACCEL accel_table,
-                                           int accel_table_entry_count) {
-  external_accel_table_ = accel_table;
-  external_accel_entry_count_ = accel_table_entry_count;
-}
+////////////////////////////////////////////////////////////////////////////////
+// ExternalTabContainer, private:
 
-void ExternalTabContainer::ProcessUnhandledAccelerator(const MSG& msg) {
-  // We just received an accelerator key that we had sent to external host
-  // back. Since the external host was not interested in handling this, we
-  // need to dispatch this message as if we had just peeked this out. (we
-  // also need to call TranslateMessage to generate a WM_CHAR if needed).
-  TranslateMessage(&msg);
-  DispatchMessage(&msg);
-}
+void ExternalTabContainer::Uninitialize(HWND window) {
+  if (::IsWindow(window)) {
+    views::FocusManager* focus_manager =
+        views::FocusManager::GetFocusManager(window);
+    if (focus_manager)
+      focus_manager->RemoveKeystrokeListener(this);
+  }
 
-void ExternalTabContainer::SetInitialFocus(bool reverse) {
-  DCHECK(tab_contents_);
   if (tab_contents_) {
-    static_cast<TabContents*>(tab_contents_)->Focus();
-    static_cast<TabContents*>(tab_contents_)->SetInitialFocus(reverse);
+    NotificationService::current()->Notify(
+        NotificationType::EXTERNAL_TAB_CLOSED,
+        Source<NavigationController>(&tab_contents_->controller()),
+        Details<ExternalTabContainer>(this));
+
+    delete tab_contents_;
+    tab_contents_ = NULL;
   }
 }
 
-// static
-bool ExternalTabContainer::IsExternalTabContainer(HWND window) {
-  std::wstring class_name = win_util::GetClassName(window);
-  return _wcsicmp(class_name.c_str(), chrome::kExternalTabWindowClass) == 0;
-}
-
-ExtensionFunctionDispatcher* ExternalTabContainer::
-    CreateExtensionFunctionDispatcher(RenderViewHost* render_view_host,
-                                      const std::string& extension_id) {
-  return new ExtensionFunctionDispatcher(render_view_host, NULL, extension_id);
-}
-
-// static
-ExternalTabContainer* ExternalTabContainer::GetContainerForTab(
-    HWND tab_window) {
-  HWND parent_window = ::GetParent(tab_window);
-  if (!::IsWindow(parent_window)) {
-    return NULL;
-  }
-  if (!IsExternalTabContainer(parent_window)) {
-    return NULL;
-  }
-  ExternalTabContainer* container = reinterpret_cast<ExternalTabContainer*>(
-      GetProp(parent_window, kWindowObjectKey));
-  return container;
-}