Remove user gesture tracking

sky/engine/core/dom/Element.cpp

 /*
  * Copyright (C) 1999 Lars Knoll (knoll@kde.org)
  *           (C) 1999 Antti Koivisto (koivisto@kde.org)
  *           (C) 2001 Peter Kelly (pmk@post.com)
  *           (C) 2001 Dirk Mueller (mueller@kde.org)
  *           (C) 2007 David Smith (catfish.man@gmail.com)
  * Copyright (C) 2004, 2005, 2006, 2007, 2008, 2009, 2010, 2012, 2013 Apple Inc. All rights reserved.
  *           (C) 2007 Eric Seidel (eric@webkit.org)
  *
  * This library is free software; you can redistribute it and/or
@@ skipping 65 matching lines @@
 #include "sky/engine/core/html/HTMLTemplateElement.h"
 #include "sky/engine/core/html/parser/HTMLDocumentParser.h"
 #include "sky/engine/core/html/parser/HTMLParserIdioms.h"
 #include "sky/engine/core/page/Chrome.h"
 #include "sky/engine/core/page/ChromeClient.h"
 #include "sky/engine/core/page/FocusController.h"
 #include "sky/engine/core/page/Page.h"
 #include "sky/engine/core/rendering/RenderLayer.h"
 #include "sky/engine/core/rendering/RenderView.h"
 #include "sky/engine/platform/EventDispatchForbiddenScope.h"
-#include "sky/engine/platform/UserGestureIndicator.h"
 #include "sky/engine/platform/scroll/ScrollableArea.h"
 #include "sky/engine/wtf/BitVector.h"
 #include "sky/engine/wtf/HashFunctions.h"
 #include "sky/engine/wtf/text/CString.h"
 #include "sky/engine/wtf/text/StringBuilder.h"
 #include "sky/engine/wtf/text/TextPosition.h"
 
 namespace blink {
 
 PassRefPtr<Element> Element::create(const QualifiedName& tagName, Document* document)
@@ skipping 1108 matching lines @@
     RefPtr<Node> protect(this);
     if (!document().page()->focusController().setFocusedElement(this, document().frame(), type))
         return;
 
     // Setting the focused node above might have invalidated the layout due to scripts.
     document().updateLayout();
     if (!isFocusable())
         return;
     updateFocusAppearance(restorePreviousSelection);
 
-    if (UserGestureIndicator::processedUserGestureSinceLoad()) {
-        // Bring up the keyboard in the context of anything triggered by a user
-        // gesture. Since tracking that across arbitrary boundaries (eg.
-        // animations) is difficult, for now we match IE's heuristic and bring
-        // up the keyboard if there's been any gesture since load.
-        document().page()->chrome().client().showImeIfNeeded();
-    }
+    // Bring up the keyboard in the context of anything triggered by a user
+    // gesture. Since tracking that across arbitrary boundaries (eg.
+    // animations) is difficult, for now we match IE's heuristic and bring
+    // up the keyboard if there's been any gesture since load.

[eseidel, 2015/01/22 23:49:29]

This comment is meaningless, no?

[abarth-chromium, 2015/01/22 23:59:13]

Yes!  Thanks.

+    document().page()->chrome().client().showImeIfNeeded();
 }
 
 void Element::updateFocusAppearance(bool /*restorePreviousSelection*/)
 {
     if (isRootEditableElement()) {
         // Taking the ownership since setSelection() may release the last reference to |frame|.
         RefPtr<LocalFrame> frame(document().frame());
         if (!frame)
             return;
 
@@ skipping 554 matching lines @@
         return false;
     if (treeScope().scopedStyleResolver().hasSelectorForId(idValue))
         return true;
     // Host rules could also have effects.
     if (ShadowRoot* shadowRoot = this->shadowRoot())
         return shadowRoot->scopedStyleResolver().hasSelectorForId(idValue);
     return false;
 }
 
 } // namespace blink