Refactor SiteInstanceImpl::DoesSiteRequireDedicatedProcess().

- Splits DoesSiteRequireDedicatedProcess() into 2 pieces.
  DoesSiteRequireDedicatedProcess() accepts normal URLs as before, and
  the new DoesSiteURLRequireDedicatedProcess() accepts a site URL. This
  split will allow code that already has a site URL to determine whether
  the site requires a dedicated process w/o having to do an unneccessary
  GetSiteForURL() call. This also will enable the default site URL
  computation to be moved to GetSiteForURLInternal() in a followup CL.

- Updated all DoesSiteRequireDedicatedProcess() calls that pass
  a site URL to call DoesSiteURLRequireDedicatedProcess() instead. This
  avoids making an extra GetSiteForURL() call that isn't necessary.


Bug: 958060
Change-Id: Idbe42c514954bdde8de8cc1a88534601611424db
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1596222
Reviewed-by: Alex Moshchuk <alexmos@chromium.org>
Commit-Queue: Aaron Colwell <acolwell@chromium.org>
Cr-Commit-Position: refs/heads/master@{#657020}
2 files changed