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

Unified Diff: android_webview/native/aw_contents.cc

Issue 1658003002: Enable range-based for loops to loop over a WebContents' frame hosts. (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: Fix miswrapped comment. Created 4 years, 10 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 | « no previous file | chrome/browser/site_details.cc » ('j') | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: android_webview/native/aw_contents.cc
diff --git a/android_webview/native/aw_contents.cc b/android_webview/native/aw_contents.cc
index 523c63772923cf611ed6c97dc7e318ca50a76687..3aa233712d3c7e6d85780beabf349db51cba5348 100644
--- a/android_webview/native/aw_contents.cc
+++ b/android_webview/native/aw_contents.cc
@@ -137,13 +137,6 @@ class AwContentsUserData : public base::SupportsUserData::Data {
base::subtle::Atomic32 g_instance_count = 0;
-void OnIoThreadClientReady(content::RenderFrameHost* rfh) {
- int render_process_id = rfh->GetProcess()->GetID();
- int render_frame_id = rfh->GetRoutingID();
- AwResourceDispatcherHostDelegate::OnIoThreadClientReady(
- render_process_id, render_frame_id);
-}
-
} // namespace
// static
@@ -245,7 +238,12 @@ void AwContents::SetJavaPeers(
env, intercept_navigation_delegate)));
// Finally, having setup the associations, release any deferred requests
- web_contents_->ForEachFrame(base::Bind(&OnIoThreadClientReady));
+ for (content::RenderFrameHost* rfh : web_contents_->GetAllFrames()) {
+ int render_process_id = rfh->GetProcess()->GetID();
+ int render_frame_id = rfh->GetRoutingID();
+ AwResourceDispatcherHostDelegate::OnIoThreadClientReady(render_process_id,
+ render_frame_id);
+ }
}
void AwContents::SetSaveFormData(bool enabled) {
« no previous file with comments | « no previous file | chrome/browser/site_details.cc » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698