Title: [201410] trunk/Source/WebCore
Revision
201410
Author
[email protected]
Date
2016-05-25 16:35:00 -0700 (Wed, 25 May 2016)

Log Message

Use HashMap::add() instead of HashMap::set() in Node::ensureEventTargetData()
https://bugs.webkit.org/show_bug.cgi?id=158092

Reviewed by Ryosuke Niwa.

Use HashMap::add() instead of HashMap::set() in Node::ensureEventTargetData()
as we already checked that the key is not present in the HashMap.

* dom/Node.cpp:
(WebCore::Node::ensureEventTargetData):

Modified Paths

Diff

Modified: trunk/Source/WebCore/ChangeLog (201409 => 201410)


--- trunk/Source/WebCore/ChangeLog	2016-05-25 23:18:29 UTC (rev 201409)
+++ trunk/Source/WebCore/ChangeLog	2016-05-25 23:35:00 UTC (rev 201410)
@@ -1,3 +1,16 @@
+2016-05-25  Chris Dumez  <[email protected]>
+
+        Use HashMap::add() instead of HashMap::set() in Node::ensureEventTargetData()
+        https://bugs.webkit.org/show_bug.cgi?id=158092
+
+        Reviewed by Ryosuke Niwa.
+
+        Use HashMap::add() instead of HashMap::set() in Node::ensureEventTargetData()
+        as we already checked that the key is not present in the HashMap.
+
+        * dom/Node.cpp:
+        (WebCore::Node::ensureEventTargetData):
+
 2016-05-25  Joseph Pecoraro  <[email protected]>
 
         REGRESSION (r191531): Web Inspector: WebSQL databases are no longer shown when first opening Web Inspector

Modified: trunk/Source/WebCore/dom/Node.cpp (201409 => 201410)


--- trunk/Source/WebCore/dom/Node.cpp	2016-05-25 23:18:29 UTC (rev 201409)
+++ trunk/Source/WebCore/dom/Node.cpp	2016-05-25 23:35:00 UTC (rev 201410)
@@ -2004,7 +2004,7 @@
         return *eventTargetDataMap().get(this);
 
     setHasEventTargetData(true);
-    return *eventTargetDataMap().set(this, std::make_unique<EventTargetData>()).iterator->value;
+    return *eventTargetDataMap().add(this, std::make_unique<EventTargetData>()).iterator->value;
 }
 
 void Node::clearEventTargetData()
_______________________________________________
webkit-changes mailing list
[email protected]
https://lists.webkit.org/mailman/listinfo/webkit-changes

Reply via email to