DefaultLocalizationStrategy::htmlSelectMultipleItems() should use size_t instead...
authorbenjamin@webkit.org <benjamin@webkit.org@268f45cc-cd09-0410-ab3c-d52691b4dbfc>
Fri, 10 Feb 2012 22:17:30 +0000 (22:17 +0000)
committerbenjamin@webkit.org <benjamin@webkit.org@268f45cc-cd09-0410-ab3c-d52691b4dbfc>
Fri, 10 Feb 2012 22:17:30 +0000 (22:17 +0000)
https://bugs.webkit.org/show_bug.cgi?id=78374

Patch by Benjamin Poulain <bpoulain@apple.com> on 2012-02-10
Reviewed by Joseph Pecoraro.

The value comes from size_t and is converted to int for no good reason. We should use
size_t.

* platform/DefaultLocalizationStrategy.cpp:
(WebCore::DefaultLocalizationStrategy::htmlSelectMultipleItems):
* platform/DefaultLocalizationStrategy.h:
(DefaultLocalizationStrategy):

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

Source/WebCore/ChangeLog
Source/WebCore/platform/DefaultLocalizationStrategy.cpp
Source/WebCore/platform/DefaultLocalizationStrategy.h

index 34524ab..c52ee04 100644 (file)
@@ -1,3 +1,18 @@
+2012-02-10  Benjamin Poulain  <bpoulain@apple.com>
+
+        DefaultLocalizationStrategy::htmlSelectMultipleItems() should use size_t instead of int
+        https://bugs.webkit.org/show_bug.cgi?id=78374
+
+        Reviewed by Joseph Pecoraro.
+
+        The value comes from size_t and is converted to int for no good reason. We should use
+        size_t.
+
+        * platform/DefaultLocalizationStrategy.cpp:
+        (WebCore::DefaultLocalizationStrategy::htmlSelectMultipleItems):
+        * platform/DefaultLocalizationStrategy.h:
+        (DefaultLocalizationStrategy):
+
 2012-02-10  Anders Carlsson  <andersca@apple.com>
 
         More ScrollableArea cleanup
index a0d2e34..aa3a486 100644 (file)
@@ -768,7 +768,7 @@ String DefaultLocalizationStrategy::keygenKeychainItemName(const String& host)
 }
 
 #if PLATFORM(IOS)
-String DefaultLocalizationStrategy::htmlSelectMultipleItems(int count)
+String DefaultLocalizationStrategy::htmlSelectMultipleItems(size_t count)
 {
     switch (count) {
     case 0:
@@ -776,7 +776,7 @@ String DefaultLocalizationStrategy::htmlSelectMultipleItems(int count)
     case 1:
         return WEB_UI_STRING("1 Item", "Present the element <select multiple> when a single <option> is selected (iOS only)");
     default:
-        return formatLocalizedString(WEB_UI_STRING("%d Items", "Present the number of selected <option> items in a <select multiple> element (iOS only)"), count);
+        return formatLocalizedString(WEB_UI_STRING("%zu Items", "Present the number of selected <option> items in a <select multiple> element (iOS only)"), count);
     }
 }
 #endif // PLATFORM(IOS)
index 4fa4d9c..a22e9df 100644 (file)
@@ -170,7 +170,7 @@ public:
     virtual String keygenMenuItem2048();
     virtual String keygenKeychainItemName(const String& host);
 #if PLATFORM(IOS)
-    virtual String htmlSelectMultipleItems(int);
+    virtual String htmlSelectMultipleItems(size_t);
 #endif // PLATFORM(IOS)
 #endif // PLATFORM(MAC)
     virtual String imageTitle(const String& filename, const IntSize&);