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

Side by Side Diff: chrome/browser/printing/print_dialog_cloud_unittest.cc

Issue 7477008: Remove explicit keyword from multi-argument constructors (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src
Patch Set: Created 9 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 unified diff | Download patch | Annotate | Revision Log
OLDNEW
1 // Copyright (c) 2011 The Chromium Authors. All rights reserved. 1 // Copyright (c) 2011 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/printing/print_dialog_cloud.h" 5 #include "chrome/browser/printing/print_dialog_cloud.h"
6 #include "chrome/browser/printing/print_dialog_cloud_internal.h" 6 #include "chrome/browser/printing/print_dialog_cloud_internal.h"
7 7
8 #include <string> 8 #include <string>
9 #include <vector> 9 #include <vector>
10 10
(...skipping 63 matching lines...) Expand 10 before | Expand all | Expand 10 after
74 << "' (expected '" << expected_string << "')"; 74 << "' (expected '" << expected_string << "')";
75 return false; 75 return false;
76 } 76 }
77 77
78 namespace internal_cloud_print_helpers { 78 namespace internal_cloud_print_helpers {
79 79
80 class MockCloudPrintFlowHandler 80 class MockCloudPrintFlowHandler
81 : public CloudPrintFlowHandler, 81 : public CloudPrintFlowHandler,
82 public base::SupportsWeakPtr<MockCloudPrintFlowHandler> { 82 public base::SupportsWeakPtr<MockCloudPrintFlowHandler> {
83 public: 83 public:
84 explicit MockCloudPrintFlowHandler(const FilePath& path, 84 MockCloudPrintFlowHandler(const FilePath& path,
85 const string16& title, 85 const string16& title,
86 const std::string& file_type) 86 const std::string& file_type)
87 : CloudPrintFlowHandler(path, title, file_type) {} 87 : CloudPrintFlowHandler(path, title, file_type) {}
88 MOCK_METHOD0(DestructorCalled, void()); 88 MOCK_METHOD0(DestructorCalled, void());
89 MOCK_METHOD0(RegisterMessages, void()); 89 MOCK_METHOD0(RegisterMessages, void());
90 MOCK_METHOD3(Observe, 90 MOCK_METHOD3(Observe,
91 void(int type, 91 void(int type,
92 const NotificationSource& source, 92 const NotificationSource& source,
93 const NotificationDetails& details)); 93 const NotificationDetails& details));
94 MOCK_METHOD1(SetDialogDelegate, 94 MOCK_METHOD1(SetDialogDelegate,
95 void(CloudPrintHtmlDialogDelegate* delegate)); 95 void(CloudPrintHtmlDialogDelegate* delegate));
96 MOCK_METHOD0(CreateCloudPrintDataSender, 96 MOCK_METHOD0(CreateCloudPrintDataSender,
(...skipping 241 matching lines...) Expand 10 before | Expand all | Expand 10 after
338 delegate_->GetWebUIMessageHandlers(&handlers); 338 delegate_->GetWebUIMessageHandlers(&handlers);
339 delegate_.reset(); 339 delegate_.reset();
340 EXPECT_THAT(mock_flow_handler_.get(), NotNull()); 340 EXPECT_THAT(mock_flow_handler_.get(), NotNull());
341 } 341 }
342 342
343 // Testing for ExternalHtmlDialogUI needs a mock TabContents, mock 343 // Testing for ExternalHtmlDialogUI needs a mock TabContents, mock
344 // CloudPrintHtmlDialogDelegate (provided through the mock 344 // CloudPrintHtmlDialogDelegate (provided through the mock
345 // tab_contents) 345 // tab_contents)
346 346
347 // Testing for PrintDialogCloud needs a mock Browser. 347 // Testing for PrintDialogCloud needs a mock Browser.
OLDNEW
« no previous file with comments | « chrome/browser/printing/print_dialog_cloud_internal.h ('k') | chrome/browser/process_singleton_uitest.cc » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698