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

Unified Diff: trunk/src/chrome/browser/extensions/api/image_writer_private/write_from_file_operation_unittest.cc

Issue 384513002: Revert 282130 "Adds API test for imageWriterPrivate.writeFromFile" (Closed) Base URL: svn://svn.chromium.org/chrome/
Patch Set: Created 6 years, 5 months 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 side-by-side diff with in-line comments
Download patch
Index: trunk/src/chrome/browser/extensions/api/image_writer_private/write_from_file_operation_unittest.cc
===================================================================
--- trunk/src/chrome/browser/extensions/api/image_writer_private/write_from_file_operation_unittest.cc (revision 282139)
+++ trunk/src/chrome/browser/extensions/api/image_writer_private/write_from_file_operation_unittest.cc (working copy)
@@ -27,10 +27,10 @@
scoped_refptr<WriteFromFileOperation> op =
new WriteFromFileOperation(manager_.AsWeakPtr(),
kDummyExtensionId,
- test_utils_.GetImagePath(),
- test_utils_.GetDevicePath().AsUTF8Unsafe());
+ test_image_path_,
+ test_device_path_.AsUTF8Unsafe());
- base::DeleteFile(test_utils_.GetImagePath(), false);
+ base::DeleteFile(test_image_path_, false);
EXPECT_CALL(manager_, OnProgress(kDummyExtensionId, _, _)).Times(0);
EXPECT_CALL(manager_, OnComplete(kDummyExtensionId)).Times(0);
@@ -50,8 +50,13 @@
scoped_refptr<WriteFromFileOperation> op =
new WriteFromFileOperation(manager_.AsWeakPtr(),
kDummyExtensionId,
- test_utils_.GetImagePath(),
- test_utils_.GetDevicePath().AsUTF8Unsafe());
+ test_image_path_,
+ test_device_path_.AsUTF8Unsafe());
+#if !defined(OS_CHROMEOS)
+ scoped_refptr<FakeImageWriterClient> client = FakeImageWriterClient::Create();
+ op->SetUtilityClientForTesting(client);
+#endif
+
EXPECT_CALL(manager_,
OnProgress(kDummyExtensionId, image_writer_api::STAGE_WRITE, _))
.Times(AnyNumber());
@@ -85,15 +90,15 @@
base::RunLoop().RunUntilIdle();
#if !defined(OS_CHROMEOS)
- test_utils_.GetUtilityClient()->Progress(0);
- test_utils_.GetUtilityClient()->Progress(50);
- test_utils_.GetUtilityClient()->Progress(100);
- test_utils_.GetUtilityClient()->Success();
+ client->Progress(0);
+ client->Progress(50);
+ client->Progress(100);
+ client->Success();
base::RunLoop().RunUntilIdle();
- test_utils_.GetUtilityClient()->Progress(0);
- test_utils_.GetUtilityClient()->Progress(50);
- test_utils_.GetUtilityClient()->Progress(100);
- test_utils_.GetUtilityClient()->Success();
+ client->Progress(0);
+ client->Progress(50);
+ client->Progress(100);
+ client->Success();
base::RunLoop().RunUntilIdle();
#endif
}

Powered by Google App Engine
This is Rietveld 408576698