Title: [277236] trunk/Source/WebKitLegacy/mac
Revision
277236
Author
[email protected]
Date
2021-05-08 18:26:37 -0700 (Sat, 08 May 2021)

Log Message

[Cocoa] Remove the one use of String::toInt from WebKitLegacy
https://bugs.webkit.org/show_bug.cgi?id=225568

Reviewed by Sam Weinig.

* DOM/DOMHTMLInputElement.mm:
(-[DOMHTMLInputElement setSize:]): Use -[NSString intValue] instead of WTF::String::toInt.
Both functions ignore leading spaces, and both ignore trailing junk, so there is no
significant change in behavior.

Modified Paths

Diff

Modified: trunk/Source/WebKitLegacy/mac/ChangeLog (277235 => 277236)


--- trunk/Source/WebKitLegacy/mac/ChangeLog	2021-05-09 01:17:13 UTC (rev 277235)
+++ trunk/Source/WebKitLegacy/mac/ChangeLog	2021-05-09 01:26:37 UTC (rev 277236)
@@ -1,3 +1,15 @@
+2021-05-08  Darin Adler  <[email protected]>
+
+        [Cocoa] Remove the one use of String::toInt from WebKitLegacy
+        https://bugs.webkit.org/show_bug.cgi?id=225568
+
+        Reviewed by Sam Weinig.
+
+        * DOM/DOMHTMLInputElement.mm:
+        (-[DOMHTMLInputElement setSize:]): Use -[NSString intValue] instead of WTF::String::toInt.
+        Both functions ignore leading spaces, and both ignore trailing junk, so there is no
+        significant change in behavior.
+
 2021-05-07  Wenson Hsieh  <[email protected]>
 
         [macOS] Set the -isSourceEditable property when presenting webpage translation popup

Modified: trunk/Source/WebKitLegacy/mac/DOM/DOMHTMLInputElement.mm (277235 => 277236)


--- trunk/Source/WebKitLegacy/mac/DOM/DOMHTMLInputElement.mm	2021-05-09 01:17:13 UTC (rev 277235)
+++ trunk/Source/WebKitLegacy/mac/DOM/DOMHTMLInputElement.mm	2021-05-09 01:26:37 UTC (rev 277236)
@@ -397,7 +397,7 @@
 - (void)setSize:(NSString *)newSize
 {
     WebCore::JSMainThreadNullState state;
-    IMPL->setSize(WTF::String(newSize).toInt());
+    IMPL->setSize(newSize.intValue);
 }
 
 - (NSString *)src
_______________________________________________
webkit-changes mailing list
[email protected]
https://lists.webkit.org/mailman/listinfo/webkit-changes

Reply via email to