This came up in bug 237842. We should try to see if we can use a <button> element in nsComboBoxControlFrame instead of the currently used <input type=button>, which has way more overhead.
Created attachment 521091 [details] [diff] [review] Like so? I left the SetAttr(type, button) call, because a <button> with no type defaults to submit, unfortunately. This seems to be about 40ms faster on construction, and aabout 15ms faster on destruction on the testcase in bug 237842. We could win another 40ms or so on construction by using a <div> or something, but then we'd need to do all the styling for it correctly. Ehsan, are you willing to test this on Windows? I think that's the one place we still show the dropmarker when using the system appearance....
Attachment #521091 - Flags: review?(ehsan)
Even more ideally we would just not have a separate dropmarker element. That would take more work, though.
Whiteboard: [need review]
Comment on attachment 521091 [details] [diff] [review] Like so? This looks sane to me. I'll test it on Windows too.
Attachment #521091 - Flags: review?(ehsan) → review+
This looks to be fine on Windows, too!
Whiteboard: [need review] → [need landing]
Status: NEW → RESOLVED
Last Resolved: 8 years ago
Resolution: --- → FIXED
Whiteboard: [need landing]
Target Milestone: --- → mozilla2.2
You need to log in before you can comment on or make changes to this bug.