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

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

Issue 10388251: Support maximize window command. (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src
Patch Set: |BrowserWindow| (gtk and cocoa) will notify when a window is maximized. Created 8 years, 7 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/automation/testing_automation_provider.cc
diff --git a/chrome/browser/automation/testing_automation_provider.cc b/chrome/browser/automation/testing_automation_provider.cc
index 8513ec0f4d187784a7a4eacc3ca85950b258ad68..a283eddb886c1d556420ff66be249b09852d460c 100644
--- a/chrome/browser/automation/testing_automation_provider.cc
+++ b/chrome/browser/automation/testing_automation_provider.cc
@@ -1698,6 +1698,8 @@ void TestingAutomationProvider::SendJSONRequest(int handle,
&TestingAutomationProvider::CloseTabJSON;
handler_map["SetViewBounds"] =
&TestingAutomationProvider::SetViewBounds;
+ handler_map["MaximizeView"] =
+ &TestingAutomationProvider::MaximizeView;
handler_map["WebkitMouseMove"] =
&TestingAutomationProvider::WebkitMouseMove;
handler_map["WebkitMouseClick"] =
@@ -6485,10 +6487,28 @@ void TestingAutomationProvider::SetViewBounds(
reply.SendError(Error(automation::kInvalidId, error));
return;
}
- browser->window()->SetBounds(gfx::Rect(x, y, width, height));
+ BrowserWindow* browser_window = browser->window();
+ if (browser_window->IsMaximized()) {
+ browser_window->Restore();
+ }
+ browser_window->SetBounds(gfx::Rect(x, y, width, height));
reply.SendSuccess(NULL);
}
+void TestingAutomationProvider::MaximizeView(
+ base::DictionaryValue* args,
+ IPC::Message* reply_message) {
+ Browser* browser;
+ std::string error;
+ if (!GetBrowserFromJSONArgs(args, &browser, &error)) {
+ AutomationJSONReply(this, reply_message)
+ .SendError(Error(automation::kInvalidId, error));
+ return;
+ }
+ new WindowMaximizedObserver(this, reply_message);
+ browser->window()->Maximize();
+}
+
void TestingAutomationProvider::ActivateTabJSON(
DictionaryValue* args,
IPC::Message* reply_message) {

Powered by Google App Engine
This is Rietveld 408576698