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

Unified Diff: content/browser/web_contents/web_contents_impl.h

Issue 23018005: Start renderer in ContentViewCoreImpl::EvaluateJavaScript. (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: Address comments Created 7 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 side-by-side diff with in-line comments
Download patch
Index: content/browser/web_contents/web_contents_impl.h
diff --git a/content/browser/web_contents/web_contents_impl.h b/content/browser/web_contents/web_contents_impl.h
index c653c1aa97a2bdc4c4a8bfde380dea1c32121fbf..8ab662d16d9c209a3c529f6365092a421f34bbd0 100644
--- a/content/browser/web_contents/web_contents_impl.h
+++ b/content/browser/web_contents/web_contents_impl.h
@@ -153,6 +153,10 @@ class CONTENT_EXPORT WebContentsImpl
JavaBridgeDispatcherHostManager* java_bridge_dispatcher_host_manager() const {
return java_bridge_dispatcher_host_manager_.get();
}
+
+ // In Android WebView, the RenderView needs created even there is no
+ // navigation entry, so the JavaScript could executed without loading page.
+ bool CreateRenderViewWithoutNavigationEntry(RenderViewHost* render_view_host);
Charlie Reis 2013/08/20 22:03:49 This is not ok to add. For security reasons, ther
Charlie Reis 2013/08/21 22:06:26 If we are going to go down this road, we need to b
#endif
// Expose the render manager for testing.
@@ -751,6 +755,10 @@ class CONTENT_EXPORT WebContentsImpl
// Clear all PowerSaveBlockers, leave power_save_blocker_ empty.
void ClearAllPowerSaveBlockers();
+ bool CreateRenderView(RenderViewHost* render_view_host,
Charlie Reis 2013/08/20 22:03:49 Please do not add this. max_page_id is an extreme
+ int opener_route_id,
+ int max_page_id);
+
// Data for core operation ---------------------------------------------------
// Delegate for notifying our owner about stuff. Not owned by us.

Powered by Google App Engine
This is Rietveld 408576698