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

chrome/browser/web_contents.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.h"
 
 #include "base/command_line.h"
 #include "base/compiler_specific.h"
 #include "base/file_version_info.h"
 #include "chrome/app/locales/locale_settings.h"
@@ skipping 16 matching lines @@
 #include "chrome/browser/navigation_entry.h"
 #include "chrome/browser/password_manager.h"
 #include "chrome/browser/plugin_installer.h"
 #include "chrome/browser/plugin_service.h"
 #include "chrome/browser/printing/print_job.h"
 #include "chrome/browser/render_view_host.h"
 #include "chrome/browser/render_widget_host_view_win.h"  // TODO(brettw) delete me.
 #include "chrome/browser/template_url_fetcher.h"
 #include "chrome/browser/template_url_model.h"
 #include "chrome/browser/views/hung_renderer_view.h"  // TODO(brettw) delete me.
-#include "chrome/browser/views/sad_tab_view.h"  // FIXME(brettw) delete me.
 #include "chrome/browser/web_contents_view.h"
 #include "chrome/browser/web_contents_view_win.h"
 #include "chrome/common/chrome_switches.h"
 #include "chrome/common/l10n_util.h"
 #include "chrome/common/pref_names.h"
 #include "chrome/common/pref_service.h"
 #include "chrome/common/resource_bundle.h"
 #include "net/base/mime_util.h"
 #include "net/base/registry_controlled_domain.h"
 #include "webkit/glue/webkit_glue.h"
@@ skipping 122 matching lines @@
 WebContents::WebContents(Profile* profile,
                          SiteInstance* site_instance,
                          RenderViewHostFactory* render_view_factory,
                          int routing_id,
                          HANDLE modal_dialog_event)
     : TabContents(TAB_CONTENTS_WEB),
       view_(new WebContentsViewWin(this)),
       ALLOW_THIS_IN_INITIALIZER_LIST(
           render_manager_(render_view_factory, this, this)),
       render_view_factory_(render_view_factory),
-      has_page_title_(false),
+      received_page_title_(false),
       is_starred_(false),
       printing_(*this),
       notify_disconnection_(false),
       message_box_active_(CreateEvent(NULL, TRUE, FALSE, NULL)),
       ALLOW_THIS_IN_INITIALIZER_LIST(fav_icon_helper_(this)),
       suppress_javascript_messages_(false),
       load_state_(net::LOAD_STATE_IDLE) {
   InitWebContentsClass();
 
   pending_install_.page_id = 0;
@@ skipping 548 matching lines @@
   if (!printing_.OnRendererGone(rvh))
     return;
   if (rvh != render_view_host()) {
     // The pending or interstitial page's RenderViewHost is gone.  If we are
     // showing an interstitial, this may mean that the original RenderViewHost
     // is gone.  If so, we will call RendererGone again if we try to swap that
     // RenderViewHost back in, in SwapToRenderView.
     return;
   }
 
-  // TODO(brettw) move the platform-specific view stuff here to the view.
-
-  // Force an invalidation here to render sad tab.  however, it is possible for
-  // our window to have already gone away (since we may be in the process of
-  // closing this render view).
-  if (::IsWindow(view_->GetContainerHWND()))
-    InvalidateRect(view_->GetContainerHWND(), NULL, FALSE);
-
   SetIsLoading(false, NULL);
-
-  // Ensure that this browser window is enabled.  This deals with the case where
-  // a renderer crashed while showing a modal dialog.  We're assuming that the
-  // browser code will never show a modal dialog, so we could only be disabled
-  // by something the renderer (or some plug-in) did.
-  HWND root_window = ::GetAncestor(view_->GetContainerHWND(), GA_ROOT);
-  if (!::IsWindowEnabled(root_window))
-    ::EnableWindow(root_window, TRUE);
-
   NotifyDisconnected();
   SetIsCrashed(true);
+
+  // Force an invalidation to render sad tab. The view will notice we crashed
+  // when it paints.
+  view_->Invalidate();
+
   // Hide any visible hung renderer warning for this web contents' process.
   HungRendererWarning::HideForWebContents(this);
 }
 
 void WebContents::DidNavigate(RenderViewHost* rvh,
                               const ViewHostMsg_FrameNavigate_Params& params) {
   if (PageTransition::IsMainFrame(params.transition))
     render_manager_.DidNavigateMainFrame(rvh);
 
   // In the case of interstitial, we don't mess with the navigation entries.
@@ skipping 71 matching lines @@
   unsigned changed_flags = 0;
 
   // Update the URL.
   if (url != entry->url()) {
     changed_flags |= INVALIDATE_URL;
     if (entry == controller()->GetActiveEntry())
       fav_icon_helper_.FetchFavIcon(url);
     entry->set_url(url);
   }
 
-  // For file URLs without a title, use the pathname instead.
-  std::wstring final_title;
-  if (url.SchemeIsFile() && title.empty()) {
-    final_title = UTF8ToWide(url.ExtractFileName());
-  } else {
-    TrimWhitespace(title, TRIM_ALL, &final_title);
-  }
-  if (final_title != entry->title()) {
+  // Save the new title if it changed.
+  if (UpdateTitleForEntry(entry, title))
     changed_flags |= INVALIDATE_TITLE;
-    entry->set_title(final_title);
-
-    // Update the history system for this page.
-    if (!profile()->IsOffTheRecord()) {
-      HistoryService* hs =
-          profile()->GetHistoryService(Profile::IMPLICIT_ACCESS);
-      if (hs)
-        hs->SetPageTitle(entry->display_url(), final_title);
-    }
-  }
-  // TODO(brettw) move this to the view.
-  if (view_->GetContainerHWND()) {
-    // It's possible to get this after the hwnd has been destroyed.
-    ::SetWindowText(view_->GetContainerHWND(), title.c_str());
-    ::SetWindowText(render_widget_host_view()->GetPluginHWND(), title.c_str());
-  }
 
   // Update the state (forms, etc.).
   if (state != entry->content_state())
     entry->set_content_state(state);
 
   // Notify everybody of the changes (only when the current page changed).
   if (changed_flags && entry == controller()->GetActiveEntry())
     NotifyNavigationStateChanged(changed_flags);
   controller()->NotifyEntryChanged(entry, entry_index);
 }
@@ skipping 13 matching lines @@
     // We are showing an interstitial page in a different RenderViewHost, so
     // the page_id is not sufficient to find the entry from the controller.
     // (both RenderViewHost page_ids overlap).  We know it is the active entry,
     // so just use that.
     entry = controller()->GetActiveEntry();
   } else {
     entry = controller()->GetEntryWithPageID(type(), GetSiteInstance(),
                                              page_id);
   }
 
-  if (!entry)
+  if (!entry || !UpdateTitleForEntry(entry, title))
     return;
 
-  std::wstring trimmed_title;
-  TrimWhitespace(title, TRIM_ALL, &trimmed_title);
-  if (title == entry->title())
-    return;  // Title did not change, do nothing.
-
-  entry->set_title(trimmed_title);
-
   // Broadcast notifications when the UI should be updated.
   if (entry == controller()->GetEntryAtOffset(0))
     NotifyNavigationStateChanged(INVALIDATE_TITLE);
-
-  // Update the history system for this page.
-  if (profile()->IsOffTheRecord())
-    return;
-
-  HistoryService* hs = profile()->GetHistoryService(Profile::IMPLICIT_ACCESS);
-  if (hs && !has_page_title_ && !trimmed_title.empty()) {
-    hs->SetPageTitle(entry->display_url(), trimmed_title);
-    has_page_title_ = true;
-  }
 }
 
 
 void WebContents::UpdateEncoding(RenderViewHost* render_view_host,
                                  const std::wstring& encoding_name) {
   set_encoding(encoding_name);
 }
 
 void WebContents::UpdateTargetURL(int32 page_id, const GURL& url) {
   if (delegate())
@@ skipping 163 matching lines @@
   // request. If we end up with another place invoking DownloadImage, probably
   // best to refactor out into notification service, or something similar.
   if (errored)
     fav_icon_helper_.FavIconDownloadFailed(id);
   else
     fav_icon_helper_.SetFavIcon(id, image_url, image);
   if (web_app_.get() && !errored)
     web_app_->SetImage(image_url, image);
 }
 
-void WebContents::ShowContextMenu(
-    const ViewHostMsg_ContextMenu_Params& params) {
-  view_->ShowContextMenu(params);
-}
-
-void WebContents::StartDragging(const WebDropData& drop_data) {
-  view_->StartDragging(drop_data);
-}
-
-void WebContents::UpdateDragCursor(bool is_drop_target) {
-  view_->UpdateDragCursor(is_drop_target);
-}
-
 void WebContents::RequestOpenURL(const GURL& url,
                                  WindowOpenDisposition disposition) {
   OpenURL(url, disposition, PageTransition::LINK);
 }
 
 void WebContents::DomOperationResponse(const std::string& json_string,
                                        int automation_id) {
   DomOperationNotificationDetails details(json_string, automation_id);
   NotificationService::current()->Notify(
       NOTIFY_DOM_OPERATION_RESPONSE, Source<WebContents>(this),
@@ skipping 82 matching lines @@
                                     reply_msg, this);
     delegate()->ShowHtmlDialog(dialog_delegate, NULL);
   }
 }
 
 void WebContents::PasswordFormsSeen(
     const std::vector<PasswordForm>& forms) {
   GetPasswordManager()->PasswordFormsSeen(forms);
 }
 
-void WebContents::TakeFocus(bool reverse) {
-  // TODO(brettw) move this to the view.
-  views::FocusManager* focus_manager =
-      views::FocusManager::GetFocusManager(view_->GetContainerHWND());
-
-  // We may not have a focus manager if the tab has been switched before this
-  // message arrived.
-  if (focus_manager) {
-    focus_manager->AdvanceFocus(reverse);
-  }
-}
-
 // Checks to see if we should generate a keyword based on the OSDD, and if
 // necessary uses TemplateURLFetcher to download the OSDD and create a keyword.
 void WebContents::PageHasOSDD(RenderViewHost* render_view_host,
                               int32 page_id, const GURL& url,
                               bool autodetected) {
   // Make sure page_id is the current page, and the TemplateURLModel is loaded.
   DCHECK(url.is_valid());
   if (!controller() || !IsActiveEntry(page_id))
     return;
   TemplateURLModel* url_model = profile()->GetTemplateURLModel();
@@ skipping 224 matching lines @@
                   &GearsCreateShortcutCallbackFunctor::Run));
 }
 
 void WebContents::OnEnterOrSpace() {
   // See comment in RenderViewHostDelegate::OnEnterOrSpace as to why we do this.
   DownloadRequestManager* drm = g_browser_process->download_request_manager();
   if (drm)
     drm->OnUserGesture(this);
 }
 
-// Stupid pass-through for RenderViewHostDelegate.
-void WebContents::HandleKeyboardEvent(const WebKeyboardEvent& event) {
-  view_->HandleKeyboardEvent(event);
-}
-
 void WebContents::FileSelected(const std::wstring& path, void* params) {
   render_view_host()->FileSelected(path);
 }
 
 void WebContents::FileSelectionCanceled(void* params) {
   // If the user cancels choosing a file to upload we need to pass back the
   // empty string.
   render_view_host()->FileSelected(std::wstring());
 }
 
@@ skipping 99 matching lines @@
     // TODO(brettw) bug 1343111: Password manager stuff in here needs to be
     // cleaned up and covered by tests.
     if (!params.password_form.origin.is_valid())
       GetPasswordManager()->DidNavigate();
   }
 
   // The keyword generator uses the navigation entries, so must be called after
   // the commit.
   GenerateKeywordIfNecessary(params);
 
-  // We no longer know the title after this navigation.
-  has_page_title_ = false;
+  // Allow the new page to set the title again.
+  received_page_title_ = false;
 
   // Update contents MIME type of the main webframe.
   contents_mime_type_ = params.contents_mime_type;
 
   // Get the favicon, either from history or request it from the net.
   fav_icon_helper_.FetchFavIcon(details.entry->url());
 
   // Close constrained popups if necessary.
   MaybeCloseChildWindows(params);
 
@@ skipping 153 matching lines @@
         redirects.back() = display_url;
       hs->AddPage(display_url, this, params.page_id, params.referrer,
                   params.transition, redirects);
     } else {
       hs->AddPage(params.url, this, params.page_id, params.referrer,
                   params.transition, params.redirects);
     }
   }
 }
 
+bool WebContents::UpdateTitleForEntry(NavigationEntry* entry,
+                                      const std::wstring& title) {
+  // For file URLs without a title, use the pathname instead. In the case of a
+  // synthesized title, we don't want the update to count toward the "one set
+  // per page of the title to history."
+  std::wstring final_title;
+  bool explicit_set;
+  if (entry->url().SchemeIsFile() && title.empty()) {
+    final_title = UTF8ToWide(entry->url().ExtractFileName());
+    explicit_set = false;  // Don't count synthetic titles toward the set limit.
+  } else {
+    TrimWhitespace(title, TRIM_ALL, &final_title);
+    explicit_set = true;
+  }
+
+  if (final_title != entry->title())

[Ben Goodger (Google), 2008/10/18 05:08:13]

do you mean ==?

+    return false;  // Nothing changed, don't bother.
+
+  entry->set_title(final_title);
+
+  // Update the history system for this page.
+  if (!profile()->IsOffTheRecord() && !received_page_title_) {
+    HistoryService* hs =
+        profile()->GetHistoryService(Profile::IMPLICIT_ACCESS);
+    if (hs)
+      hs->SetPageTitle(entry->display_url(), final_title);
+    
+    // Don't allow the title to be saved again for explicitly set ones.
+    received_page_title_ = explicit_set;
+  }
+
+  // Lastly, set the title for the view.
+  view_->SetPageTitle(final_title);
+
+  return true;
+}
+
 void WebContents::NotifySwapped() {
   // After sending out a swap notification, we need to send a disconnect
   // notification so that clients that pick up a pointer to |this| can NULL the
   // pointer.  See Bug 1230284.
   notify_disconnection_ = true;
   NotificationService::current()->
       Notify(NOTIFY_WEB_CONTENTS_SWAPPED,
              Source<WebContents>(this),
              NotificationService::NoDetails());
 }
@@ skipping 84 matching lines @@
     // The favicon url isn't valid. This means there really isn't a favicon,
     // or the favicon url wasn't obtained before the load started. This assumes
     // the later.
     // TODO(sky): Need a way to set the favicon that doesn't involve generating
     // its url.
     new_url->SetFavIconURL(TemplateURL::GenerateFaviconURL(params.referrer));
   }
   new_url->set_safe_for_autoreplace(true);
   url_model->Add(new_url);
 }