Disable ShelfViewVisibleBoundsTest flaky tests

ash/shelf/shelf_view_unittest.cc

 // Copyright (c) 2012 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 "ash/common/shelf/shelf_view.h"
 
 #include <algorithm>
 #include <memory>
 #include <utility>
 #include <vector>
@@ skipping 1939 matching lines @@
   CheckAllItemsAreInBounds();
   // Same for overflow case.
   while (!test_api_->IsOverflowButtonVisible()) {
     AddAppShortcut();
   }
   test_api_->RunMessageLoopUntilAnimationsDone();
   CheckAllItemsAreInBounds();
 }
 
 INSTANTIATE_TEST_CASE_P(LtrRtl, ShelfViewTextDirectionTest, testing::Bool());
-INSTANTIATE_TEST_CASE_P(VisibleBounds,
+
+// This test seems to be flaky with material design shelf (see
+// https://crbug.com/625671) and even in non-material mode (see
+// https://crbug.com/619344 which is fixed using a hack). Disabling for now.
+// TODO(mohsen): Hopefully, the fix for https://crbug.com/634128 that fixes
+// issues with shelf spacing, will fix this flake, too. Re-enable when that
+// issue is fixed.
+INSTANTIATE_TEST_CASE_P(DISABLED_VisibleBounds,
                         ShelfViewVisibleBoundsTest,
                         testing::Bool());
 
 namespace {
 
 // An InkDrop implementation that wraps another InkDrop instance to keep track
 // of state changes requested on it. Note that this will only track transitions
 // routed through AnimateToState() and not the ones performed directly on the
 // ripple inside the contained |ink_drop|.
 class InkDropSpy : public views::InkDrop {
@@ skipping 1188 matching lines @@
                 IsEmpty());
 
     ASSERT_TRUE(test_api_->IsShowingOverflowBubble());
   }
 }
 
 #endif  // !defined(OS_WIN)
 
 }  // namespace test
 }  // namespace ash