Don't swap processes for javascript: URLs.

content/browser/site_instance.cc

Index: content/browser/site_instance.cc
diff --git a/content/browser/site_instance.cc b/content/browser/site_instance.cc
index 2c5a1d76622c93def01d846d1193a0be108a680e..711978d2643815e15ab3110bc8bae48b50ecdca9 100644
--- a/content/browser/site_instance.cc
+++ b/content/browser/site_instance.cc
@@ -129,6 +129,11 @@ bool SiteInstance::HasWrongProcessForURL(const GURL& url) const {
   if (!HasProcess())
     return false;
 
+  // If the URL to navigate to can be associated with any site instance,
+  // we want to run in the same procses.

[Charlie Reis, 2012/01/12 22:01:14]

Nit: Let's say "we want to keep it in the same process" just to be clear.

Also, s/procses/process/

[nasko, 2012/01/12 22:47:47]

Done.

+  if (IsURLSameAsAnySiteInstance(url))
+    return false;
+
   // If the site URL is an extension (e.g., for hosted apps or WebUI) but the
   // process is not (or vice versa), make sure we notice and fix it.
   GURL site_url = GetSiteForURL(browsing_instance_->browser_context(), url);