2009-12-01 Daniel Bates <dbates@webkit.org>
authoreric@webkit.org <eric@webkit.org@268f45cc-cd09-0410-ab3c-d52691b4dbfc>
Tue, 1 Dec 2009 22:12:00 +0000 (22:12 +0000)
committereric@webkit.org <eric@webkit.org@268f45cc-cd09-0410-ab3c-d52691b4dbfc>
Tue, 1 Dec 2009 22:12:00 +0000 (22:12 +0000)
        Reviewed by Pavel Feldman.

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

        Updated layout test to reflect change to localized string and
        rebased the result.

        * inspector/elements-img-tooltip-expected.txt:
        * inspector/elements-img-tooltip.html:
2009-12-01  Daniel Bates  <dbates@webkit.org>

        Reviewed by Pavel Feldman.

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

        Added missing localized strings (that I left out of the patch for bug #21554):
        %d × %d pixels
        %d × %d pixels (Natural: %d × %d pixels)

        Also, changed formatting of these stings to conform with existing ones (added
        a space on both sides of the multiply sign).

        * English.lproj/localizedStrings.js: Added stings.
        * inspector/front-end/ElementsTreeOutline.js:
        (WebInspector.ElementsTreeElement.prototype.createTooltipForImageNode):

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

LayoutTests/ChangeLog
LayoutTests/inspector/elements-img-tooltip-expected.txt
LayoutTests/inspector/elements-img-tooltip.html
WebCore/ChangeLog
WebCore/English.lproj/localizedStrings.js
WebCore/inspector/front-end/ElementsTreeOutline.js

index 279edc921feebb38b47ee31cd8927515e5828034..1104d366aabf1c2494fd01e89b5f9dae7a76986a 100644 (file)
@@ -1,3 +1,15 @@
+2009-12-01  Daniel Bates  <dbates@webkit.org>
+
+        Reviewed by Pavel Feldman.
+
+        https://bugs.webkit.org/show_bug.cgi?id=32001
+
+        Updated layout test to reflect change to localized string and
+        rebased the result.
+
+        * inspector/elements-img-tooltip-expected.txt:
+        * inspector/elements-img-tooltip.html:
+
 2009-12-01  Adam Roben  <aroben@apple.com>
 
         Skip a test that fails with some versions of CFNetwork on Windows
index 1c80d6f6ac9bbcf121b1589786f190612eaf769d..9e480ee71d93de8b779ea9bf7a053b29c7b69443 100644 (file)
@@ -1,5 +1,5 @@
-Tests that the tooltip text for the image (below) is "215×174 pixels".
+Tests that the tooltip text for the image (below) is "215 × 174 pixels".
 
 
-PASSED, tooltip text is "215×174 pixels".
+PASSED, tooltip text is "215 × 174 pixels".
 
index 8140e317dbe24baf82398c059e6b9556f3dd773e..da8302cf9117e91426511202fce12271ef446cc5 100644 (file)
@@ -4,7 +4,7 @@
 <script src="inspector-test.js"></script>
 <script src="elements-tests.js"></script>
 <script>
-var EXPECTED_TOOLTIP_TEXT = "215\xd7174 pixels";
+var EXPECTED_TOOLTIP_TEXT = "215 \xd7 174 pixels";
 function doit()
 {
     function callback(tooltipText)
index affad175f33a80a75598cd0963adf4bfa55e96c8..76822ac0f05e2b317f7a9965ca781f47d443acc5 100644 (file)
@@ -1,3 +1,20 @@
+2009-12-01  Daniel Bates  <dbates@webkit.org>
+
+        Reviewed by Pavel Feldman.
+
+        https://bugs.webkit.org/show_bug.cgi?id=32001
+
+        Added missing localized strings (that I left out of the patch for bug #21554):
+        %d × %d pixels
+        %d × %d pixels (Natural: %d × %d pixels)
+
+        Also, changed formatting of these stings to conform with existing ones (added
+        a space on both sides of the multiply sign).
+
+        * English.lproj/localizedStrings.js: Added stings.
+        * inspector/front-end/ElementsTreeOutline.js:
+        (WebInspector.ElementsTreeElement.prototype.createTooltipForImageNode):
+
 2009-12-01  Dave Hyatt  <hyatt@apple.com>
 
         Reviewed by Simon Fraser.
index ef981c78d851631cc0bc895f52f6bc4bea4980a4..e0b974c6b5675e569a22f7c2b3bcf57d3db6ffa2 100644 (file)
Binary files a/WebCore/English.lproj/localizedStrings.js and b/WebCore/English.lproj/localizedStrings.js differ
index 68807704a9487c68eddc4a00ef35de5ed3273a4f..d6326f0c2dfefcc65963843ac56e234f8cd3592e 100644 (file)
@@ -336,9 +336,9 @@ WebInspector.ElementsTreeElement.prototype = {
 
             var tooltipText = null;
             if (properties.offsetHeight === properties.naturalHeight && properties.offsetWidth === properties.naturalWidth)
-                tooltipText = WebInspector.UIString("%d\u00d7%d pixels", properties.offsetWidth, properties.offsetHeight);
+                tooltipText = WebInspector.UIString("%d × %d pixels", properties.offsetWidth, properties.offsetHeight);
             else
-                tooltipText = WebInspector.UIString("%d\u00d7%d pixels (Natural: %d\u00d7%d pixels)", properties.offsetWidth, properties.offsetHeight, properties.naturalWidth, properties.naturalHeight);
+                tooltipText = WebInspector.UIString("%d × %d pixels (Natural: %d × %d pixels)", properties.offsetWidth, properties.offsetHeight, properties.naturalWidth, properties.naturalHeight);
             callback(tooltipText);
         }
         var objectProxy = new WebInspector.ObjectProxy(node.id);