Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(4055)

Unified Diff: chrome/browser/guest_view/web_view/javascript_dialog_helper.cc

Issue 347113002: Refactor PluginPermissionHelper as WebViewPermissionHelper (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: Changes are made. Created 6 years, 5 months ago
Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.
Jump to:
View side-by-side diff with in-line comments
Download patch
Index: chrome/browser/guest_view/web_view/javascript_dialog_helper.cc
diff --git a/chrome/browser/guest_view/web_view/javascript_dialog_helper.cc b/chrome/browser/guest_view/web_view/javascript_dialog_helper.cc
index 4209c3bdcce2982a92f1ac166e15dec9bfc2b1b9..ad3d2433cf153778a4af16da0af0640d6d111cc3 100644
--- a/chrome/browser/guest_view/web_view/javascript_dialog_helper.cc
+++ b/chrome/browser/guest_view/web_view/javascript_dialog_helper.cc
@@ -9,6 +9,7 @@
#include "chrome/browser/guest_view/guest_view_constants.h"
#include "chrome/browser/guest_view/web_view/web_view_constants.h"
#include "chrome/browser/guest_view/web_view/web_view_guest.h"
+#include "chrome/browser/guest_view/web_view/web_view_permission_helper.h"
#include "chrome/browser/guest_view/web_view/web_view_permission_types.h"
namespace {
@@ -31,7 +32,7 @@ std::string JavaScriptMessageTypeToString(
} // namespace
JavaScriptDialogHelper::JavaScriptDialogHelper(WebViewGuest* guest)
- : webview_guest_(guest) {
+ : web_view_guest_(guest) {
}
JavaScriptDialogHelper::~JavaScriptDialogHelper() {
@@ -60,7 +61,9 @@ void JavaScriptDialogHelper::RunJavaScriptDialog(
request_info.Set(
guestview::kUrl,
base::Value::CreateStringValue(origin_url.spec()));
- webview_guest_->RequestPermission(
+ WebViewPermissionHelper* web_view_permission_helper =
+ WebViewPermissionHelper::FromWebContents(web_contents);
+ web_view_permission_helper->RequestPermission(
WEB_VIEW_PERMISSION_TYPE_JAVASCRIPT_DIALOG,
request_info,
base::Bind(&JavaScriptDialogHelper::OnPermissionResponse,
@@ -98,6 +101,6 @@ void JavaScriptDialogHelper::OnPermissionResponse(
const DialogClosedCallback& callback,
bool allow,
const std::string& user_input) {
- callback.Run(allow && webview_guest_->attached(),
+ callback.Run(allow && web_view_guest_->attached(),
base::UTF8ToUTF16(user_input));
}

Powered by Google App Engine
This is Rietveld 408576698