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

Unified Diff: content/renderer/render_view_impl.cc

Issue 2381493003: Move ViewMsg_New to mojom (Closed)
Patch Set: . 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
« no previous file with comments | « content/renderer/render_view_impl.h ('k') | content/test/layouttest_support.cc » ('j') | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: content/renderer/render_view_impl.cc
diff --git a/content/renderer/render_view_impl.cc b/content/renderer/render_view_impl.cc
index 38554435d10dc67adcd4483912e8892f7bba896e..7ca413ddeb9f0fc7e806b4dadf0cef5eddfc1e8d 100644
--- a/content/renderer/render_view_impl.cc
+++ b/content/renderer/render_view_impl.cc
@@ -293,7 +293,7 @@ const size_t kContentIntentDelayMilliseconds = 700;
static RenderViewImpl* (*g_create_render_view_impl)(
CompositorDependencies* compositor_deps,
- const ViewMsg_New_Params&) = nullptr;
+ const mojom::CreateViewParams&) = nullptr;
// static
Referrer RenderViewImpl::GetReferrerFromRequest(
@@ -660,7 +660,7 @@ class AlwaysDrawSwapPromise : public cc::SwapPromise {
} // namespace
RenderViewImpl::RenderViewImpl(CompositorDependencies* compositor_deps,
- const ViewMsg_New_Params& params)
+ const mojom::CreateViewParams& params)
: RenderWidget(compositor_deps,
blink::WebPopupTypeNone,
params.initial_size.screen_info,
@@ -700,7 +700,7 @@ RenderViewImpl::RenderViewImpl(CompositorDependencies* compositor_deps,
GetWidget()->set_owner_delegate(this);
}
-void RenderViewImpl::Initialize(const ViewMsg_New_Params& params,
+void RenderViewImpl::Initialize(const mojom::CreateViewParams& params,
bool was_created_by_renderer) {
SetRoutingID(params.view_id);
@@ -1189,7 +1189,7 @@ void RenderView::ApplyWebPreferences(const WebPreferences& prefs,
/*static*/
RenderViewImpl* RenderViewImpl::Create(CompositorDependencies* compositor_deps,
- const ViewMsg_New_Params& params,
+ const mojom::CreateViewParams& params,
bool was_created_by_renderer) {
DCHECK(params.view_id != MSG_ROUTING_NONE);
RenderViewImpl* render_view = NULL;
@@ -1205,7 +1205,7 @@ RenderViewImpl* RenderViewImpl::Create(CompositorDependencies* compositor_deps,
// static
void RenderViewImpl::InstallCreateHook(RenderViewImpl* (
*create_render_view_impl)(CompositorDependencies* compositor_deps,
- const ViewMsg_New_Params&)) {
+ const mojom::CreateViewParams&)) {
CHECK(!g_create_render_view_impl);
g_create_render_view_impl = create_render_view_impl;
}
@@ -1574,7 +1574,7 @@ WebView* RenderViewImpl::createView(WebLocalFrame* creator,
// return from this call synchronously, we just have to make our best guess
// and rely on the browser sending a WasHidden / WasShown message if it
// disagrees.
- ViewMsg_New_Params view_params;
+ mojom::CreateViewParams view_params;
RenderFrameImpl* creator_frame = RenderFrameImpl::FromWebFrame(creator);
view_params.opener_frame_route_id = creator_frame->GetRoutingID();
« no previous file with comments | « content/renderer/render_view_impl.h ('k') | content/test/layouttest_support.cc » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698