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

Unified Diff: content/public/test/mock_render_thread.cc

Issue 2824193002: Enable use_once_callback for //content/common/*.mojom (Closed)
Patch Set: push_messaging fix Created 3 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 side-by-side diff with in-line comments
Download patch
Index: content/public/test/mock_render_thread.cc
diff --git a/content/public/test/mock_render_thread.cc b/content/public/test/mock_render_thread.cc
index c953a5d206d4ea5511e8e3791a0d461bdf6f2e6d..3298c8934e83f3a4739c3d4f1c07b75ec44facae 100644
--- a/content/public/test/mock_render_thread.cc
+++ b/content/public/test/mock_render_thread.cc
@@ -36,13 +36,13 @@ class MockRenderMessageFilterImpl : public mojom::RenderMessageFilter {
~MockRenderMessageFilterImpl() override {}
// mojom::RenderMessageFilter:
- void GenerateRoutingID(const GenerateRoutingIDCallback& callback) override {
+ void GenerateRoutingID(GenerateRoutingIDCallback callback) override {
NOTREACHED();
- callback.Run(MSG_ROUTING_NONE);
+ std::move(callback).Run(MSG_ROUTING_NONE);
}
void CreateNewWindow(mojom::CreateNewWindowParamsPtr params,
- const CreateNewWindowCallback& callback) override {
+ CreateNewWindowCallback callback) override {
// NOTE: This implementation of mojom::RenderMessageFilter is used client-
// side only. Because sync mojom methods have a different interface for
// bindings- and client-side, we only implement the client-side interface
@@ -58,8 +58,8 @@ class MockRenderMessageFilterImpl : public mojom::RenderMessageFilter {
}
void CreateNewWidget(int32_t opener_id,
- blink::WebPopupType popup_type,
- const CreateNewWidgetCallback& callback) override {
+ blink::WebPopupType popup_type,
+ CreateNewWidgetCallback callback) override {
// See comment in CreateNewWindow().
NOTREACHED();
}
@@ -73,7 +73,7 @@ class MockRenderMessageFilterImpl : public mojom::RenderMessageFilter {
void CreateFullscreenWidget(
int opener_id,
- const CreateFullscreenWidgetCallback& callback) override {
+ CreateFullscreenWidgetCallback callback) override {
NOTREACHED();
}

Powered by Google App Engine
This is Rietveld 408576698