Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(678)

Side by Side Diff: chrome/browser/media_galleries/fileapi/picasa_data_provider_browsertest.cc

Issue 2520623006: Disable flaky PicasaDataProviderNoDatabaseGetListTest.NoDatabaseGetList (Closed)
Patch Set: Created 4 years, 1 month ago
Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.
Jump to:
View unified diff | Download patch
« no previous file with comments | « no previous file | no next file » | no next file with comments »
Toggle Intra-line Diffs ('i') | Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
OLDNEW
1 // Copyright 2013 The Chromium Authors. All rights reserved. 1 // Copyright 2013 The Chromium Authors. All rights reserved.
2 // Use of this source code is governed by a BSD-style license that can be 2 // Use of this source code is governed by a BSD-style license that can be
3 // found in the LICENSE file. 3 // found in the LICENSE file.
4 4
5 #include "chrome/browser/media_galleries/fileapi/picasa_data_provider.h" 5 #include "chrome/browser/media_galleries/fileapi/picasa_data_provider.h"
6 6
7 #include <memory> 7 #include <memory>
8 #include <utility> 8 #include <utility>
9 #include <vector> 9 #include <vector>
10 10
(...skipping 277 matching lines...) Expand 10 before | Expand all | Expand 10 after
288 protected: 288 protected:
289 PicasaDataProvider::DataType RequestedDataType() const override { 289 PicasaDataProvider::DataType RequestedDataType() const override {
290 return PicasaDataProvider::LIST_OF_ALBUMS_AND_FOLDERS_DATA; 290 return PicasaDataProvider::LIST_OF_ALBUMS_AND_FOLDERS_DATA;
291 } 291 }
292 void VerifyRefreshResults(bool parse_success) override { 292 void VerifyRefreshResults(bool parse_success) override {
293 EXPECT_FALSE(parse_success); 293 EXPECT_FALSE(parse_success);
294 TestDone(); 294 TestDone();
295 } 295 }
296 }; 296 };
297 297
298 // This test is flaky. See crbug.com/667059
298 IN_PROC_BROWSER_TEST_F(PicasaDataProviderNoDatabaseGetListTest, 299 IN_PROC_BROWSER_TEST_F(PicasaDataProviderNoDatabaseGetListTest,
299 NoDatabaseGetList) { 300 DISABLED_NoDatabaseGetList) {
300 RunTest(); 301 RunTest();
301 } 302 }
302 303
303 class PicasaDataProviderNoDatabaseGetAlbumsImagesTest 304 class PicasaDataProviderNoDatabaseGetAlbumsImagesTest
304 : public PicasaDataProviderTest { 305 : public PicasaDataProviderTest {
305 protected: 306 protected:
306 PicasaDataProvider::DataType RequestedDataType() const override { 307 PicasaDataProvider::DataType RequestedDataType() const override {
307 return PicasaDataProvider::ALBUMS_IMAGES_DATA; 308 return PicasaDataProvider::ALBUMS_IMAGES_DATA;
308 } 309 }
309 void VerifyRefreshResults(bool parse_success) override { 310 void VerifyRefreshResults(bool parse_success) override {
(...skipping 248 matching lines...) Expand 10 before | Expand all | Expand 10 after
558 base::Unretained(this))); 559 base::Unretained(this)));
559 } 560 }
560 }; 561 };
561 562
562 IN_PROC_BROWSER_TEST_F(PicasaDataProviderInvalidateInflightAlbumsIndexerTest, 563 IN_PROC_BROWSER_TEST_F(PicasaDataProviderInvalidateInflightAlbumsIndexerTest,
563 InvalidateInflightAlbumsIndexerTest) { 564 InvalidateInflightAlbumsIndexerTest) {
564 RunTest(); 565 RunTest();
565 } 566 }
566 567
567 } // namespace picasa 568 } // namespace picasa
OLDNEW
« no previous file with comments | « no previous file | no next file » | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698