WebCore:
authoraroben <aroben@268f45cc-cd09-0410-ab3c-d52691b4dbfc>
Fri, 15 Sep 2006 22:10:25 +0000 (22:10 +0000)
committeraroben <aroben@268f45cc-cd09-0410-ab3c-d52691b4dbfc>
Fri, 15 Sep 2006 22:10:25 +0000 (22:10 +0000)
        Reviewed by timothy.

        Fixes http://bugzilla.opendarwin.org/show_bug.cgi?id=10865
        New file upload control should match the width of the old one

        Make width of new file upload control match the old one as closely as
        possible.

        * rendering/RenderFileUploadControl.cpp:

LayoutTests:
        Reviewed by timothy.

        Part of http://bugzilla.opendarwin.org/show_bug.cgi?id=10865
        New file upload control should match the width of the old one

        Update tests to match new file upload control.

        * fast/forms/file-input-disabled-expected.txt:
        * fast/forms/form-element-geometry-expected.txt:
        * fast/forms/input-appearance-height-expected.txt:
        * fast/forms/input-value-expected.txt:

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

LayoutTests/ChangeLog
LayoutTests/fast/forms/file-input-disabled-expected.txt
LayoutTests/fast/forms/form-element-geometry-expected.txt
LayoutTests/fast/forms/input-appearance-height-expected.txt
LayoutTests/fast/forms/input-value-expected.txt
WebCore/ChangeLog
WebCore/rendering/RenderFileUploadControl.cpp

index e781e565feea44eff7914f8a99e083334c1039f2..3a5b530e3a7b65f0976d80636633b3a69002d897 100644 (file)
@@ -1,3 +1,17 @@
+2006-09-15  Adam Roben  <aroben@apple.com>
+
+        Reviewed by timothy.
+
+        Part of http://bugzilla.opendarwin.org/show_bug.cgi?id=10865
+        New file upload control should match the width of the old one
+
+        Update tests to match new file upload control.
+
+        * fast/forms/file-input-disabled-expected.txt:
+        * fast/forms/form-element-geometry-expected.txt:
+        * fast/forms/input-appearance-height-expected.txt:
+        * fast/forms/input-value-expected.txt:
+
 2006-09-15  Kevin McCullough  <KMcCullough@apple.com>
 
         - just layout test result changes 
index 16f7c85b51344aa7822c87a39267df594c9992b7..913fe368b776c14ba40fd0d4dc4914f7b0db8dd2 100644 (file)
@@ -13,7 +13,15 @@ layer at (0,0) size 800x600
           RenderBR {BR} at (0,19) size 0x18
           RenderText {#text} at (0,38) size 86x18
             text run at (0,38) width 86: "  Select File:  "
-          RenderFileButton {INPUT} at (88,39) size 234x18
-          RenderBR {BR} at (324,52) size 0x0
+          RenderFileUploadControl {INPUT} at (88,39) size 236x18
+            RenderButton {INPUT} at (0,0) size 78x18 [bgcolor=#C0C0C0]
+              RenderBlock (anonymous) at (8,2) size 62x13
+                RenderText at (0,0) size 62x13
+                  text run at (0,0) width 62: "Choose File"
+            RenderFileUploadInnerFileBox at (78,2) size 158x13
+              RenderBlock (anonymous) at (4,0) size 0x13
+                RenderText at (0,0) size 0x13
+                  text run at (0,0) width 0: "no file selected"
+          RenderBR {BR} at (326,52) size 0x0
         RenderTable {TABLE} at (0,59) size 4x2
           RenderTableSection {TBODY} at (0,0) size 4x2
index f47421fb51f141da5873afc81da488575310e599..7b9c549d624e8c8a89a9902df8dcfb9afb7b7eb0 100644 (file)
@@ -89,19 +89,27 @@ layer at (0,0) size 785x627
               RenderBlock {DIV} at (1,1) size 16x16 [border: (2px solid #0000FF)]
                 RenderInline {FONT} at (0,0) size 12x12
                   RenderBlock {INPUT} at (2,2) size 12x12
-      RenderTable {TABLE} at (0,226) size 547x68
-        RenderTableSection {TBODY} at (0,0) size 547x68
-          RenderTableRow {TR} at (0,2) size 547x64
+      RenderTable {TABLE} at (0,226) size 549x68
+        RenderTableSection {TBODY} at (0,0) size 549x68
+          RenderTableRow {TR} at (0,2) size 549x64
             RenderTableCell {TD} at (2,2) size 84x25 [r=0 c=0 rs=1 cs=1]
               RenderBlock {DIV} at (1,1) size 82x23 [border: (2px solid #0000FF)]
                 RenderTextField {INPUT} at (2,2) size 78x19 [bgcolor=#FFFFFF] [border: (2px inset #000000)]
             RenderTableCell {TD} at (88,2) size 40x64 [r=0 c=1 rs=1 cs=1]
               RenderBlock {DIV} at (1,1) size 38x62 [border: (2px solid #0000FF)]
                 DeprecatedRenderSelect {SELECT} at (2,2) size 34x58
-            RenderTableCell {TD} at (130,2) size 240x24 [r=0 c=2 rs=1 cs=1]
-              RenderBlock {DIV} at (1,1) size 238x22 [border: (2px solid #0000FF)]
-                RenderFileButton {INPUT} at (2,2) size 234x18
-            RenderTableCell {TD} at (372,2) size 173x38 [r=0 c=3 rs=1 cs=1]
+            RenderTableCell {TD} at (130,2) size 242x24 [r=0 c=2 rs=1 cs=1]
+              RenderBlock {DIV} at (1,1) size 240x22 [border: (2px solid #0000FF)]
+                RenderFileUploadControl {INPUT} at (2,2) size 236x18
+                  RenderButton {INPUT} at (0,0) size 78x18 [bgcolor=#C0C0C0]
+                    RenderBlock (anonymous) at (8,2) size 62x13
+                      RenderText at (0,0) size 62x13
+                        text run at (0,0) width 62: "Choose File"
+                  RenderFileUploadInnerFileBox at (78,2) size 158x13
+                    RenderBlock (anonymous) at (4,0) size 0x13
+                      RenderText at (0,0) size 0x13
+                        text run at (0,0) width 0: "no file selected"
+            RenderTableCell {TD} at (374,2) size 173x38 [r=0 c=3 rs=1 cs=1]
               RenderBlock {DIV} at (1,1) size 171x36 [border: (2px solid #0000FF)]
                 RenderTextField {TEXTAREA} at (4,4) size 163x28 [bgcolor=#FFFFFF] [border: (1px solid #000000)]
       RenderBlock {H2} at (0,313) size 769x28
@@ -175,10 +183,18 @@ layer at (0,0) size 785x627
         RenderTextField {INPUT} at (29,18) size 78x19 [bgcolor=#FFFFFF] [border: (2px inset #000000)]
         RenderText {#text} at (109,18) size 4x18
           text run at (109,18) width 4: " "
-        RenderFileButton {INPUT} at (115,19) size 234x18
-        RenderText {#text} at (351,18) size 4x18
-          text run at (351,18) width 4: " "
-        RenderTextField {TEXTAREA} at (357,2) size 163x28 [bgcolor=#FFFFFF] [border: (1px solid #000000)]
+        RenderFileUploadControl {INPUT} at (115,19) size 236x18
+          RenderButton {INPUT} at (0,0) size 78x18 [bgcolor=#C0C0C0]
+            RenderBlock (anonymous) at (8,2) size 62x13
+              RenderText at (0,0) size 62x13
+                text run at (0,0) width 62: "Choose File"
+          RenderFileUploadInnerFileBox at (78,2) size 158x13
+            RenderBlock (anonymous) at (4,0) size 0x13
+              RenderText at (0,0) size 0x13
+                text run at (0,0) width 0: "no file selected"
+        RenderText {#text} at (353,18) size 4x18
+          text run at (353,18) width 4: " "
+        RenderTextField {TEXTAREA} at (359,2) size 163x28 [bgcolor=#FFFFFF] [border: (1px solid #000000)]
         RenderText {#text} at (0,0) size 0x0
       RenderBlock {H2} at (0,491) size 769x28
         RenderText {#text} at (0,0) size 197x28
@@ -236,7 +252,7 @@ layer at (16,242) size 72x13
   RenderBlock {DIV} at (3,3) size 72x13
     RenderText {#text} at (1,0) size 49x13
       text run at (1,0) width 49: "text field"
-layer at (386,242) size 161x26
+layer at (388,242) size 161x26
   RenderBlock {DIV} at (1,1) size 161x26
     RenderText {#text} at (3,0) size 44x13
       text run at (3,0) width 44: "textarea"
@@ -244,7 +260,7 @@ layer at (40,462) size 72x13
   RenderBlock {DIV} at (3,3) size 72x13
     RenderText {#text} at (1,0) size 49x13
       text run at (1,0) width 49: "text field"
-layer at (366,444) size 161x26
+layer at (368,444) size 161x26
   RenderBlock {DIV} at (1,1) size 161x26
     RenderText {#text} at (3,0) size 44x13
       text run at (3,0) width 44: "textarea"
index 76cd250c969996a9e721c855b612c09549ed8483..05e9757535ddc1278fab4fdd01f3c433b0253d09 100644 (file)
@@ -27,10 +27,18 @@ layer at (0,0) size 800x600
         RenderBR {BR} at (87,60) size 0x0
         RenderText {#text} at (0,66) size 24x18
           text run at (0,66) width 24: "file "
-        RenderFileButton {INPUT} at (26,67) size 234x18
-        RenderText {#text} at (262,66) size 4x18
-          text run at (262,66) width 4: " "
-        RenderBR {BR} at (266,80) size 0x0
+        RenderFileUploadControl {INPUT} at (26,67) size 236x18
+          RenderButton {INPUT} at (0,0) size 78x18 [bgcolor=#C0C0C0]
+            RenderBlock (anonymous) at (8,2) size 62x13
+              RenderText at (0,0) size 62x13
+                text run at (0,0) width 62: "Choose File"
+          RenderFileUploadInnerFileBox at (78,2) size 158x13
+            RenderBlock (anonymous) at (4,0) size 0x13
+              RenderText at (0,0) size 0x13
+                text run at (0,0) width 0: "no file selected"
+        RenderText {#text} at (264,66) size 4x18
+          text run at (264,66) width 4: " "
+        RenderBR {BR} at (268,80) size 0x0
         RenderText {#text} at (0,87) size 42x18
           text run at (0,87) width 42: "image "
         RenderImage {INPUT} at (42,100) size 10x1
index 44815860bd4b413b2b8527aa7c363e2b3a65f4fb..e6bef1ae6d08dd9536871efd7242ec451f89d1e6 100644 (file)
@@ -12,188 +12,196 @@ layer at (0,0) size 800x600
           text run at (0,0) width 713: "Results that match Gecko are like WinIE, but with \"before\" for the attribute in the first two rows and the last row."
       RenderBlock {HR} at (0,86) size 784x2 [border: (1px inset #000000)]
       RenderBlock {FORM} at (0,96) size 784x368
-        RenderTable {TABLE} at (0,0) size 758x368
-          RenderTableSection {THEAD} at (0,0) size 758x24
-            RenderTableRow {TR} at (0,2) size 758x20
+        RenderTable {TABLE} at (0,0) size 760x368
+          RenderTableSection {THEAD} at (0,0) size 760x24
+            RenderTableRow {TR} at (0,2) size 760x20
               RenderTableCell {TH} at (2,2) size 385x20 [r=0 c=0 rs=1 cs=1]
                 RenderText {#text} at (1,1) size 55x18
                   text run at (1,1) width 55: "test case"
-              RenderTableCell {TH} at (389,2) size 240x20 [r=0 c=1 rs=1 cs=1]
+              RenderTableCell {TH} at (389,2) size 242x20 [r=0 c=1 rs=1 cs=1]
                 RenderText {#text} at (1,1) size 89x18
                   text run at (1,1) width 89: "form element"
-              RenderTableCell {TH} at (631,2) size 62x20 [r=0 c=2 rs=1 cs=1]
+              RenderTableCell {TH} at (633,2) size 62x20 [r=0 c=2 rs=1 cs=1]
                 RenderText {#text} at (1,1) size 60x18
                   text run at (1,1) width 60: "property"
-              RenderTableCell {TH} at (695,2) size 61x20 [r=0 c=3 rs=1 cs=1]
+              RenderTableCell {TH} at (697,2) size 61x20 [r=0 c=3 rs=1 cs=1]
                 RenderText {#text} at (1,1) size 59x18
                   text run at (1,1) width 59: "attribute"
-          RenderTableSection {TBODY} at (0,24) size 758x344
-            RenderTableRow {TR} at (0,2) size 758x25
+          RenderTableSection {TBODY} at (0,24) size 760x344
+            RenderTableRow {TR} at (0,2) size 760x25
               RenderTableCell {TD} at (2,4) size 385x20 [r=0 c=0 rs=1 cs=1]
                 RenderText {#text} at (1,1) size 207x18
                   text run at (1,1) width 207: "text with value property changed"
-              RenderTableCell {TD} at (389,2) size 240x25 [r=0 c=1 rs=1 cs=1]
+              RenderTableCell {TD} at (389,2) size 242x25 [r=0 c=1 rs=1 cs=1]
                 RenderTextField {INPUT} at (3,3) size 148x19 [bgcolor=#FFFFFF] [border: (2px inset #000000)]
-              RenderTableCell {TD} at (631,4) size 62x20 [r=0 c=2 rs=1 cs=1]
+              RenderTableCell {TD} at (633,4) size 62x20 [r=0 c=2 rs=1 cs=1]
                 RenderText {#text} at (1,1) size 28x18
                   text run at (1,1) width 28: "after"
-              RenderTableCell {TD} at (695,4) size 61x20 [r=0 c=3 rs=1 cs=1]
+              RenderTableCell {TD} at (697,4) size 61x20 [r=0 c=3 rs=1 cs=1]
                 RenderText {#text} at (1,1) size 40x18
                   text run at (1,1) width 40: "before"
-            RenderTableRow {TR} at (0,29) size 758x25
+            RenderTableRow {TR} at (0,29) size 760x25
               RenderTableCell {TD} at (2,31) size 385x20 [r=1 c=0 rs=1 cs=1]
                 RenderText {#text} at (1,1) size 244x18
                   text run at (1,1) width 244: "password with value property changed"
-              RenderTableCell {TD} at (389,29) size 240x25 [r=1 c=1 rs=1 cs=1]
+              RenderTableCell {TD} at (389,29) size 242x25 [r=1 c=1 rs=1 cs=1]
                 RenderTextField {INPUT} at (3,3) size 148x19 [bgcolor=#FFFFFF] [border: (2px inset #000000)]
-              RenderTableCell {TD} at (631,31) size 62x20 [r=1 c=2 rs=1 cs=1]
+              RenderTableCell {TD} at (633,31) size 62x20 [r=1 c=2 rs=1 cs=1]
                 RenderText {#text} at (1,1) size 28x18
                   text run at (1,1) width 28: "after"
-              RenderTableCell {TD} at (695,31) size 61x20 [r=1 c=3 rs=1 cs=1]
+              RenderTableCell {TD} at (697,31) size 61x20 [r=1 c=3 rs=1 cs=1]
                 RenderText {#text} at (1,1) size 40x18
                   text run at (1,1) width 40: "before"
-            RenderTableRow {TR} at (0,56) size 758x20
+            RenderTableRow {TR} at (0,56) size 760x20
               RenderTableCell {TD} at (2,56) size 385x20 [r=2 c=0 rs=1 cs=1]
                 RenderText {#text} at (1,1) size 249x18
                   text run at (1,1) width 249: "check box with value property changed"
-              RenderTableCell {TD} at (389,56) size 240x20 [r=2 c=1 rs=1 cs=1]
+              RenderTableCell {TD} at (389,56) size 242x20 [r=2 c=1 rs=1 cs=1]
                 RenderBlock {INPUT} at (4,4) size 12x12
-              RenderTableCell {TD} at (631,56) size 62x20 [r=2 c=2 rs=1 cs=1]
+              RenderTableCell {TD} at (633,56) size 62x20 [r=2 c=2 rs=1 cs=1]
                 RenderText {#text} at (1,1) size 28x18
                   text run at (1,1) width 28: "after"
-              RenderTableCell {TD} at (695,56) size 61x20 [r=2 c=3 rs=1 cs=1]
+              RenderTableCell {TD} at (697,56) size 61x20 [r=2 c=3 rs=1 cs=1]
                 RenderText {#text} at (1,1) size 28x18
                   text run at (1,1) width 28: "after"
-            RenderTableRow {TR} at (0,78) size 758x20
+            RenderTableRow {TR} at (0,78) size 760x20
               RenderTableCell {TD} at (2,78) size 385x20 [r=3 c=0 rs=1 cs=1]
                 RenderText {#text} at (1,1) size 227x18
                   text run at (1,1) width 227: "hidden with value property changed"
-              RenderTableCell {TD} at (389,87) size 240x2 [r=3 c=1 rs=1 cs=1]
-              RenderTableCell {TD} at (631,78) size 62x20 [r=3 c=2 rs=1 cs=1]
+              RenderTableCell {TD} at (389,87) size 242x2 [r=3 c=1 rs=1 cs=1]
+              RenderTableCell {TD} at (633,78) size 62x20 [r=3 c=2 rs=1 cs=1]
                 RenderText {#text} at (1,1) size 28x18
                   text run at (1,1) width 28: "after"
-              RenderTableCell {TD} at (695,78) size 61x20 [r=3 c=3 rs=1 cs=1]
+              RenderTableCell {TD} at (697,78) size 61x20 [r=3 c=3 rs=1 cs=1]
                 RenderText {#text} at (1,1) size 28x18
                   text run at (1,1) width 28: "after"
-            RenderTableRow {TR} at (0,100) size 758x24
+            RenderTableRow {TR} at (0,100) size 760x24
               RenderTableCell {TD} at (2,102) size 385x20 [r=4 c=0 rs=1 cs=1]
                 RenderText {#text} at (1,1) size 224x18
                   text run at (1,1) width 224: "button with value property changed"
-              RenderTableCell {TD} at (389,100) size 240x24 [r=4 c=1 rs=1 cs=1]
+              RenderTableCell {TD} at (389,100) size 242x24 [r=4 c=1 rs=1 cs=1]
                 RenderButton {INPUT} at (3,3) size 41x18 [bgcolor=#C0C0C0]
                   RenderBlock (anonymous) at (8,2) size 25x13
                     RenderText at (0,0) size 25x13
                       text run at (0,0) width 25: "after"
-              RenderTableCell {TD} at (631,102) size 62x20 [r=4 c=2 rs=1 cs=1]
+              RenderTableCell {TD} at (633,102) size 62x20 [r=4 c=2 rs=1 cs=1]
                 RenderText {#text} at (1,1) size 28x18
                   text run at (1,1) width 28: "after"
-              RenderTableCell {TD} at (695,102) size 61x20 [r=4 c=3 rs=1 cs=1]
+              RenderTableCell {TD} at (697,102) size 61x20 [r=4 c=3 rs=1 cs=1]
                 RenderText {#text} at (1,1) size 28x18
                   text run at (1,1) width 28: "after"
-            RenderTableRow {TR} at (0,126) size 758x20
+            RenderTableRow {TR} at (0,126) size 760x20
               RenderTableCell {TD} at (2,126) size 385x20 [r=5 c=0 rs=1 cs=1]
                 RenderText {#text} at (1,1) size 222x18
                   text run at (1,1) width 222: "image with value property changed"
-              RenderTableCell {TD} at (389,135) size 240x2 [r=5 c=1 rs=1 cs=1]
+              RenderTableCell {TD} at (389,135) size 242x2 [r=5 c=1 rs=1 cs=1]
                 RenderImage {INPUT} at (1,1) size 0x0
-              RenderTableCell {TD} at (631,126) size 62x20 [r=5 c=2 rs=1 cs=1]
+              RenderTableCell {TD} at (633,126) size 62x20 [r=5 c=2 rs=1 cs=1]
                 RenderText {#text} at (1,1) size 28x18
                   text run at (1,1) width 28: "after"
-              RenderTableCell {TD} at (695,126) size 61x20 [r=5 c=3 rs=1 cs=1]
+              RenderTableCell {TD} at (697,126) size 61x20 [r=5 c=3 rs=1 cs=1]
                 RenderText {#text} at (1,1) size 28x18
                   text run at (1,1) width 28: "after"
-            RenderTableRow {TR} at (0,148) size 758x21
+            RenderTableRow {TR} at (0,148) size 760x21
               RenderTableCell {TD} at (2,148) size 385x20 [r=6 c=0 rs=1 cs=1]
                 RenderText {#text} at (1,1) size 216x18
                   text run at (1,1) width 216: "radio with value property changed"
-              RenderTableCell {TD} at (389,148) size 240x21 [r=6 c=1 rs=1 cs=1]
+              RenderTableCell {TD} at (389,148) size 242x21 [r=6 c=1 rs=1 cs=1]
                 RenderBlock {INPUT} at (4,4) size 12x13
-              RenderTableCell {TD} at (631,148) size 62x20 [r=6 c=2 rs=1 cs=1]
+              RenderTableCell {TD} at (633,148) size 62x20 [r=6 c=2 rs=1 cs=1]
                 RenderText {#text} at (1,1) size 28x18
                   text run at (1,1) width 28: "after"
-              RenderTableCell {TD} at (695,148) size 61x20 [r=6 c=3 rs=1 cs=1]
+              RenderTableCell {TD} at (697,148) size 61x20 [r=6 c=3 rs=1 cs=1]
                 RenderText {#text} at (1,1) size 28x18
                   text run at (1,1) width 28: "after"
-            RenderTableRow {TR} at (0,171) size 758x25
+            RenderTableRow {TR} at (0,171) size 760x25
               RenderTableCell {TD} at (2,173) size 385x20 [r=7 c=0 rs=1 cs=1]
                 RenderText {#text} at (1,1) size 205x18
                   text run at (1,1) width 205: "text with value attribute changed"
-              RenderTableCell {TD} at (389,171) size 240x25 [r=7 c=1 rs=1 cs=1]
+              RenderTableCell {TD} at (389,171) size 242x25 [r=7 c=1 rs=1 cs=1]
                 RenderTextField {INPUT} at (3,3) size 148x19 [bgcolor=#FFFFFF] [border: (2px inset #000000)]
-              RenderTableCell {TD} at (631,173) size 62x20 [r=7 c=2 rs=1 cs=1]
+              RenderTableCell {TD} at (633,173) size 62x20 [r=7 c=2 rs=1 cs=1]
                 RenderText {#text} at (1,1) size 28x18
                   text run at (1,1) width 28: "after"
-              RenderTableCell {TD} at (695,173) size 61x20 [r=7 c=3 rs=1 cs=1]
+              RenderTableCell {TD} at (697,173) size 61x20 [r=7 c=3 rs=1 cs=1]
                 RenderText {#text} at (1,1) size 28x18
                   text run at (1,1) width 28: "after"
-            RenderTableRow {TR} at (0,198) size 758x20
+            RenderTableRow {TR} at (0,198) size 760x20
               RenderTableCell {TD} at (2,198) size 385x20 [r=8 c=0 rs=1 cs=1]
                 RenderText {#text} at (1,1) size 247x18
                   text run at (1,1) width 247: "check box with value attribute changed"
-              RenderTableCell {TD} at (389,198) size 240x20 [r=8 c=1 rs=1 cs=1]
+              RenderTableCell {TD} at (389,198) size 242x20 [r=8 c=1 rs=1 cs=1]
                 RenderBlock {INPUT} at (4,4) size 12x12
-              RenderTableCell {TD} at (631,198) size 62x20 [r=8 c=2 rs=1 cs=1]
+              RenderTableCell {TD} at (633,198) size 62x20 [r=8 c=2 rs=1 cs=1]
                 RenderText {#text} at (1,1) size 28x18
                   text run at (1,1) width 28: "after"
-              RenderTableCell {TD} at (695,198) size 61x20 [r=8 c=3 rs=1 cs=1]
+              RenderTableCell {TD} at (697,198) size 61x20 [r=8 c=3 rs=1 cs=1]
                 RenderText {#text} at (1,1) size 28x18
                   text run at (1,1) width 28: "after"
-            RenderTableRow {TR} at (0,220) size 758x20
+            RenderTableRow {TR} at (0,220) size 760x20
               RenderTableCell {TD} at (2,220) size 385x20 [r=9 c=0 rs=1 cs=1]
                 RenderText {#text} at (1,1) size 383x18
                   text run at (1,1) width 383: "text with value property changed, then turned into check box"
-              RenderTableCell {TD} at (389,220) size 240x20 [r=9 c=1 rs=1 cs=1]
+              RenderTableCell {TD} at (389,220) size 242x20 [r=9 c=1 rs=1 cs=1]
                 RenderBlock {INPUT} at (4,4) size 12x12
-              RenderTableCell {TD} at (631,220) size 62x20 [r=9 c=2 rs=1 cs=1]
+              RenderTableCell {TD} at (633,220) size 62x20 [r=9 c=2 rs=1 cs=1]
                 RenderText {#text} at (1,1) size 28x18
                   text run at (1,1) width 28: "after"
-              RenderTableCell {TD} at (695,220) size 61x20 [r=9 c=3 rs=1 cs=1]
+              RenderTableCell {TD} at (697,220) size 61x20 [r=9 c=3 rs=1 cs=1]
                 RenderText {#text} at (1,1) size 28x18
                   text run at (1,1) width 28: "after"
-            RenderTableRow {TR} at (0,242) size 758x25
+            RenderTableRow {TR} at (0,242) size 760x25
               RenderTableCell {TD} at (2,244) size 385x20 [r=10 c=0 rs=1 cs=1]
                 RenderText {#text} at (1,1) size 383x18
                   text run at (1,1) width 383: "check box with value property changed, then turned into text"
-              RenderTableCell {TD} at (389,242) size 240x25 [r=10 c=1 rs=1 cs=1]
+              RenderTableCell {TD} at (389,242) size 242x25 [r=10 c=1 rs=1 cs=1]
                 RenderTextField {INPUT} at (3,3) size 148x19 [bgcolor=#FFFFFF] [border: (2px inset #000000)]
-              RenderTableCell {TD} at (631,244) size 62x20 [r=10 c=2 rs=1 cs=1]
+              RenderTableCell {TD} at (633,244) size 62x20 [r=10 c=2 rs=1 cs=1]
                 RenderText {#text} at (1,1) size 28x18
                   text run at (1,1) width 28: "after"
-              RenderTableCell {TD} at (695,244) size 61x20 [r=10 c=3 rs=1 cs=1]
+              RenderTableCell {TD} at (697,244) size 61x20 [r=10 c=3 rs=1 cs=1]
                 RenderText {#text} at (1,1) size 28x18
                   text run at (1,1) width 28: "after"
-            RenderTableRow {TR} at (0,269) size 758x20
+            RenderTableRow {TR} at (0,269) size 760x20
               RenderTableCell {TD} at (2,269) size 385x20 [r=11 c=0 rs=1 cs=1]
                 RenderText {#text} at (1,1) size 381x18
                   text run at (1,1) width 381: "text with value attribute changed, then turned into check box"
-              RenderTableCell {TD} at (389,269) size 240x20 [r=11 c=1 rs=1 cs=1]
+              RenderTableCell {TD} at (389,269) size 242x20 [r=11 c=1 rs=1 cs=1]
                 RenderBlock {INPUT} at (4,4) size 12x12
-              RenderTableCell {TD} at (631,269) size 62x20 [r=11 c=2 rs=1 cs=1]
+              RenderTableCell {TD} at (633,269) size 62x20 [r=11 c=2 rs=1 cs=1]
                 RenderText {#text} at (1,1) size 28x18
                   text run at (1,1) width 28: "after"
-              RenderTableCell {TD} at (695,269) size 61x20 [r=11 c=3 rs=1 cs=1]
+              RenderTableCell {TD} at (697,269) size 61x20 [r=11 c=3 rs=1 cs=1]
                 RenderText {#text} at (1,1) size 28x18
                   text run at (1,1) width 28: "after"
-            RenderTableRow {TR} at (0,291) size 758x25
+            RenderTableRow {TR} at (0,291) size 760x25
               RenderTableCell {TD} at (2,293) size 385x20 [r=12 c=0 rs=1 cs=1]
                 RenderText {#text} at (1,1) size 381x18
                   text run at (1,1) width 381: "check box with value attribute changed, then turned into text"
-              RenderTableCell {TD} at (389,291) size 240x25 [r=12 c=1 rs=1 cs=1]
+              RenderTableCell {TD} at (389,291) size 242x25 [r=12 c=1 rs=1 cs=1]
                 RenderTextField {INPUT} at (3,3) size 148x19 [bgcolor=#FFFFFF] [border: (2px inset #000000)]
-              RenderTableCell {TD} at (631,293) size 62x20 [r=12 c=2 rs=1 cs=1]
+              RenderTableCell {TD} at (633,293) size 62x20 [r=12 c=2 rs=1 cs=1]
                 RenderText {#text} at (1,1) size 28x18
                   text run at (1,1) width 28: "after"
-              RenderTableCell {TD} at (695,293) size 61x20 [r=12 c=3 rs=1 cs=1]
+              RenderTableCell {TD} at (697,293) size 61x20 [r=12 c=3 rs=1 cs=1]
                 RenderText {#text} at (1,1) size 28x18
                   text run at (1,1) width 28: "after"
-            RenderTableRow {TR} at (0,318) size 758x24
+            RenderTableRow {TR} at (0,318) size 760x24
               RenderTableCell {TD} at (2,320) size 385x20 [r=13 c=0 rs=1 cs=1]
                 RenderText {#text} at (1,1) size 204x18
                   text run at (1,1) width 204: "file with value property changed"
-              RenderTableCell {TD} at (389,318) size 240x24 [r=13 c=1 rs=1 cs=1]
-                RenderFileButton {INPUT} at (3,3) size 234x18
-              RenderTableCell {TD} at (631,329) size 62x2 [r=13 c=2 rs=1 cs=1]
-              RenderTableCell {TD} at (695,320) size 61x20 [r=13 c=3 rs=1 cs=1]
+              RenderTableCell {TD} at (389,318) size 242x24 [r=13 c=1 rs=1 cs=1]
+                RenderFileUploadControl {INPUT} at (3,3) size 236x18
+                  RenderButton {INPUT} at (0,0) size 78x18 [bgcolor=#C0C0C0]
+                    RenderBlock (anonymous) at (8,2) size 62x13
+                      RenderText at (0,0) size 62x13
+                        text run at (0,0) width 62: "Choose File"
+                  RenderFileUploadInnerFileBox at (78,2) size 158x13
+                    RenderBlock (anonymous) at (4,0) size 0x13
+                      RenderText at (0,0) size 0x13
+                        text run at (0,0) width 0: "no file selected"
+              RenderTableCell {TD} at (633,329) size 62x2 [r=13 c=2 rs=1 cs=1]
+              RenderTableCell {TD} at (697,320) size 61x20 [r=13 c=3 rs=1 cs=1]
                 RenderText {#text} at (1,1) size 40x18
                   text run at (1,1) width 40: "before"
 layer at (403,136) size 142x13
index 9741eac745641ab2f41cae6165578aadc2d4cac8..ff947a7d16a306d0809cbe617dcb176719e33463 100644 (file)
@@ -1,3 +1,15 @@
+2006-09-15  Adam Roben  <aroben@apple.com>
+
+        Reviewed by timothy.
+
+        Fixes http://bugzilla.opendarwin.org/show_bug.cgi?id=10865
+        New file upload control should match the width of the old one
+
+        Make width of new file upload control match the old one as closely as
+        possible.
+
+        * rendering/RenderFileUploadControl.cpp:
+
 2006-09-15  Timothy Hatcher  <timothy@apple.com>
 
         Reviewed by Eric.
index bd84a89e34ff926b4efab480cebca663592c7420..3f4a44840fe51814f32a6f7b2932f029944842b9 100644 (file)
@@ -43,7 +43,7 @@ const int afterButtonSpacing = 4;
 const int iconHeight = 16;
 const int iconWidth = 16;
 const int iconFilenameSpacing = 2;
-const int defaultFilenameNumChars = 23;
+const int defaultFilenameNumChars = 22;
 
 using namespace HTMLNames;