Pass focus to browser after login.

chrome/browser/chromeos/login/session_login_browsertest.cc

+// Copyright 2013 The Chromium Authors. All rights reserved.
+// Use of this source code is governed by a BSD-style license that can be
+// found in the LICENSE file.
+
+#include "chrome/browser/chromeos/login/login_manager_test.h"
+#include "chrome/browser/chromeos/login/startup_utils.h"
+#include "chrome/browser/profiles/profile_manager.h"
+#include "chrome/browser/ui/browser.h"
+#include "chrome/browser/ui/browser_finder.h"
+#include "chrome/browser/ui/browser_window.h"
+#include "chrome/common/chrome_switches.h"
+#include "testing/gtest/include/gtest/gtest.h"
+

[oshima, 2013/08/20 14:56:01]

namespace chromeos {

[Nikita (slow), 2013/08/20 16:04:39]

Done.

+namespace {
+
+const char* kTestUser = "test-user@gmail.com";

[oshima, 2013/08/20 14:56:01]

char[]

[Nikita (slow), 2013/08/20 16:04:39]

Done.

+
+}  // anonymous namespace
+
+class BrowserLoginTest : public chromeos::LoginManagerTest {
+ public:
+  BrowserLoginTest() : LoginManagerTest(true) {}

[oshima, 2013/08/20 14:56:01]

virtual BrowserLOginTest() {}

[Nikita (slow), 2013/08/20 16:04:39]

Do you mean virtual destructor?

[oshima, 2013/08/20 17:52:47]

yep sorry for typo.

[Nikita (slow), 2013/08/20 18:04:21]

Done.

+
+  virtual void SetUpCommandLine(CommandLine* command_line) OVERRIDE {
+    LoginManagerTest::SetUpCommandLine(command_line);
+    command_line->AppendSwitch(::switches::kCreateBrowserOnStartupForTests);
+  }
+};
+
+IN_PROC_BROWSER_TEST_F(BrowserLoginTest, PRE_BrowserActive) {
+  RegisterUser(kTestUser);
+  chromeos::StartupUtils::MarkOobeCompleted();
+}
+
+IN_PROC_BROWSER_TEST_F(BrowserLoginTest, BrowserActive) {
+  LoginUser(kTestUser);
+  Browser* browser = FindAnyBrowser(ProfileManager::GetDefaultProfile(),
+                                    false,
+                                    chrome::HOST_DESKTOP_TYPE_ASH);
+  EXPECT_TRUE(browser != NULL);
+  EXPECT_TRUE(browser->window()->IsActive());

[oshima, 2013/08/20 14:56:01]

can you test if the omnibox has focus as well? (they're similar but different
concept)

[Nikita (slow), 2013/08/20 16:04:39]

Done.

+}