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

Side by Side Diff: components/test_runner/mock_color_chooser.cc

Issue 1852603002: Replacing most of web_task.h with base::Closure + base::WeakPtrFactory. (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@move-mocks-to-test-runner
Patch Set: Rebasing... Created 4 years, 8 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 "components/test_runner/mock_color_chooser.h" 5 #include "components/test_runner/mock_color_chooser.h"
6 6
7 #include "base/bind.h"
7 #include "components/test_runner/test_runner.h" 8 #include "components/test_runner/test_runner.h"
9 #include "components/test_runner/web_task.h"
8 #include "components/test_runner/web_test_delegate.h" 10 #include "components/test_runner/web_test_delegate.h"
9 11
10 namespace test_runner { 12 namespace test_runner {
11 13
12 namespace {
13 class HostMethodTask : public WebMethodTask<MockColorChooser> {
14 public:
15 typedef void (MockColorChooser::*CallbackMethodType)();
16 HostMethodTask(MockColorChooser* object, CallbackMethodType callback)
17 : WebMethodTask<MockColorChooser>(object),
18 callback_(callback) {}
19
20 void RunIfValid() override { (object_->*callback_)(); }
21
22 private:
23 CallbackMethodType callback_;
24 };
25
26 } // namespace
27
28 MockColorChooser::MockColorChooser(blink::WebColorChooserClient* client, 14 MockColorChooser::MockColorChooser(blink::WebColorChooserClient* client,
29 WebTestDelegate* delegate, 15 WebTestDelegate* delegate,
30 TestRunner* test_runner) 16 TestRunner* test_runner)
31 : client_(client), delegate_(delegate), test_runner_(test_runner) { 17 : client_(client),
18 delegate_(delegate),
19 test_runner_(test_runner),
20 weak_factory_(this) {
32 test_runner_->DidOpenChooser(); 21 test_runner_->DidOpenChooser();
33 } 22 }
34 23
35 MockColorChooser::~MockColorChooser() { 24 MockColorChooser::~MockColorChooser() {
36 test_runner_->DidCloseChooser(); 25 test_runner_->DidCloseChooser();
37 } 26 }
38 27
39 void MockColorChooser::setSelectedColor(const blink::WebColor color) {} 28 void MockColorChooser::setSelectedColor(const blink::WebColor color) {}
40 29
41 void MockColorChooser::endChooser() { 30 void MockColorChooser::endChooser() {
42 delegate_->PostDelayedTask( 31 delegate_->PostDelayedTask(
43 new HostMethodTask(this, &MockColorChooser::InvokeDidEndChooser), 0); 32 new WebCallbackTask(base::Bind(&MockColorChooser::InvokeDidEndChooser,
33 weak_factory_.GetWeakPtr())),
34 0);
44 } 35 }
45 36
46 void MockColorChooser::InvokeDidEndChooser() { 37 void MockColorChooser::InvokeDidEndChooser() {
47 client_->didEndChooser(); 38 client_->didEndChooser();
48 } 39 }
49 40
50 } // namespace test_runner 41 } // namespace test_runner
OLDNEW
« no previous file with comments | « components/test_runner/mock_color_chooser.h ('k') | components/test_runner/mock_web_media_stream_center.h » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698