Remove usage of RenderViewHost which aren't needed anymore.

Remove usage of RenderViewHost which aren't needed anymore.

BUG=304341
Committed: https://crrev.com/55b3eab0bc43d35f7883dde31d06d122d2c20f47
Cr-Commit-Position: refs/heads/master@{#314648}

[nasko, 2015-02-04 19:50:16]

nasko@chromium.org changed reviewers:
+ creis@chromium.org

[nasko, 2015-02-04 19:50:17]

Hey Charlie,
This is some simple cleanup, which you suggested should be in a separate CL. Can
you take a look?
Thanks in advance!
Nasko

[Charlie Reis, 2015-02-04 20:37:14]

Let's use 304341 as the bug number.  LGTM with nits!

https://codereview.chromium.org/902683002/diff/1/content/browser/frame_host/n...
File content/browser/frame_host/navigator_impl.cc (right):

https://codereview.chromium.org/902683002/diff/1/content/browser/frame_host/n...
content/browser/frame_host/navigator_impl.cc:268:
static_cast<SiteInstanceImpl*>(render_frame_host->GetSiteInstance()));
We can drop the static_cast here.  (Co-variant return types FTW.)

https://codereview.chromium.org/902683002/diff/1/content/browser/frame_host/n...
content/browser/frame_host/navigator_impl.cc:570:
static_cast<SiteInstanceImpl*>(render_frame_host->GetSiteInstance());
Let's drop the cast here as well.

[nasko, 2015-02-04 20:41:08]

New patchsets have been uploaded after l-g-t-m from creis@chromium.org

[nasko, 2015-02-04 20:42:44]

Fixed the nits.

https://codereview.chromium.org/902683002/diff/1/content/browser/frame_host/n...
File content/browser/frame_host/navigator_impl.cc (right):

https://codereview.chromium.org/902683002/diff/1/content/browser/frame_host/n...
content/browser/frame_host/navigator_impl.cc:268:
static_cast<SiteInstanceImpl*>(render_frame_host->GetSiteInstance()));
On 2015/02/04 20:37:14, Charlie Reis wrote:
> We can drop the static_cast here.  (Co-variant return types FTW.)

Ah, good point!

https://codereview.chromium.org/902683002/diff/1/content/browser/frame_host/n...
content/browser/frame_host/navigator_impl.cc:570:
static_cast<SiteInstanceImpl*>(render_frame_host->GetSiteInstance());
On 2015/02/04 20:37:14, Charlie Reis wrote:
> Let's drop the cast here as well.

Done.

[nasko, 2015-02-04 20:42:50]

The CQ bit was checked by nasko@chromium.org

[commit-bot: I haz the power, 2015-02-04 20:43:04]

CQ is trying da patch. Follow status at
 https://chromium-cq-status.appspot.com/patch-status/902683002/20001

[commit-bot: I haz the power, 2015-02-04 21:54:45]

Committed patchset #2 (id:20001)

[commit-bot: I haz the power, 2015-02-04 21:55:39]

Patchset 2 (id:??) landed as
https://crrev.com/55b3eab0bc43d35f7883dde31d06d122d2c20f47
Cr-Commit-Position: refs/heads/master@{#314648}