Reimplement inline signin with iframe

chrome/browser/chrome_content_browser_client.cc

 // Copyright (c) 2012 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 "chrome/browser/chrome_content_browser_client.h"
 
 #include <set>
 #include <utility>
 #include <vector>
 
@@ skipping 781 matching lines @@
 
     if (is_isolated) {
       CHECK(site.has_host());
       // For extensions with isolated storage, the the host of the |site| is
       // the |partition_domain|. The |in_memory| and |partition_name| are only
       // used in guest schemes so they are cleared here.
       *partition_domain = site.host();
       *in_memory = false;
       partition_name->clear();
     }
+  } else if (site.SchemeIs(chrome::kChromeUIScheme) &&
+             site.host() == chrome::kChromeUIChromeSigninHost) {

[jam, 2014/01/26 02:51:46]

nit: site.GetOrigin().spec() == kChromeUIChromeSigninURL would be shorter

[guohui, 2014/01/29 12:50:51]

Done.

+    // Chrome signin page has an embedded iframe of extension and web content,
+    // thus it must be isolated from other webUI pages.
+    *partition_domain = chrome::kChromeUIChromeSigninHost;
   }
 
   // Assert that if |can_be_default| is false, the code above must have found a
   // non-default partition.  If this fails, the caller has a serious logic
   // error about which StoragePartition they expect to be in and it is not
   // safe to continue.
   CHECK(can_be_default || !partition_domain->empty());
 }
 
 content::WebContentsViewDelegate*
@@ skipping 1849 matching lines @@
   // releases of Chrome. Permitting "Unknown" allows these APIs to be used on
   // Chromium builds as well.
   return channel <= chrome::VersionInfo::CHANNEL_DEV;
 #else
   return false;
 #endif
 }
 
 
 }  // namespace chrome