diff options
| author | jinlong.zhai <jinlong.zhai@samsung.com> | 2015-02-09 07:54:29 -0800 |
|---|---|---|
| committer | Commit bot <commit-bot@chromium.org> | 2015-02-09 15:54:57 +0000 |
| commit | 9f9a53536cd585291f4a63f8a789fa64612dc73a (patch) | |
| tree | de0c1a0a662528ad3da1d45ec234c28584895c01 /content/browser/site_instance_impl.cc | |
| parent | 1418c1251a9d1ca13e2e1afb84d31891fe77e80e (diff) | |
| download | chromium_src-9f9a53536cd585291f4a63f8a789fa64612dc73a.zip chromium_src-9f9a53536cd585291f4a63f8a789fa64612dc73a.tar.gz chromium_src-9f9a53536cd585291f4a63f8a789fa64612dc73a.tar.bz2 | |
[Clean up] Fix the comment for SiteIntance::CreateForURL
In SiteIntance::CreateForURL there will certainly create a new SiteInstance
and BrowsingIntance since in this new BrowsingIntance we will not find any
existing SiteIntance. So BrowsingIntance will never be deleted even beyond
the scope of the method.
The original comments will make some confusion.
BUG=11629
Review URL: https://codereview.chromium.org/880303004
Cr-Commit-Position: refs/heads/master@{#315308}
Diffstat (limited to 'content/browser/site_instance_impl.cc')
| -rw-r--r-- | content/browser/site_instance_impl.cc | 3 |
1 files changed, 1 insertions, 2 deletions
diff --git a/content/browser/site_instance_impl.cc b/content/browser/site_instance_impl.cc index 96af8fc..5930d83 100644 --- a/content/browser/site_instance_impl.cc +++ b/content/browser/site_instance_impl.cc @@ -236,8 +236,7 @@ SiteInstance* SiteInstance::Create(BrowserContext* browser_context) { /*static*/ SiteInstance* SiteInstance::CreateForURL(BrowserContext* browser_context, const GURL& url) { - // This BrowsingInstance may be deleted if it returns an existing - // SiteInstance. + // This will create a new SiteInstance and BrowsingInstance. scoped_refptr<BrowsingInstance> instance( new BrowsingInstance(browser_context)); return instance->GetSiteInstanceForURL(url); |