Move RequestPpapiBrokerPermission from WebContentsObserver to WebContentsDelegate.

content/public/browser/web_contents_delegate.cc

Index: content/public/browser/web_contents_delegate.cc
diff --git a/content/public/browser/web_contents_delegate.cc b/content/public/browser/web_contents_delegate.cc
index f3b03d5836aa4dc344e8aab1cf44ae860a543a24..e056376c5c6b7c70b0922fa5545ec501dbc69570 100644
--- a/content/public/browser/web_contents_delegate.cc
+++ b/content/public/browser/web_contents_delegate.cc
@@ -143,6 +143,14 @@ void WebContentsDelegate::WebIntentDispatch(
   delete intents_dispatcher;
 }
 
+void WebContentsDelegate::RequestPpapiBrokerPermission(
+    WebContents* web_contents,
+    const GURL& url,
+    const FilePath& plugin_path,
+    const base::Callback<void(bool)>& callback) {
+  callback.Run(false);

[jam, 2012/11/16 17:46:40]

nit: don't add this function body, just leave it empty. in general, void
functions are empty in the delegate interfaces. these are not meant to be
"sample" or "basic" interfaces. if you really want, you can have a bool return
value here since we do implement bodies with return values.

[Bernhard Bauer, 2012/11/19 08:55:17]

Done. I use a return value of false here and true in "real" implementations now
to make sure that we don't ever drop the callback (which would result in very
subtle issues).

+}
+
 WebContentsDelegate::~WebContentsDelegate() {
   while (!attached_contents_.empty()) {
     WebContents* web_contents = *attached_contents_.begin();