Unify frame IDs with RenderFrameHost routing IDs.

android_webview/renderer/print_web_view_helper.cc

Index: android_webview/renderer/print_web_view_helper.cc
diff --git a/android_webview/renderer/print_web_view_helper.cc b/android_webview/renderer/print_web_view_helper.cc
index 6039f940a092af61a31822d0e22b33e51d9f0b28..7fc7ecb5f6b921749fd554df942263b2c4baef93 100644
--- a/android_webview/renderer/print_web_view_helper.cc
+++ b/android_webview/renderer/print_web_view_helper.cc
@@ -654,7 +654,9 @@ void PrepareFrameAndViewForPrint::CopySelection(
   blink::WebView* web_view = blink::WebView::create(this);
   owns_web_view_ = true;
   content::ApplyWebPreferences(prefs, web_view);
-  web_view->setMainFrame(blink::WebFrame::create(this));
+
+  // We don't need to use the frame ID for this frame, so we pass 0 for it.
+  web_view->setMainFrame(blink::WebFrame::create(this, 0));

[awong, 2014/02/20 00:49:16]

Are we going to document the "0 means don't care" in the Blink layer?

This seems like an odd API semantic though perhaps it's unavoidable?

[Charlie Reis, 2014/02/20 00:51:32]

Daniel can correct me if I'm wrong, but I think we'd like to eventually find a
way to not have other creators of WebFrames.  Maybe these classes can hold
RenderFrames, for example.

Anyway, he's working on a Blink CL to remove generateEmbedderIdentifier(), so
perhaps he can add a comment like that.

[dcheng, 2014/02/20 01:28:03]

I've done that in https://codereview.chromium.org/172543005/

   frame_.Reset(web_view->mainFrame());
   node_to_print_.reset();