<webview>: Implement WebRequest API

content/browser/web_contents/web_contents_impl.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 "content/browser/web_contents/web_contents_impl.h"
 
 #include <utility>
 
 #include "base/command_line.h"
 #include "base/metrics/histogram.h"
@@ skipping 319 matching lines @@
   }
   created_widgets_.clear();
 
   // Clear out any JavaScript state.
   if (dialog_manager_)
     dialog_manager_->ResetJavaScriptState(this);
 
   if (color_chooser_)
     color_chooser_->End();
 
+  if (browser_plugin_guest_)
+    browser_plugin_guest_->Destroy();

[Charlie Reis, 2013/05/17 21:13:00]

Is this something new that we didn't need before, or is it a fix for an
independent bug?  (If it's the latter, I'd suggest fixing it in a separate CL,
in case we want to merge it or if the rest gets reverted for some reason.)

+
   NotifyDisconnected();
 
   // Notify any observer that have a reference on this WebContents.
   NotificationService::current()->Notify(
       NOTIFICATION_WEB_CONTENTS_DESTROYED,
       Source<WebContents>(this),
       NotificationService::NoDetails());
 
   // TODO(brettw) this should be moved to the view.
 #if defined(OS_WIN) && !defined(USE_AURA)
@@ skipping 3173 matching lines @@
 }
 
 BrowserPluginGuestManager*
     WebContentsImpl::GetBrowserPluginGuestManager() const {
   return static_cast<BrowserPluginGuestManager*>(
       GetBrowserContext()->GetUserData(
           browser_plugin::kBrowserPluginGuestManagerKeyName));
 }
 
 }  // namespace content