Canvas2d: fix event.region being null on mouseleave/out events

third_party/WebKit/Source/modules/canvas2d/CanvasRenderingContext2D.cpp

 /*
  * Copyright (C) 2004, 2005, 2006, 2007, 2008, 2009, 2010, 2011, 2012 Apple Inc. All rights reserved.
  * Copyright (C) 2008, 2010 Nokia Corporation and/or its subsidiary(-ies)
  * Copyright (C) 2007 Alp Toker <alp@atoker.com>
  * Copyright (C) 2008 Eric Seidel <eric@webkit.org>
  * Copyright (C) 2008 Dirk Schulze <krit@webkit.org>
  * Copyright (C) 2010 Torch Mobile (Beijing) Co. Ltd. All rights reserved.
  * Copyright (C) 2012, 2013 Intel Corporation. All rights reserved.
  * Copyright (C) 2013 Adobe Systems Incorporated. All rights reserved.
  *
@@ skipping 587 matching lines @@
     HitRegion* hitRegion = hitRegionAtPoint(localPos);
     if (hitRegion) {
         Element* control = hitRegion->control();
         if (control && canvas()->isSupportedInteractiveCanvasFallback(*control))
             return std::make_pair(hitRegion->control(), hitRegion->id());
         return std::make_pair(nullptr, hitRegion->id());
     }
     return std::make_pair(nullptr, String());
 }
 
+String CanvasRenderingContext2D::getIdFromControl(const Element* element)
+{
+    if (hitRegionsCount() <= 0)
+        return String();
+
+    if (HitRegion* hitRegion = m_hitRegionManager->getHitRegionByControl(const_cast<Element*>(element)))

[dtapuska, 2016/04/08 15:18:31]

ugh this const_cast is ugly. Why does the hit region manager need a non-const
Element* does it mutate the object?

+        return hitRegion->id();
+    return String();
+}
+
 String CanvasRenderingContext2D::textAlign() const
 {
     return textAlignName(state().getTextAlign());
 }
 
 void CanvasRenderingContext2D::setTextAlign(const String& s)
 {
     TextAlign align;
     if (!parseTextAlign(s, align))
         return;
@@ skipping 424 matching lines @@
 
 unsigned CanvasRenderingContext2D::hitRegionsCount() const
 {
     if (m_hitRegionManager)
         return m_hitRegionManager->getHitRegionsCount();
 
     return 0;
 }
 
 } // namespace blink