Index: chrome/browser/ui/search/instant_extended_interactive_uitest.cc |
diff --git a/chrome/browser/ui/search/instant_extended_interactive_uitest.cc b/chrome/browser/ui/search/instant_extended_interactive_uitest.cc |
index fba91068a59350c86c58c83bc1dbae40f208fe17..08f93d51b6945b9285b3ea06d65d0fd08b7a9cdb 100644 |
--- a/chrome/browser/ui/search/instant_extended_interactive_uitest.cc |
+++ b/chrome/browser/ui/search/instant_extended_interactive_uitest.cc |
@@ -874,6 +874,7 @@ IN_PROC_BROWSER_TEST_F(InstantExtendedTest, PreloadedNTPRenderViewGone) { |
EXPECT_EQ(NULL, instant()->ntp()); |
// Open new tab. Should use local NTP. |
+ // TODO(creis): Same problem. |
ui_test_utils::NavigateToURLWithDisposition( |
browser(), |
GURL(chrome::kChromeUINewTabURL), |
@@ -896,6 +897,7 @@ IN_PROC_BROWSER_TEST_F(InstantExtendedTest, PreloadedNTPDoesntSupportInstant) { |
EXPECT_TRUE(instant()->ntp()->IsLocal()); |
// Open new tab. Should use local NTP. |
+ // TODO(creis): Same problem. |
ui_test_utils::NavigateToURLWithDisposition( |
browser(), |
GURL(chrome::kChromeUINewTabURL), |
@@ -2322,6 +2324,10 @@ class InstantExtendedFirstTabTest : public InProcessBrowserTest, |
IN_PROC_BROWSER_TEST_F( |
InstantExtendedFirstTabTest, MAYBE_RedirectToLocalOnLoadFailure) { |
// Create a new window to test the first NTP load. |
+ // TODO(creis): This fails in BrowserInstantController's |
+ // MaybeSwapInInstantNTPContents because the pending entry hasn't |
+ // committed yet. It's trying to SwapInInstantNTP as part of the OpenURL |
+ // call, which can't work. (What if the navigation were to fail?) |
ui_test_utils::NavigateToURLWithDisposition( |
browser(), |
GURL(chrome::kChromeUINewTabURL), |
@@ -2353,6 +2359,7 @@ IN_PROC_BROWSER_TEST_F( |
// Instant tab contents should be preloaded. |
ASSERT_NE(static_cast<InstantTab*>(NULL), instant()->instant_tab()); |
+ // TODO(creis): This is where the test fails. |
EXPECT_TRUE(instant()->instant_tab()->IsLocal()); |
} |