OOPIF: Data URLs are now rendered in the renderer that initiated the navigation.

content/browser/frame_host/navigation_entry_impl.h

 // Copyright 2013 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.
 
 #ifndef CONTENT_BROWSER_FRAME_HOST_NAVIGATION_ENTRY_IMPL_H_
 #define CONTENT_BROWSER_FRAME_HOST_NAVIGATION_ENTRY_IMPL_H_
 
 #include "base/basictypes.h"
 #include "base/memory/ref_counted.h"
 #include "content/browser/site_instance_impl.h"
@@ skipping 89 matching lines @@
   // counted pointer to a shared site instance.
   //
   // Note that the SiteInstance should usually not be changed after it is set,
   // but this may happen if the NavigationEntry was cloned and needs to use a
   // different SiteInstance.
   void set_site_instance(SiteInstanceImpl* site_instance);
   SiteInstanceImpl* site_instance() const {
     return site_instance_.get();
   }
 
+  // The source_site_instance is used to identify the SiteInstance that

[Charlie Reis, 2014/12/06 00:18:50]

nit: the SiteInstance of the frame that initiated the navigation.

[lfg, 2014/12/08 20:45:32]

Done.

+  // initiated the navigation in case of a cross-site navigation.
+  void set_source_site_instance(SiteInstanceImpl* source_site_instance);
+  SiteInstanceImpl* source_site_instance() const {
+    return source_site_instance_.get();
+  }
+
   // Remember the set of bindings granted to this NavigationEntry at the time
   // of commit, to ensure that we do not grant it additional bindings if we
   // navigate back to it in the future.  This can only be changed once.
   void SetBindings(int bindings);
   int bindings() const {
     return bindings_;
   }
 
   void set_page_type(PageType page_type) {
     page_type_ = page_type;
@@ skipping 100 matching lines @@
   // WARNING WARNING WARNING WARNING WARNING WARNING WARNING WARNING WARNING
   // Session/Tab restore save portions of this class so that it can be recreated
   // later. If you add a new field that needs to be persisted you'll have to
   // update SessionService/TabRestoreService and Android WebView
   // state_serializer.cc appropriately.
   // WARNING WARNING WARNING WARNING WARNING WARNING WARNING WARNING WARNING
 
   // See the accessors above for descriptions.
   int unique_id_;
   scoped_refptr<SiteInstanceImpl> site_instance_;
+  scoped_refptr<SiteInstanceImpl> source_site_instance_;

[Charlie Reis, 2014/12/06 00:18:50]

This is solely used as input for determining site_instance_ before the entry
commits, so we will not persist it.  Please move it further down near the fields
that get cleared, and give it a comment saying that (plus that we'll clear it in
|ResetForCommit|).

[lfg, 2014/12/08 20:45:32]

Done.

   // TODO(creis): Persist bindings_. http://crbug.com/173672.
   int bindings_;
   PageType page_type_;
   GURL url_;
   Referrer referrer_;
   GURL virtual_url_;
   bool update_virtual_url_with_url_;
   base::string16 title_;
   FaviconStatus favicon_;
   PageState page_state_;
@@ skipping 95 matching lines @@
   // persisted, unless specific data is taken out/put back in at save/restore
   // time (see TabNavigation for an example of this).
   std::map<std::string, base::string16> extra_data_;
 
   // Copy and assignment is explicitly allowed for this class.
 };
 
 }  // namespace content
 
 #endif  // CONTENT_BROWSER_FRAME_HOST_NAVIGATION_ENTRY_IMPL_H_