UserGestureIndicator is a mess. Clean it up.

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

Index: third_party/WebKit/Source/core/input/TouchEventManager.cpp
diff --git a/third_party/WebKit/Source/core/input/TouchEventManager.cpp b/third_party/WebKit/Source/core/input/TouchEventManager.cpp
index 52ae9710746a4ad865c38584592e5da2331c8e9c..2bc292fedb82d34ea6102bb9b91979b488cb5469 100644
--- a/third_party/WebKit/Source/core/input/TouchEventManager.cpp
+++ b/third_party/WebKit/Source/core/input/TouchEventManager.cpp
@@ -524,21 +524,19 @@ WebInputEventResult TouchEventManager::handleTouchEvent(
 
   std::unique_ptr<UserGestureIndicator> gestureIndicator;
   if (isTap || isSameOrigin) {
-    UserGestureUtilizedCallback* callback = 0;
+    gestureIndicator = wrapUnique(

[Rick Byers, 2016/10/11 19:28:12]

I think you can simplify this further now to eliminate the unique_ptr
indirection, right?  Something like:

if (isTap || isSameOrigin) {
  if (!m_touchSequenceUserGestureToken)
    m_touchSequenceUserGestureToken = UserGestureToken::create();
  // These are ...
  ...
}
// If we have a token, active it for the duration of dispatch
UserGestureIndicator holder(m_touchSequenceUserGestureToken)

+        new UserGestureIndicator(m_touchSequenceUserGestureToken
+                                     ? m_touchSequenceUserGestureToken.release()
+                                     : UserGestureToken::create()));
+
+    m_touchSequenceUserGestureToken = UserGestureIndicator::currentToken();
     // These are cases we'd like to migrate to not hold a user gesture.
     if (event.type() == PlatformEvent::TouchStart ||
         event.type() == PlatformEvent::TouchMove ||
         (event.type() == PlatformEvent::TouchEnd && m_touchScrollStarted)) {
       // Collect metrics in userGestureUtilized().
-      callback = this;
+      m_touchSequenceUserGestureToken->setUserGestureUtilizedCallback(this);
     }
-    if (m_touchSequenceUserGestureToken)
-      gestureIndicator = wrapUnique(new UserGestureIndicator(
-          m_touchSequenceUserGestureToken.release(), callback));
-    else
-      gestureIndicator = wrapUnique(
-          new UserGestureIndicator(DefinitelyProcessingUserGesture, callback));
-    m_touchSequenceUserGestureToken = UserGestureIndicator::currentToken();
   }
 
   return dispatchTouchEvents(event, touchInfos, allTouchesReleased);