Avoid parsing the webstore base url so much.
This is a followup to [1] which modified the extension url API to vend
GURLs rather than std::string for webstore update urls.
This patch does the same for the webstore base url
(the webstore "launch" url). We only parse this url in
ChromeResourceDispatcherHostDelegate::OnResponseStarted so there is less
of a performance win than [1], but still we should avoid doing needless
work.
Parsing this URL takes ~12% of the CPU time of
ResourceLoader::CompleteResponseStarted.
[1] https://codereview.chromium.org/2493053002/
BUG=664174
Review-Url: https://codereview.chromium.org/2506713002
Cr-Commit-Position: refs/heads/master@{#433963}
15 files changed