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

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

Issue 2357043003: Remove WebCallbackTask (Closed)
Patch Set: remove WebTaskRunner::postTask(base::Closure). git cl format Created 4 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 "components/test_runner/mock_webrtc_dtmf_sender_handler.h" 5 #include "components/test_runner/mock_webrtc_dtmf_sender_handler.h"
6 6
7 #include "base/bind.h" 7 #include "base/bind.h"
8 #include "base/bind_helpers.h" 8 #include "base/bind_helpers.h"
9 #include "base/logging.h" 9 #include "base/logging.h"
10 #include "components/test_runner/web_task.h"
11 #include "components/test_runner/web_test_delegate.h" 10 #include "components/test_runner/web_test_delegate.h"
12 #include "third_party/WebKit/public/platform/WebMediaStreamSource.h" 11 #include "third_party/WebKit/public/platform/WebMediaStreamSource.h"
13 #include "third_party/WebKit/public/platform/WebRTCDTMFSenderHandlerClient.h" 12 #include "third_party/WebKit/public/platform/WebRTCDTMFSenderHandlerClient.h"
14 13
15 using namespace blink; 14 using namespace blink;
16 15
17 namespace test_runner { 16 namespace test_runner {
18 17
19 MockWebRTCDTMFSenderHandler::MockWebRTCDTMFSenderHandler( 18 MockWebRTCDTMFSenderHandler::MockWebRTCDTMFSenderHandler(
20 const WebMediaStreamTrack& track, 19 const WebMediaStreamTrack& track,
(...skipping 22 matching lines...) Expand all
43 bool MockWebRTCDTMFSenderHandler::insertDTMF(const WebString& tones, 42 bool MockWebRTCDTMFSenderHandler::insertDTMF(const WebString& tones,
44 long duration, 43 long duration,
45 long inter_tone_gap) { 44 long inter_tone_gap) {
46 DCHECK(client_); 45 DCHECK(client_);
47 if (!canInsertDTMF()) 46 if (!canInsertDTMF())
48 return false; 47 return false;
49 48
50 tone_buffer_ = tones; 49 tone_buffer_ = tones;
51 base::Closure closure = base::Bind(&MockWebRTCDTMFSenderHandler::PlayTone, 50 base::Closure closure = base::Bind(&MockWebRTCDTMFSenderHandler::PlayTone,
52 weak_factory_.GetWeakPtr()); 51 weak_factory_.GetWeakPtr());
53 delegate_->PostTask(new WebCallbackTask(closure)); 52 delegate_->PostTask(closure);
54 delegate_->PostTask(new WebCallbackTask(closure)); 53 delegate_->PostTask(closure);
55 return true; 54 return true;
56 } 55 }
57 56
58 void MockWebRTCDTMFSenderHandler::PlayTone() { 57 void MockWebRTCDTMFSenderHandler::PlayTone() {
59 WebString tones = currentToneBuffer(); 58 WebString tones = currentToneBuffer();
60 ClearToneBuffer(); 59 ClearToneBuffer();
61 client_->didPlayTone(tones); 60 client_->didPlayTone(tones);
62 } 61 }
63 62
64 } // namespace test_runner 63 } // namespace test_runner
OLDNEW

Powered by Google App Engine
This is Rietveld 408576698