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

Side by Side Diff: chrome/utility/image_writer/image_writer_unittest.cc

Issue 653773004: Standardize usage of virtual/override/final in chrome/ (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: Created 6 years, 2 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 unified diff | Download patch
OLDNEW
1 // Copyright 2014 The Chromium Authors. All rights reserved. 1 // Copyright 2014 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 <string> 5 #include <string>
6 6
7 #include "base/files/file_util.h" 7 #include "base/files/file_util.h"
8 #include "base/files/scoped_temp_dir.h" 8 #include "base/files/scoped_temp_dir.h"
9 #include "base/run_loop.h" 9 #include "base/run_loop.h"
10 #include "chrome/utility/image_writer/error_messages.h" 10 #include "chrome/utility/image_writer/error_messages.h"
(...skipping 49 matching lines...) Expand 10 before | Expand all | Expand 10 after
60 MOCK_METHOD0(SendSucceeded, void()); 60 MOCK_METHOD0(SendSucceeded, void());
61 MOCK_METHOD1(OnMessageReceived, bool(const IPC::Message& message)); 61 MOCK_METHOD1(OnMessageReceived, bool(const IPC::Message& message));
62 }; 62 };
63 63
64 // This Mock has the additional feature that it will start verification when 64 // This Mock has the additional feature that it will start verification when
65 // the write completes. 65 // the write completes.
66 class VerifyingHandler : public MockHandler { 66 class VerifyingHandler : public MockHandler {
67 public: 67 public:
68 VerifyingHandler() : image_writer_(NULL), verified_(false) {} 68 VerifyingHandler() : image_writer_(NULL), verified_(false) {}
69 69
70 virtual void SendSucceeded() override { 70 void SendSucceeded() override {
71 MockHandler::SendSucceeded(); 71 MockHandler::SendSucceeded();
72 if (!verified_) { 72 if (!verified_) {
73 image_writer_->Verify(); 73 image_writer_->Verify();
74 verified_ = true; 74 verified_ = true;
75 } 75 }
76 } 76 }
77 ImageWriter* image_writer_; 77 ImageWriter* image_writer_;
78 78
79 private: 79 private:
80 bool verified_; 80 bool verified_;
(...skipping 140 matching lines...) Expand 10 before | Expand all | Expand 10 after
221 EXPECT_CALL(mock_handler, SendFailed(_)).Times(0); 221 EXPECT_CALL(mock_handler, SendFailed(_)).Times(0);
222 222
223 FillDefault(image_path_); 223 FillDefault(image_path_);
224 224
225 image_writer.Write(); 225 image_writer.Write();
226 226
227 base::RunLoop().RunUntilIdle(); 227 base::RunLoop().RunUntilIdle();
228 } 228 }
229 229
230 } // namespace image_writer 230 } // namespace image_writer
OLDNEW
« no previous file with comments | « chrome/utility/image_writer/image_writer_handler.h ('k') | chrome/utility/importer/bookmarks_file_importer.h » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698