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

Side by Side Diff: android_webview/native/aw_contents.cc

Issue 2258823002: Re-write many calls to WrapUnique() with MakeUnique() (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: Created 4 years, 4 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 2012 The Chromium Authors. All rights reserved. 1 // Copyright 2012 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 "android_webview/native/aw_contents.h" 5 #include "android_webview/native/aw_contents.h"
6 6
7 #include <limits> 7 #include <limits>
8 #include <utility> 8 #include <utility>
9 9
10 #include "android_webview/browser/aw_browser_context.h" 10 #include "android_webview/browser/aw_browser_context.h"
(...skipping 216 matching lines...) Expand 10 before | Expand all | Expand 10 after
227 227
228 contents_client_bridge_.reset( 228 contents_client_bridge_.reset(
229 new AwContentsClientBridge(env, contents_client_bridge)); 229 new AwContentsClientBridge(env, contents_client_bridge));
230 AwContentsClientBridgeBase::Associate(web_contents_.get(), 230 AwContentsClientBridgeBase::Associate(web_contents_.get(),
231 contents_client_bridge_.get()); 231 contents_client_bridge_.get());
232 232
233 AwContentsIoThreadClientImpl::Associate(web_contents_.get(), 233 AwContentsIoThreadClientImpl::Associate(web_contents_.get(),
234 io_thread_client); 234 io_thread_client);
235 235
236 InterceptNavigationDelegate::Associate( 236 InterceptNavigationDelegate::Associate(
237 web_contents_.get(), base::WrapUnique(new InterceptNavigationDelegate( 237 web_contents_.get(), base::MakeUnique<InterceptNavigationDelegate>(
238 env, intercept_navigation_delegate))); 238 env, intercept_navigation_delegate));
239 239
240 // Finally, having setup the associations, release any deferred requests 240 // Finally, having setup the associations, release any deferred requests
241 for (content::RenderFrameHost* rfh : web_contents_->GetAllFrames()) { 241 for (content::RenderFrameHost* rfh : web_contents_->GetAllFrames()) {
242 int render_process_id = rfh->GetProcess()->GetID(); 242 int render_process_id = rfh->GetProcess()->GetID();
243 int render_frame_id = rfh->GetRoutingID(); 243 int render_frame_id = rfh->GetRoutingID();
244 AwResourceDispatcherHostDelegate::OnIoThreadClientReady(render_process_id, 244 AwResourceDispatcherHostDelegate::OnIoThreadClientReady(render_process_id,
245 render_frame_id); 245 render_frame_id);
246 } 246 }
247 } 247 }
248 248
(...skipping 1036 matching lines...) Expand 10 before | Expand all | Expand 10 after
1285 int routing_id = new_host->GetRoutingID(); 1285 int routing_id = new_host->GetRoutingID();
1286 // At this point, the current RVH may or may not contain a compositor. So 1286 // At this point, the current RVH may or may not contain a compositor. So
1287 // compositor_ may be nullptr, in which case 1287 // compositor_ may be nullptr, in which case
1288 // BrowserViewRenderer::DidInitializeCompositor() callback is time when the 1288 // BrowserViewRenderer::DidInitializeCompositor() callback is time when the
1289 // new compositor is constructed. 1289 // new compositor is constructed.
1290 browser_view_renderer_.SetActiveCompositorID( 1290 browser_view_renderer_.SetActiveCompositorID(
1291 CompositorID(process_id, routing_id)); 1291 CompositorID(process_id, routing_id));
1292 } 1292 }
1293 1293
1294 } // namespace android_webview 1294 } // namespace android_webview
OLDNEW
« no previous file with comments | « android_webview/native/android_protocol_handler.cc ('k') | android_webview/native/aw_contents_client_bridge_unittest.cc » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698