Title: [151696] trunk/Source/WebCore
Revision
151696
Author
[email protected]
Date
2013-06-18 14:25:53 -0700 (Tue, 18 Jun 2013)

Log Message

Simplify FormKeyGenerator::willDeleteForm
https://bugs.webkit.org/show_bug.cgi?id=117751

Reviewed by Antti Koivisto.

Merge https://chromium.googlesource.com/chromium/blink/+/c6399efb9da863eb1d1fc98b17ef925998b985ac

* html/FormController.cpp:
(WebCore::FormKeyGenerator::willDeleteForm):

Modified Paths

Diff

Modified: trunk/Source/WebCore/ChangeLog (151695 => 151696)


--- trunk/Source/WebCore/ChangeLog	2013-06-18 21:25:07 UTC (rev 151695)
+++ trunk/Source/WebCore/ChangeLog	2013-06-18 21:25:53 UTC (rev 151696)
@@ -1,5 +1,17 @@
 2013-06-18  Ryosuke Niwa  <[email protected]>
 
+        Simplify FormKeyGenerator::willDeleteForm
+        https://bugs.webkit.org/show_bug.cgi?id=117751
+
+        Reviewed by Antti Koivisto.
+
+        Merge https://chromium.googlesource.com/chromium/blink/+/c6399efb9da863eb1d1fc98b17ef925998b985ac
+
+        * html/FormController.cpp:
+        (WebCore::FormKeyGenerator::willDeleteForm):
+
+2013-06-18  Ryosuke Niwa  <[email protected]>
+
         REGRESSION(r147602): Search text field doesn't render selection when it has some :focus rules
         https://bugs.webkit.org/show_bug.cgi?id=117747
 

Modified: trunk/Source/WebCore/html/FormController.cpp (151695 => 151696)


--- trunk/Source/WebCore/html/FormController.cpp	2013-06-18 21:25:07 UTC (rev 151695)
+++ trunk/Source/WebCore/html/FormController.cpp	2013-06-18 21:25:53 UTC (rev 151696)
@@ -368,12 +368,7 @@
 void FormKeyGenerator::willDeleteForm(HTMLFormElement* form)
 {
     ASSERT(form);
-    if (m_formToKeyMap.isEmpty())
-        return;
-    FormToKeyMap::iterator it = m_formToKeyMap.find(form);
-    if (it == m_formToKeyMap.end())
-        return;
-    m_formToKeyMap.remove(it);
+    m_formToKeyMap.remove(form);
 }
 
 // ----------------------------------------------------------------------------
_______________________________________________
webkit-changes mailing list
[email protected]
https://lists.webkit.org/mailman/listinfo/webkit-changes

Reply via email to