REGRESSION (r157328): popover to check into flight ba.com dismisses instantly when...

https://bugs.webkit.org/show_bug.cgi?id=131949

Address the review comment.

* dom/EventDispatcher.cpp:
(WebCore::EventRelatedNodeResolver::findHostOfTreeScopeInTargetTreeScope):

git-svn-id: https://svn.webkit.org/repository/webkit/trunk@167744 268f45cc-cd09-0410-ab3c-d52691b4dbfc

diff --git a/Source/WebCore/ChangeLog b/Source/WebCore/ChangeLog
index 7532137c1488ac56f0f414190399cf8a8d947da1..482a4a6230784af6e63b5377c774acd2f4ebef55 100644 (file)
--- a/Source/WebCore/ChangeLog
+++ b/Source/WebCore/ChangeLog
@@ -1,3 +1,13 @@
+2014-04-23  Ryosuke Niwa  <rniwa@webkit.org>
+
+        REGRESSION (r157328): popover to check into flight ba.com dismisses instantly when focusing form
+        https://bugs.webkit.org/show_bug.cgi?id=131949
+
+        Address the review comment.
+
+        * dom/EventDispatcher.cpp:
+        (WebCore::EventRelatedNodeResolver::findHostOfTreeScopeInTargetTreeScope):
+
 2014-04-23  Andreas Kling  <akling@apple.com>
 
         CachedResourceLoader hoards URLs indefinitely for no good reason.

diff --git a/Source/WebCore/dom/EventDispatcher.cpp b/Source/WebCore/dom/EventDispatcher.cpp
index 4647563dd796d750c808d6859ad893029421d801..5c1d39147973aff665eeea06a1f9d4db35aee0a9 100644 (file)
--- a/Source/WebCore/dom/EventDispatcher.cpp
+++ b/Source/WebCore/dom/EventDispatcher.cpp
@@ -177,7 +177,7 @@ public:
     static Node* findHostOfTreeScopeInTargetTreeScope(const TreeScope& startingTreeScope, const TreeScope& targetScope)
     {
         ASSERT(&targetScope != &startingTreeScope);
-        Node* previousHost = 0;
+        Node* previousHost = nullptr;
         for (const TreeScope* scope = &startingTreeScope; scope; scope = scope->parentTreeScope()) {
             if (scope == &targetScope) {
                 ASSERT(previousHost);
@@ -189,7 +189,7 @@ public:
             else
                 ASSERT_WITH_SECURITY_IMPLICATION(!scope->parentTreeScope());
         }
-        return 0;
+        return nullptr;
     }
 
 private: