Re-enable two disabled tests in c/b/ui/views/location_bar/.

This updates the comments on the other tests to have TODO form.

Bug: 163931, 823543, 824418
Change-Id: Ic82109fb4513a6b8053477ac28e5ee3969799f52
Reviewed-on: https://chromium-review.googlesource.com/c/1334842Reviewed-by: Robert Liao <robliao@chromium.org>
Commit-Queue: Peter Kasting <pkasting@chromium.org>
Cr-Commit-Position: refs/heads/master@{#608959}

chrome/browser/ui/views/location_bar/location_icon_view_interactive_uitest.cc

@@ -25,7 +25,8 @@ class LocationIconViewTest : public InProcessBrowserTest {
 };
 
 #if defined(OS_MACOSX)
-// Focusing or input is not completely working on Mac: http://crbug.com/824418
+// TODO(robliao): https://crbug.com/824418  Focusing or input is not completely
+// working on Mac.
 #define MAYBE_HideOnSecondClick DISABLED_HideOnSecondClick
 #else
 #define MAYBE_HideOnSecondClick HideOnSecondClick
@@ -65,7 +66,8 @@ IN_PROC_BROWSER_TEST_F(LocationIconViewTest, MAYBE_HideOnSecondClick) {
 }
 
 #if defined(OS_MACOSX)
-// Widget activation doesn't work on Mac: https://crbug.com/823543
+// TODO(robliao): https://crbug.com/823543  Widget activation doesn't work on
+// Mac.
 #define MAYBE_ActivateFirstInactiveBubbleForAccessibility \
   DISABLED_ActivateFirstInactiveBubbleForAccessibility
 #else

chrome/browser/ui/views/location_bar/star_view_browsertest.cc

@@ -44,10 +44,9 @@ class StarViewTest : public InProcessBrowserTest {
 
 // Verify that clicking the bookmark star a second time hides the bookmark
 // bubble.
-#if defined(OS_LINUX) && defined(USE_AURA) && !defined(OS_CHROMEOS)
-#define MAYBE_HideOnSecondClick DISABLED_HideOnSecondClick
-#elif defined(OS_MACOSX)
-// Focusing or input is not completely working on Mac: http://crbug.com/824418
+#if defined(OS_MACOSX)
+// TODO(robliao): https://crbug.com/824418  Focusing or input is not completely
+// working on Mac.
 #define MAYBE_HideOnSecondClick DISABLED_HideOnSecondClick
 #else
 #define MAYBE_HideOnSecondClick HideOnSecondClick

chrome/browser/ui/views/location_bar/zoom_bubble_view_browsertest.cc

@@ -39,15 +39,9 @@ void ShowInActiveTab(Browser* browser) {
 
 }  // namespace
 
-// TODO(linux_aura) http://crbug.com/163931
-#if defined(OS_LINUX) && !defined(OS_CHROMEOS) && defined(USE_AURA)
-#define MAYBE_NonImmersiveFullscreen DISABLED_NonImmersiveFullscreen
-#else
-#define MAYBE_NonImmersiveFullscreen NonImmersiveFullscreen
-#endif
 // Test whether the zoom bubble is anchored and whether it is visible when in
 // non-immersive fullscreen.
-IN_PROC_BROWSER_TEST_F(ZoomBubbleBrowserTest, MAYBE_NonImmersiveFullscreen) {
+IN_PROC_BROWSER_TEST_F(ZoomBubbleBrowserTest, NonImmersiveFullscreen) {
   BrowserView* browser_view = static_cast<BrowserView*>(browser()->window());
   content::WebContents* web_contents = browser_view->GetActiveWebContents();