Reenable framebusting deprecation, change it to allow navigation if iframe has ever had a user gestu

third_party/WebKit/Source/core/input/PointerEventManager.cpp

 // Copyright 2016 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 "core/input/PointerEventManager.h"
 
 #include "core/dom/ElementTraversal.h"
 #include "core/dom/shadow/FlatTreeTraversal.h"
 #include "core/events/MouseEvent.h"
 #include "core/frame/FrameView.h"
@@ skipping 269 matching lines @@
     removePointer(pointerEvent);
   }
 }
 
 void PointerEventManager::unblockTouchPointers() {
   m_inCanceledStateForPointerTypeTouch = false;
 }
 
 WebInputEventResult PointerEventManager::handleTouchEvents(
     const PlatformTouchEvent& event) {
+  // TODO(japhet): UserGestureIndicator should setHasReceivedUserGesture()
+  // on the Document.
+  if (UserGestureIndicator::processingUserGesture())

[Rick Byers, 2016/10/06 16:26:47]

I think this is also always the root frame document.  But it doesn't matter
because (I think) there's never a UGI at this point anyway
(http://crbug.com/609363).  Maybe just drop this for now and ask Navid to make
sure he adds the necessary call when he fixes his bug?

+    m_frame->document()->setHasReceivedUserGesture();
   if (event.type() == PlatformEvent::TouchScrollStarted) {
     blockTouchPointers();
     m_touchEventManager->setTouchScrollStarted();
     return WebInputEventResult::HandledSystem;
   }
 
   bool newTouchSequence = true;
   for (const auto& touchPoint : event.touchPoints()) {
     if (touchPoint.state() != PlatformTouchPoint::TouchPressed) {
       newTouchSequence = false;
@@ skipping 374 matching lines @@
       return true;
   }
   return false;
 }
 
 EventTarget* PointerEventManager::getMouseCapturingNode() {
   return getCapturingNode(PointerEventFactory::s_mouseId);
 }
 
 }  // namespace blink