Miscellaneous cleanups from several months ago I never got around to landing.

ui/views/controls/webview/webview.cc

 // Copyright (c) 2012 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 "ui/views/controls/webview/webview.h"
 
 #include "content/public/browser/browser_context.h"
 #include "content/public/browser/navigation_controller.h"
 #include "content/public/browser/notification_details.h"
 #include "content/public/browser/notification_registrar.h"
@@ skipping 199 matching lines @@
         content::NOTIFICATION_WEB_CONTENTS_DESTROYED,
         content::Source<content::WebContents>(web_contents_));
   }
 }
 
 void WebView::DetachWebContents() {
   if (web_contents_) {
     wcv_holder_->Detach();
 #if defined(OS_WIN) && !defined(USE_AURA)
     // TODO(beng): This should either not be necessary, or be done implicitly by
-    //             NativeViewHostWin on Detach(). As it stands, this is needed
-    //             so that the view of the detached contents knows to tell the
-    //             renderer its been hidden.
+    // NativeViewHostWin on Detach(). As it stands, this is needed so that the
+    // view of the detached contents knows to tell the renderer it's been
+    // hidden.
+    //
+    // Moving this out of here would also mean we wouldn't be potentially
+    // calling member functions on a half-destroyed WebContents.
     ShowWindow(web_contents_->GetNativeView(), SW_HIDE);
 #endif
   }
   registrar_.RemoveAll();
 }
 
 void WebView::RenderViewHostChanged(content::RenderViewHost* old_host,
                                     content::RenderViewHost* new_host) {
   if (GetFocusManager()->GetFocusedView() == this)
     web_contents_->Focus();
@@ skipping 17 matching lines @@
     return content::WebContents::Create(browser_context,
                                         site_instance,
                                         MSG_ROUTING_NONE,
                                         NULL);
   }
 
   return contents;
 }
 
 }  // namespace views