Move core web_view code to extensions.

extensions/browser/guest_view/web_view/web_view_permission_helper_delegate.h

 // Copyright 2014 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.
 
 #ifndef EXTENSIONS_BROWSER_GUEST_VIEW_WEB_VIEW_WEB_VIWE_PERMISSION_HELPER_DELEGATE_H_
 #define EXTENSIONS_BROWSER_GUEST_VIEW_WEB_VIEW_WEB_VIWE_PERMISSION_HELPER_DELEGATE_H_
 
 #include "content/public/browser/web_contents.h"
 #include "content/public/browser/web_contents_observer.h"
 #include "content/public/common/media_stream_request.h"
+#include "extensions/browser/guest_view/web_view/web_view_permission_helper.h"
 
 namespace extensions {
 
 // A delegate class of WebViewPermissionHelper to request permissions that are
 // not a part of extensions.
 class WebViewPermissionHelperDelegate : public content::WebContentsObserver {
  public:
-  explicit WebViewPermissionHelperDelegate(content::WebContents* contents);
+  explicit WebViewPermissionHelperDelegate(
+      extensions::WebViewPermissionHelper* web_view_permission_helper);

[Fady Samuel, 2014/08/22 16:40:17]

Given we are already in the extensions namespace, I don't think we need
extensions::

[Xi Han, 2014/08/22 18:04:53]

Good catch, thanks. I just forget to remove these extensions::.

   virtual ~WebViewPermissionHelperDelegate();
 
   virtual void RequestMediaAccessPermission(
       content::WebContents* source,
       const content::MediaStreamRequest& request,
       const content::MediaResponseCallback& callback) {}
 
   virtual void CanDownload(
       content::RenderViewHost* render_view_host,
       const GURL& url,
@@ skipping 47 matching lines @@
   // If access was blocked due to the page's content settings,
   // |blocked_by_policy| should be true, and this function should invoke
   // OnContentBlocked.
   virtual void FileSystemAccessedSync(
       int render_process_id,
       int render_frame_id,
       const GURL& url,
       bool blocked_by_policy,
       IPC::Message* reply_msg) {}
 
+  extensions:: WebViewPermissionHelper* web_view_permission_helper() {

[Fady Samuel, 2014/08/22 16:40:17]

web_view_permission_helper() const

Also, I don't think we need extensions::

[Xi Han, 2014/08/22 18:04:52]

Done.

+    return web_view_permission_helper_;
+  }
+
  private:
+  extensions::WebViewPermissionHelper* web_view_permission_helper_;

[Fady Samuel, 2014/08/22 16:40:17]

This indicates that the WebViewPermissionHelper cannot change.
WebViewPermissionHelper* const web_view_permission_helper_;

[Xi Han, 2014/08/22 18:04:52]

Done.

+
   DISALLOW_COPY_AND_ASSIGN(WebViewPermissionHelperDelegate);
 };
 
 }  // namespace extensions
 
 #endif  // EXTENSIONS_BROWSER_GUEST_VIEW_WEB_VIEW_WEB_VIWE_PERMISSION_HELPER_DELEGATE_H_