Revise NativeWidgetWin focus hack; force focus on restore.

chrome/browser/ui/browser_focus_uitest.cc

Index: chrome/browser/ui/browser_focus_uitest.cc
diff --git a/chrome/browser/ui/browser_focus_uitest.cc b/chrome/browser/ui/browser_focus_uitest.cc
index 3e0d748857b4540d2f18ea217fe95d3f0d7b2083..43b2f6226c48d7109e1aa9c17897e8796674f19b 100644
--- a/chrome/browser/ui/browser_focus_uitest.cc
+++ b/chrome/browser/ui/browser_focus_uitest.cc
@@ -234,8 +234,7 @@ IN_PROC_BROWSER_TEST_F(BrowserFocusTest, MAYBE_ClickingMovesFocus) {
   ASSERT_TRUE(IsViewFocused(VIEW_ID_OMNIBOX));
 }
 
-// Flaky, http://crbug.com/69034.
-IN_PROC_BROWSER_TEST_F(BrowserFocusTest, DISABLED_BrowsersRememberFocus) {
+IN_PROC_BROWSER_TEST_F(BrowserFocusTest, BrowsersRememberFocus) {
   ASSERT_TRUE(ui_test_utils::BringBrowserWindowToFront(browser()));
   ASSERT_TRUE(test_server()->Start());
 
@@ -257,7 +256,12 @@ IN_PROC_BROWSER_TEST_F(BrowserFocusTest, DISABLED_BrowsersRememberFocus) {
   // Hide the window, show it again, the focus should not have changed.
   ui_test_utils::HideNativeWindow(window);
   ASSERT_TRUE(ui_test_utils::ShowAndFocusNativeWindow(window));
+#if defined(OS_CHROMEOS)
+  // Chrome OS focuses tab content on restore; see http://crbug.com/225963
+  ASSERT_TRUE(IsViewFocused(VIEW_ID_TAB_CONTAINER));

[Ben Goodger (Google), 2013/04/03 17:54:32]

can you tackle this in a different CL then? i am confused by it here.

[msw, 2013/04/03 17:57:45]

So, leave the test disabled on ChromeOS (or all platforms)?
Otherwise I'm not sure what you're asking for.

+#else
   ASSERT_TRUE(IsViewFocused(VIEW_ID_OMNIBOX));
+#endif
 
   // The rest of this test does not make sense on Linux because the behavior
   // of Activate() is not well defined and can vary by window manager.