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

Unified Diff: chrome/browser/plugins/flash_download_interception.cc

Issue 2385033002: Hookup the plugin placeholder to the Flash permission prompt (Closed)
Patch Set: Hookup the plugin placeholder to the Flash permission prompt Created 4 years, 2 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/plugins/flash_download_interception.cc
diff --git a/chrome/browser/plugins/flash_download_interception.cc b/chrome/browser/plugins/flash_download_interception.cc
index 032830b00999ce6cfd7fb95810afb94b6675123b..6951f74a160d199e009d7115cf43d1fe8b101a33 100644
--- a/chrome/browser/plugins/flash_download_interception.cc
+++ b/chrome/browser/plugins/flash_download_interception.cc
@@ -34,21 +34,21 @@ void DoNothing(blink::mojom::PermissionStatus result) {}
bool InterceptNavigation(
content::WebContents* source,
const navigation_interception::NavigationParams& params) {
- FlashDownloadInterception::InterceptFlashDownloadNavigation(source);
+ FlashDownloadInterception::ShowRunFlashPrompt(source);
return true;
}
} // namespace
// static
-void FlashDownloadInterception::InterceptFlashDownloadNavigation(
- content::WebContents* source) {
+void FlashDownloadInterception::ShowRunFlashPrompt(
+ content::WebContents* web_contents) {
DCHECK_CURRENTLY_ON(BrowserThread::UI);
PermissionManager* manager = PermissionManager::Get(
- Profile::FromBrowserContext(source->GetBrowserContext()));
+ Profile::FromBrowserContext(web_contents->GetBrowserContext()));
manager->RequestPermission(
- content::PermissionType::FLASH, source->GetMainFrame(),
- source->GetLastCommittedURL(), true, base::Bind(&DoNothing));
+ content::PermissionType::FLASH, web_contents->GetMainFrame(),
+ web_contents->GetLastCommittedURL(), true, base::Bind(&DoNothing));
}
// static
« no previous file with comments | « chrome/browser/plugins/flash_download_interception.h ('k') | chrome/browser/plugins/flash_permission_browsertest.cc » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698