Move FrameMsg_NewFrame/NewFrameProxy to mojom
Straightforward conversion, no surprises.
Also changes some (re-)initialization details of RPHI. Namely:
* The mojom::RouteProvider and mojom::Renderer associated
interface proxies are acquired immediately upon Channel creation,
before pausing the Channel. This avoids an issue where one of
these interface requests could be lazily acquired during Channel
pause, blocking a subsequent message on that interface which
should be sent immediately while the Channel is unpaused.
See https://goo.gl/ot0S32 for some explanation of this behavior.
* Any cached associated interface proxies are reset in
ProcessDied() rather than Init(), per nasko@'s suggestion.
This clarifies that they are per-process state.
BUG=612500
CQ_INCLUDE_TRYBOTS=master.tryserver.chromium.linux:linux_site_isolation
Review-Url: https://codereview.chromium.org/2383853003
Cr-Commit-Position: refs/heads/master@{#422900}
diff --git a/content/renderer/render_thread_impl.h b/content/renderer/render_thread_impl.h
index fff45f3..a5eba97 100644
--- a/content/renderer/render_thread_impl.h
+++ b/content/renderer/render_thread_impl.h
@@ -50,7 +50,6 @@
class GrContext;
class SkBitmap;
-struct FrameMsg_NewFrame_Params;
struct ViewMsg_UpdateScrollbarTheme_Params;
struct WorkerProcessMsg_CreateWorker_Params;
@@ -502,12 +501,6 @@
void InitializeWebKit(
scoped_refptr<base::SingleThreadTaskRunner>& resource_task_queue);
- void OnCreateNewFrame(FrameMsg_NewFrame_Params params);
- void OnCreateNewFrameProxy(int routing_id,
- int render_view_routing_id,
- int opener_routing_id,
- int parent_routing_id,
- const FrameReplicationState& replicated_state);
void OnTransferBitmap(const SkBitmap& bitmap, int resource_id);
#if defined(ENABLE_PLUGINS)
void OnPurgePluginListCache(bool reload_pages);
@@ -519,6 +512,12 @@
// mojom::Renderer:
void CreateView(mojom::CreateViewParamsPtr params) override;
+ void CreateFrame(mojom::CreateFrameParamsPtr params) override;
+ void CreateFrameProxy(int32_t routing_id,
+ int32_t render_view_routing_id,
+ int32_t opener_routing_id,
+ int32_t parent_routing_id,
+ const FrameReplicationState& replicated_state) override;
// device::mojom::TimeZoneClient:
void OnTimeZoneChange(const std::string& zoneId) override;
