GitLab

Removing method and updating tests that use it so that the mock behaves
more like the production code.

- Removed setter method.
- Added is_for_guest_only constructor param similar to
  RenderProcessHostImpl.
- Updated MockRenderProcessHostFactory to provide the is_for_guest_only
  parameter based on the SiteInstance::IsGuest() method just like
  RenderProcessHostImpl::CreateRenderProcessHost() does.
- Updated NavigationControllerTest.ReloadWithGuest to actually use a
  guest WebContents so that it doesn't need to change the flag on the
  process to achieve a guest reload.


Bug: 1085275
Change-Id: I23c3db1a810ea6d1f0d7ce9e554585e1caf6b297
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2354832
Commit-Queue: Nasko Oskov <nasko@chromium.org>
Auto-Submit: Aaron Colwell <acolwell@chromium.org>
Reviewed-by: Nasko Oskov <nasko@chromium.org>
Cr-Commit-Position: refs/heads/master@{#797898}