Move more platform-specific stuff from WebContents to the view....

chrome/browser/web_contents_view_win.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/web_contents_view_win.h"
 
 #include <windows.h>
 
 #include "chrome/browser/browser_process.h"
 #include "chrome/browser/download/download_request_manager.h"
@@ skipping 146 matching lines @@
   if (info_bar_view_.get() == NULL) {
     // TODO(brettw) currently the InfoBar thinks its owned by the WebContents,
     // but it should instead think it's owned by us.
     info_bar_view_.reset(new InfoBarView(web_contents_));
     // We own the info-bar.
     info_bar_view_->SetParentOwned(false);
   }
   return info_bar_view_.get();
 }
 
+void WebContentsViewWin::SetPageTitle(const std::wstring& title) {
+  if (GetContainerHWND()) {
+    // It's possible to get this after the hwnd has been destroyed.
+    ::SetWindowText(GetContainerHWND(), title.c_str());
+    // TODO(brettw) this call seems messy the way it reaches into the widget
+    // view, and I'm not sure it's necessary. Maybe we should just remove it.
+    ::SetWindowText(web_contents_->render_widget_host_view()->GetPluginHWND(),
+                    title.c_str());
+  }
+}
+
+void WebContentsViewWin::Invalidate() {
+  // Note that it's possible to get this message after the window was destroyed.
+  if (::IsWindow(GetContainerHWND()))
+    InvalidateRect(GetContainerHWND(), NULL, FALSE);
+}
+
 void WebContentsViewWin::UpdateDragCursor(bool is_drop_target) {
   drop_target_->set_is_drop_target(is_drop_target);
 }
 
-void WebContentsViewWin::ShowContextMenu(
-    const ViewHostMsg_ContextMenu_Params& params) {
-  RenderViewContextMenuController menu_controller(web_contents_, params);
-  RenderViewContextMenu menu(&menu_controller,
-                             GetHWND(),
-                             params.type,
-                             params.misspelled_word,
-                             params.dictionary_suggestions,
-                             web_contents_->profile());
+void WebContentsViewWin::TakeFocus(bool reverse) {
+  views::FocusManager* focus_manager =
+      views::FocusManager::GetFocusManager(GetContainerHWND());
 
-  POINT screen_pt = { params.x, params.y };
-  MapWindowPoints(GetHWND(), HWND_DESKTOP, &screen_pt, 1);
-
-  // Enable recursive tasks on the message loop so we can get updates while
-  // the context menu is being displayed.
-  bool old_state = MessageLoop::current()->NestableTasksAllowed();
-  MessageLoop::current()->SetNestableTasksAllowed(true);
-  menu.RunMenuAt(screen_pt.x, screen_pt.y);
-  MessageLoop::current()->SetNestableTasksAllowed(old_state);
+  // We may not have a focus manager if the tab has been switched before this
+  // message arrived.
+  if (focus_manager)
+    focus_manager->AdvanceFocus(reverse);
 }
 
 void WebContentsViewWin::HandleKeyboardEvent(const WebKeyboardEvent& event) {
   // The renderer returned a keyboard event it did not process. This may be
   // a keyboard shortcut that we have to process.
   if (event.type == WebInputEvent::KEY_DOWN) {
     views::FocusManager* focus_manager =
         views::FocusManager::GetFocusManager(GetHWND());
     // We may not have a focus_manager at this point (if the tab has been
     // switched by the time this message returned).
@@ skipping 11 matching lines @@
   }
 
   // Any unhandled keyboard/character messages should be defproced.
   // This allows stuff like Alt+F4, etc to work correctly.
   DefWindowProc(event.actual_message.hwnd,
                 event.actual_message.message,
                 event.actual_message.wParam,
                 event.actual_message.lParam);
 }
 
+void WebContentsViewWin::ShowContextMenu(
+    const ViewHostMsg_ContextMenu_Params& params) {
+  RenderViewContextMenuController menu_controller(web_contents_, params);
+  RenderViewContextMenu menu(&menu_controller,
+                             GetHWND(),
+                             params.type,
+                             params.misspelled_word,
+                             params.dictionary_suggestions,
+                             web_contents_->profile());
+
+  POINT screen_pt = { params.x, params.y };
+  MapWindowPoints(GetHWND(), HWND_DESKTOP, &screen_pt, 1);
+
+  // Enable recursive tasks on the message loop so we can get updates while
+  // the context menu is being displayed.
+  bool old_state = MessageLoop::current()->NestableTasksAllowed();
+  MessageLoop::current()->SetNestableTasksAllowed(true);
+  menu.RunMenuAt(screen_pt.x, screen_pt.y);
+  MessageLoop::current()->SetNestableTasksAllowed(old_state);
+}
+
 WebContents* WebContentsViewWin::CreateNewWindowInternal(
     int route_id,
     HANDLE modal_dialog_event) {
   // Create the new web contents. This will automatically create the new
   // WebContentsView. In the future, we may want to create the view separately.
   WebContents* new_contents =
       new WebContents(web_contents_->profile(),
                       web_contents_->GetSiteInstance(),
                       web_contents_->render_view_factory_,
                       route_id,
@@ skipping 119 matching lines @@
     default:
       break;
   }
 
   return 0;
 }
 
 void WebContentsViewWin::OnPaint(HDC junk_dc) {
   if (web_contents_->render_view_host() &&
       !web_contents_->render_view_host()->IsRenderViewLive()) {
-    if (!web_contents_->sad_tab_.get())
-      web_contents_->sad_tab_.reset(new SadTabView);
+    if (!sad_tab_.get())
+      sad_tab_.reset(new SadTabView);
     CRect cr;
     GetClientRect(&cr);
-    web_contents_->sad_tab_->SetBounds(gfx::Rect(cr));
+    sad_tab_->SetBounds(gfx::Rect(cr));
     ChromeCanvasPaint canvas(GetHWND(), true);
-    web_contents_->sad_tab_->ProcessPaint(&canvas);
+    sad_tab_->ProcessPaint(&canvas);
     return;
   }
 
   // We need to do this to validate the dirty area so we don't end up in a
   // WM_PAINTstorm that causes other mysterious bugs (such as WM_TIMERs not
   // firing etc). It doesn't matter that we don't have any non-clipped area.
   CPaintDC dc(GetHWND());
   SetMsgHandled(FALSE);
 }
 
@@ skipping 137 matching lines @@
   }
   return false;
 }
 
 void WebContentsViewWin::WheelZoom(int distance) {
   if (web_contents_->delegate()) {
     bool zoom_in = distance > 0;
     web_contents_->delegate()->ContentsZoomChange(zoom_in);
   }
 }