Disable hover state change for touch event on page with viewport meta and mobile.

third_party/WebKit/Source/core/dom/Document.cpp

 /*
  * Copyright (C) 1999 Lars Knoll (knoll@kde.org)
  *           (C) 1999 Antti Koivisto (koivisto@kde.org)
  *           (C) 2001 Dirk Mueller (mueller@kde.org)
  *           (C) 2006 Alexey Proskuryakov (ap@webkit.org)
  * Copyright (C) 2004, 2005, 2006, 2007, 2008, 2009, 2011, 2012 Apple Inc. All
  * rights reserved.
  * Copyright (C) 2008, 2009 Torch Mobile Inc. All rights reserved.
  * (http://www.torchmobile.com/)
  * Copyright (C) 2008, 2009, 2011, 2012 Google Inc. All rights reserved.
@@ skipping 126 matching lines @@
 #include "core/frame/EventHandlerRegistry.h"
 #include "core/frame/FrameConsole.h"
 #include "core/frame/History.h"
 #include "core/frame/HostsUsingFeatures.h"
 #include "core/frame/LocalDOMWindow.h"
 #include "core/frame/LocalFrame.h"
 #include "core/frame/LocalFrameClient.h"
 #include "core/frame/LocalFrameView.h"
 #include "core/frame/PerformanceMonitor.h"
 #include "core/frame/Settings.h"
+#include "core/frame/VisualViewport.h"
 #include "core/frame/csp/ContentSecurityPolicy.h"
 #include "core/html/DocumentNameCollection.h"
 #include "core/html/HTMLAllCollection.h"
 #include "core/html/HTMLAnchorElement.h"
 #include "core/html/HTMLBaseElement.h"
 #include "core/html/HTMLBodyElement.h"
 #include "core/html/HTMLCanvasElement.h"
 #include "core/html/HTMLCollection.h"
 #include "core/html/HTMLDialogElement.h"
 #include "core/html/HTMLDocument.h"
@@ skipping 6247 matching lines @@
         !request.TouchMove()) {
       // We are setting the :active chain and freezing it. If future moves
       // happen, they will need to reference this chain.
       for (Element* element = new_active_element; element;
            element = FlatTreeTraversal::ParentElement(*element)) {
         user_action_elements_.SetInActiveChain(element, true);
       }
       SetActiveHoverElement(new_active_element);
     }
   }
+
+  // Do not set hover state if event is from touch and on mobile.
+  if (request.TouchEvent() && GetPage() &&
+      GetPage()->GetVisualViewport().ShouldDisableDesktopWorkarounds())
+    return;
+
   // If the mouse has just been pressed, set :active on the chain. Those (and
   // only those) nodes should remain :active until the mouse is released.
   bool allow_active_changes = !old_active_element && ActiveHoverElement();
 
   // If the mouse is down and if this is a mouse move event, we want to restrict
   // changes in :hover/:active to only apply to elements that are in the :active
   // chain that we froze at the time the mouse went down.
   bool must_be_in_active_chain = request.Active() && request.Move();
 
   Element* old_hover_element = HoverElement();
@@ skipping 50 matching lines @@
   for (Element* element : elements_to_remove_from_chain)
     element->SetHovered(false);
 
   bool saw_common_ancestor = false;
   for (Element* element : elements_to_add_to_chain) {
     // Elements past the common ancestor do not change hover state, but might
     // change active state.
     if (element == ancestor_element)
       saw_common_ancestor = true;
     if (allow_active_changes)
       element->SetActive(true);

[chaopeng, 2017/06/06 17:18:49]

I don't know should we keep setActive here?

[bokan, 2017/06/06 21:07:08]

Active makes sense in the context of touch so we can't just early out above -
skip only the hover related parts.

     if (!saw_common_ancestor || element == hover_element_)
       element->SetHovered(true);
   }
 }
 
 bool Document::HaveScriptBlockingStylesheetsLoaded() const {
   return style_engine_->HaveScriptBlockingStylesheetsLoaded();
 }
 
 bool Document::HaveRenderBlockingStylesheetsLoaded() const {
@@ skipping 347 matching lines @@
 }
 
 void showLiveDocumentInstances() {
   WeakDocumentSet& set = liveDocumentSet();
   fprintf(stderr, "There are %u documents currently alive:\n", set.size());
   for (blink::Document* document : set)
     fprintf(stderr, "- Document %p URL: %s\n", document,
             document->Url().GetString().Utf8().data());
 }
 #endif