158937 - comment wrong

Status: RESOLVED FIXED

(Core :: Layout, defect, P3)

Description

[Christian :Biesinger (don't email me, ping me on IRC)]

the comment reads // BUTTON CONSTRUCTION, however, the code says:
else if (aTag == nsXULAtoms::autorepeatbutton) {

(see also line 5345)

should probably be changed to AUTOREPEATBUTTON

Comment 1

[Boris Zbarsky [:bzbarsky]]

biesi, wanna fix this?  Marc's not likely to, so... ;)

Comment 2

[Christian :Biesinger (don't email me, ping me on IRC)]

taking bug

Comment 3

[Christian :Biesinger (don't email me, ping me on IRC)]

Index: style/src/nsCSSFrameConstructor.cpp
===================================================================
RCS file: /cvsroot/mozilla/layout/html/style/src/nsCSSFrameConstructor.cpp,v
retrieving revision 1.754
diff -u -r1.754 nsCSSFrameConstructor.cpp
--- style/src/nsCSSFrameConstructor.cpp	19 Jul 2002 21:58:09 -0000	1.754
+++ style/src/nsCSSFrameConstructor.cpp	24 Jul 2002 07:58:48 -0000
@@ -5321,7 +5321,7 @@
 
         } 
       } // End of BUTTON CONSTRUCTION logic
-   // BUTTON CONSTRUCTION
+      // AUTOREPEATBUTTON CONSTRUCTION
       else if (aTag == nsXULAtoms::autorepeatbutton) {
         processChildren = PR_TRUE;
         isReplaced = PR_TRUE;
@@ -5342,7 +5342,7 @@
           frameHasBeenInitialized = PR_TRUE;
 
         } 
-      } // End of BUTTON CONSTRUCTION logic
+      } // End of AUTOREPEATBUTTON CONSTRUCTION logic
 
 
 	   // TITLEBAR CONSTRUCTION

Comment 4

[Christian :Biesinger (don't email me, ping me on IRC)]

r=timeless sr=bz on irc

Comment 5

[Asa Dotzler [:asa]]

a=asa (on behalf of drivers) for checkin to 1.1

Comment 6

[Christian :Biesinger (don't email me, ping me on IRC)]

checked in