Adds a way to set the boundaries of the browser window through...

Adds a way to set the boundaries of the browser window through
automation and use that to set the size of the window in the

"New Tab Cold" to ensure we are testing the normal case and not the
small case.

BUG=None

TEST=The NewTabUIStartupTest, PerfCold should still work.


Committed: http://src.chromium.org/viewvc/chrome?view=rev&revision=20124

[stoyan, 2009-07-07 16:27:45]

lgtm

http://codereview.chromium.org/149233/diff/1008/32
File chrome/browser/automation/automation_provider.cc (right):

http://codereview.chromium.org/149233/diff/1008/32#newcode1717
Line 1717: void AutomationProvider::SetWindowBounds(int handle, const gfx::Rect&
bounds,
This method is wrapped in 
#if defined(OS_WIN) [line 1559 up to line 1745]; mac and linux builds -> unhappy

http://codereview.chromium.org/149233/diff/1008/32#newcode1725
Line 1725: ::MoveWindow(hwnd, bounds.x(), bounds.y(), bounds.width(),
bounds.height(),
Perhaps *sucess = ::MoveWindow(..)?
Just in case.

[arv (Not doing code reviews), 2009-07-08 00:11:14]

I had to add SetWindowBounds to temp_scaffolding_stubs.cc

http://codereview.chromium.org/149233/diff/1008/32
File chrome/browser/automation/automation_provider.cc (right):

http://codereview.chromium.org/149233/diff/1008/32#newcode1717
Line 1717: void AutomationProvider::SetWindowBounds(int handle, const gfx::Rect&
bounds,
On 2009/07/07 16:27:46, stoyan wrote:
> This method is wrapped in 
> #if defined(OS_WIN) [line 1559 up to line 1745]; mac and linux builds ->
unhappy
> 

Done.

http://codereview.chromium.org/149233/diff/1008/32#newcode1725
Line 1725: ::MoveWindow(hwnd, bounds.x(), bounds.y(), bounds.width(),
bounds.height(),
On 2009/07/07 16:27:46, stoyan wrote:
> Perhaps *sucess = ::MoveWindow(..)?
> Just in case.

Done.