Store the last committed origin in RenderFrameHostImpl.

content/public/browser/render_frame_host.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_PUBLIC_BROWSER_RENDER_FRAME_HOST_H_
 #define CONTENT_PUBLIC_BROWSER_RENDER_FRAME_HOST_H_
 
 #include <string>
 
 #include "base/callback_forward.h"
@@ skipping 93 matching lines @@
   // return an empty string.
   virtual const std::string& GetFrameName() = 0;
 
   // Returns true if the frame is out of process.
   virtual bool IsCrossProcessSubframe() = 0;
 
   // Returns the last committed URL of the frame.
   virtual const GURL& GetLastCommittedURL() = 0;
 
   // Returns the last committed origin of the frame.
-  //
-  // The origin is only available if this RenderFrameHost is current in the
-  // frame tree -- i.e., it would be visited by WebContents::ForEachFrame. In
-  // particular, this method may CHECK if called from
-  // WebContentsObserver::RenderFrameCreated, since non-current frames can be
-  // passed to that observer method.
-  virtual url::Origin GetLastCommittedOrigin() = 0;
+  virtual const url::Origin& GetLastCommittedOrigin() = 0;

[alexmos, 2016/12/06 17:22:40]

We should be able to remove the hack introduced in
https://codereview.chromium.org/2545523008 to work around pending delete RFHs,
but I left that for a separate CL, or maybe even just reverting that CL.

[Charlie Reis, 2016/12/08 19:51:47]

Acknowledged.

 
   // Returns the associated widget's native view.
   virtual gfx::NativeView GetNativeView() = 0;
 
   // Adds |message| to the DevTools console.
   virtual void AddMessageToConsole(ConsoleMessageLevel level,
                                    const std::string& message) = 0;
 
   // Runs some JavaScript in this frame's context. If a callback is provided, it
   // will be used to return the result, when the result is available.
@@ skipping 99 matching lines @@
 
  private:
   // This interface should only be implemented inside content.
   friend class RenderFrameHostImpl;
   RenderFrameHost() {}
 };
 
 }  // namespace content
 
 #endif  // CONTENT_PUBLIC_BROWSER_RENDER_FRAME_HOST_H_