Commit e8442a01 authored by maxmorin's avatar maxmorin Committed by Commit bot

Add some const to content/.

I think this would be nice as documentation, as these classes can be
quite confusing to people outside of content/ experts.

BUG=none
CQ_INCLUDE_TRYBOTS=master.tryserver.chromium.linux:linux_site_isolation

Review-Url: https://codereview.chromium.org/2727263004
Cr-Commit-Position: refs/heads/master@{#455768}
parent f28225ec
...@@ -891,28 +891,28 @@ class CONTENT_EXPORT RenderFrameHostImpl ...@@ -891,28 +891,28 @@ class CONTENT_EXPORT RenderFrameHostImpl
// we have a RenderViewHost for each RenderFrameHost. // we have a RenderViewHost for each RenderFrameHost.
// TODO(creis): RenderViewHost will eventually go away and be replaced with // TODO(creis): RenderViewHost will eventually go away and be replaced with
// some form of page context. // some form of page context.
RenderViewHostImpl* render_view_host_; RenderViewHostImpl* const render_view_host_;
RenderFrameHostDelegate* delegate_; RenderFrameHostDelegate* const delegate_;
// The SiteInstance associated with this RenderFrameHost. All content drawn // The SiteInstance associated with this RenderFrameHost. All content drawn
// in this RenderFrameHost is part of this SiteInstance. Cannot change over // in this RenderFrameHost is part of this SiteInstance. Cannot change over
// time. // time.
scoped_refptr<SiteInstanceImpl> site_instance_; const scoped_refptr<SiteInstanceImpl> site_instance_;
// The renderer process this RenderFrameHost is associated with. It is // The renderer process this RenderFrameHost is associated with. It is
// equivalent to the result of site_instance_->GetProcess(), but that // equivalent to the result of site_instance_->GetProcess(), but that
// method has the side effect of creating the process if it doesn't exist. // method has the side effect of creating the process if it doesn't exist.
// Cache a pointer to avoid unnecessary process creation. // Cache a pointer to avoid unnecessary process creation.
RenderProcessHost* process_; RenderProcessHost* const process_;
// Reference to the whole frame tree that this RenderFrameHost belongs to. // Reference to the whole frame tree that this RenderFrameHost belongs to.
// Allows this RenderFrameHost to add and remove nodes in response to // Allows this RenderFrameHost to add and remove nodes in response to
// messages from the renderer requesting DOM manipulation. // messages from the renderer requesting DOM manipulation.
FrameTree* frame_tree_; FrameTree* const frame_tree_;
// The FrameTreeNode which this RenderFrameHostImpl is hosted in. // The FrameTreeNode which this RenderFrameHostImpl is hosted in.
FrameTreeNode* frame_tree_node_; FrameTreeNode* const frame_tree_node_;
// The active parent RenderFrameHost for this frame, if it is a subframe. // The active parent RenderFrameHost for this frame, if it is a subframe.
// Null for the main frame. This is cached because the parent FrameTreeNode // Null for the main frame. This is cached because the parent FrameTreeNode
......
...@@ -505,7 +505,7 @@ class CONTENT_EXPORT RenderProcessHostImpl ...@@ -505,7 +505,7 @@ class CONTENT_EXPORT RenderProcessHostImpl
// called. // called.
int instance_id_ = 1; int instance_id_ = 1;
BrowserContext* browser_context_; BrowserContext* const browser_context_;
// Owned by |browser_context_|. // Owned by |browser_context_|.
StoragePartitionImpl* storage_partition_impl_; StoragePartitionImpl* storage_partition_impl_;
......
Markdown is supported
0%
or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment