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

Unified Diff: chrome/browser/automation/testing_automation_provider.cc

Issue 10700071: browser: Remove fullscreen functions and have callers call FullscreenController directly. (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src
Patch Set: Created 8 years, 6 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
« no previous file with comments | « no previous file | chrome/browser/ui/browser.h » ('j') | chrome/browser/ui/browser.cc » ('J')
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: chrome/browser/automation/testing_automation_provider.cc
diff --git a/chrome/browser/automation/testing_automation_provider.cc b/chrome/browser/automation/testing_automation_provider.cc
index 2f57098dc865a2a886a2ac1d5126f014eeb87c82..62dea9ff761b2a3d0375c462765e7abf7381b60c 100644
--- a/chrome/browser/automation/testing_automation_provider.cc
+++ b/chrome/browser/automation/testing_automation_provider.cc
@@ -5958,7 +5958,8 @@ void TestingAutomationProvider::AcceptCurrentFullscreenOrMouseLockRequest(
WebContents* fullscreen_tab = chrome::GetActiveWebContents(browser);
FullscreenExitBubbleType type =
browser->fullscreen_controller_->GetFullscreenExitBubbleType();
- browser->OnAcceptFullscreenPermission(fullscreen_tab->GetURL(), type);
+ browser->fullscreen_controller_->OnAcceptFullscreenPermission(
+ fullscreen_tab->GetURL(), type);
AutomationJSONReply(this, reply_message).SendSuccess(NULL);
}
@@ -5968,7 +5969,7 @@ void TestingAutomationProvider::DenyCurrentFullscreenOrMouseLockRequest(
IPC::Message* reply_message) {
FullscreenExitBubbleType type =
browser->fullscreen_controller_->GetFullscreenExitBubbleType();
- browser->OnDenyFullscreenPermission(type);
+ browser->fullscreen_controller_->OnDenyFullscreenPermission(type);
AutomationJSONReply(this, reply_message).SendSuccess(NULL);
}
« no previous file with comments | « no previous file | chrome/browser/ui/browser.h » ('j') | chrome/browser/ui/browser.cc » ('J')

Powered by Google App Engine
This is Rietveld 408576698