(Re-)introduce AncestorThrottle to handle 'X-Frame-Options'.

content/browser/site_per_process_browsertest.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 "content/browser/site_per_process_browsertest.h"
 
 #include <stddef.h>
 #include <stdint.h>
 
 #include <algorithm>
@@ skipping 6929 matching lines @@
     TestNavigationObserver load_observer(shell()->web_contents());
     EXPECT_TRUE(ExecuteScript(shell(), "frames[0].location.href = '" +
                                            blocked_urls[i].spec() + "';"));
     load_observer.Wait();
 
     // The blocked frame's origin should become unique.
     EXPECT_EQ("null", root->child_at(0)->current_origin().Serialize());
 
     // Ensure that we don't use the blocked URL as the blocked frame's last
     // committed URL (see https://crbug.com/622385).
-    EXPECT_NE(root->child_at(0)->current_frame_host()->GetLastCommittedURL(),
+    EXPECT_NE(root->child_at(0)->current_frame_host()->last_successful_url(),

[alexmos, 2016/12/20 01:34:39]

Is this change still necessary?  Committing a blank page should make this work
with the old check again.  (The full discussion about it was at 
https://codereview.chromium.org/2321503002)

[arthursonzogni, 2016/12/20 10:29:43]

You are right.

               blocked_urls[i]);
 
     // The blocked frame should still fire a load event in its parent's process.
     EXPECT_EQ(expected_title, title_watcher.WaitAndGetTitle());
 
     // Check that the current RenderFrameHost has stopped loading.
     EXPECT_FALSE(root->child_at(0)->current_frame_host()->is_loading());
 
     // The blocked navigation should behave like an empty 200 response. Make
     // sure that the frame's document.title is empty: this double-checks both
@@ skipping 2012 matching lines @@
   rfh->OnSwappedOut();
 
   // Wait for the new a.com navigation to finish.
   back_observer.Wait();
 
   // The RVH for a.com should've been reused, and it should be active.
   EXPECT_TRUE(rvh->is_active());
 }
 
 }  // namespace content