Add back a test lost in r37819, because it had been at the bottom of an unrel...

Add back a test lost in r37819, because it had been at the bottom of an unrelated file.

BUG=none
TEST=Test passes

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

[Paweł Hajdan Jr., 2010-07-28 23:29:07]

Drive-by with minor test comments. No need to wait for another review by me.

http://codereview.chromium.org/3029038/diff/1/2
File chrome/test/interactive_ui/keyboard_access_uitest.cc (right):

http://codereview.chromium.org/3029038/diff/1/2#newcode113
chrome/test/interactive_ui/keyboard_access_uitest.cc:113:
scoped_refptr<BrowserProxy> browser(automation()->GetBrowserWindow(0));
nit: This should ASSERT_TRUE(browser) to avoid a crash.

http://codereview.chromium.org/3029038/diff/1/2#newcode123
chrome/test/interactive_ui/keyboard_access_uitest.cc:123:
scoped_refptr<WindowProxy> window(browser->GetWindow());
nit: This should ASSERT_TRUE(window) to avoid a crash.

[Peter Kasting, 2010-07-29 00:00:02]

http://codereview.chromium.org/3029038/diff/1/2
File chrome/test/interactive_ui/keyboard_access_uitest.cc (right):

http://codereview.chromium.org/3029038/diff/1/2#newcode113
chrome/test/interactive_ui/keyboard_access_uitest.cc:113:
scoped_refptr<BrowserProxy> browser(automation()->GetBrowserWindow(0));
On 2010/07/28 23:29:08, Paweł Hajdan Jr. wrote:
> nit: This should ASSERT_TRUE(browser) to avoid a crash.

Done.

http://codereview.chromium.org/3029038/diff/1/2#newcode123
chrome/test/interactive_ui/keyboard_access_uitest.cc:123:
scoped_refptr<WindowProxy> window(browser->GetWindow());
On 2010/07/28 23:29:08, Paweł Hajdan Jr. wrote:
> nit: This should ASSERT_TRUE(window) to avoid a crash.

Done.

[Nico, 2010-07-29 21:34:56]

LG

sorry about the delay