Revise about: and chrome: url handling.

content/browser/site_instance.cc

 // Copyright (c) 2011 The Chromium Authors. All rights reserved.
 // Use of this source code is governed by a BSD-style license that can be
 // found in the LICENSE file.
 
 #include "content/browser/site_instance.h"
 
 #include "content/browser/browsing_instance.h"
 #include "content/browser/content_browser_client.h"
 #include "content/browser/renderer_host/browser_render_process_host.h"
 #include "content/browser/webui/web_ui_factory.h"
 #include "content/common/content_client.h"
 #include "content/common/notification_service.h"
 #include "content/common/url_constants.h"
 #include "net/base/registry_controlled_domain.h"
 
 static bool IsURLSameAsAnySiteInstance(const GURL& url) {
   if (!url.is_valid())
     return false;
 
-  // We treat javascript: and about:crash as the same site as any URL since they
-  // are actually modifiers on existing pages.
-  if (url.SchemeIs(chrome::kJavaScriptScheme) ||
-      url.spec() == chrome::kAboutCrashURL) {
+  // We treat javascript: as the same site as any URL since it is actually
+  // a modifier on existing pages.
+  if (url.SchemeIs(chrome::kJavaScriptScheme))
     return true;
-  }
 
   return
       content::GetContentClient()->browser()->IsURLSameAsAnySiteInstance(url);
 }
 
 int32 SiteInstance::next_site_instance_id_ = 1;
 
 SiteInstance::SiteInstance(BrowsingInstance* browsing_instance)
     : id_(next_site_instance_id_++),
       browsing_instance_(browsing_instance),
@@ skipping 199 matching lines @@
 }
 
 void SiteInstance::Observe(NotificationType type,
                            const NotificationSource& source,
                            const NotificationDetails& details) {
   DCHECK(type == NotificationType::RENDERER_PROCESS_TERMINATED);
   RenderProcessHost* rph = Source<RenderProcessHost>(source).ptr();
   if (rph == process_)
     process_ = NULL;
 }