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

Unified Diff: components/test_runner/mock_web_user_media_client.cc

Issue 2357043003: Remove WebCallbackTask (Closed)
Patch Set: remove WebTaskRunner::postTask(base::Closure). git cl format Created 4 years, 3 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: components/test_runner/mock_web_user_media_client.cc
diff --git a/components/test_runner/mock_web_user_media_client.cc b/components/test_runner/mock_web_user_media_client.cc
index 1e84687d74afc986dd4fd17c62b333c2c2e61ab0..bd774a0d08999c055413d3ee462c8f62c266e367 100644
--- a/components/test_runner/mock_web_user_media_client.cc
+++ b/components/test_runner/mock_web_user_media_client.cc
@@ -10,7 +10,6 @@
#include "base/bind_helpers.h"
#include "base/logging.h"
#include "base/macros.h"
-#include "components/test_runner/web_task.h"
#include "components/test_runner/web_test_delegate.h"
#include "third_party/WebKit/public/platform/WebMediaConstraints.h"
#include "third_party/WebKit/public/platform/WebMediaDeviceInfo.h"
@@ -54,9 +53,9 @@ void MockWebUserMediaClient::requestUserMedia(
WebUserMediaRequest request = stream_request;
if (request.ownerDocument().isNull() || !request.ownerDocument().frame()) {
- delegate_->PostTask(new WebCallbackTask(base::Bind(
+ delegate_->PostTask(base::Bind(
&WebUserMediaRequest::requestFailed,
- base::Owned(new WebUserMediaRequest(request)), WebString())));
+ base::Owned(new WebUserMediaRequest(request)), WebString()));
return;
}
@@ -89,9 +88,9 @@ void MockWebUserMediaClient::requestUserMedia(
stream.addTrack(web_track);
}
- delegate_->PostTask(new WebCallbackTask(
+ delegate_->PostTask(
base::Bind(&WebUserMediaRequest::requestSucceeded,
- base::Owned(new WebUserMediaRequest(request)), stream)));
+ base::Owned(new WebUserMediaRequest(request)), stream));
}
void MockWebUserMediaClient::cancelUserMediaRequest(
@@ -143,9 +142,9 @@ void MockWebUserMediaClient::requestMediaDevices(
WebString::fromUTF8(test_devices[i].group_id));
}
- delegate_->PostTask(new WebCallbackTask(
+ delegate_->PostTask(
base::Bind(&WebMediaDevicesRequest::requestSucceeded,
- base::Owned(new WebMediaDevicesRequest(request)), devices)));
+ base::Owned(new WebMediaDevicesRequest(request)), devices));
should_enumerate_extra_device_ = !should_enumerate_extra_device_;
if (!media_device_change_observer_.isNull())
@@ -186,9 +185,9 @@ void MockWebUserMediaClient::requestSources(
test_sources[i].facing);
}
- delegate_->PostTask(new WebCallbackTask(base::Bind(
+ delegate_->PostTask(base::Bind(
&WebMediaStreamTrackSourcesRequest::requestSucceeded,
- base::Owned(new WebMediaStreamTrackSourcesRequest(request)), sources)));
+ base::Owned(new WebMediaStreamTrackSourcesRequest(request)), sources));
}
void MockWebUserMediaClient::setMediaDeviceChangeObserver(
« no previous file with comments | « components/test_runner/mock_web_speech_recognizer.cc ('k') | components/test_runner/mock_webrtc_data_channel_handler.cc » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698