Closed Bug 500633 Opened 15 years ago Closed 15 years ago

remove obsolete ifSetAttribute method

Categories

(Toolkit :: Autocomplete, defect)

defect
Not set
trivial

Tracking

()

RESOLVED FIXED
mozilla1.9.2a1

People

(Reporter: dao, Assigned: dao)

Details

Attachments

(1 file)

Attached patch patchSplinter Review
      No description provided.
Attachment #385359 - Flags: review?(neil)
Comment on attachment 385359 [details] [diff] [review]
patch

>       <method name="_focus">
>         <!-- doesn't reset this.mController -->
>         <body><![CDATA[
>           this._dontBlur = true;
>           this.focus();
>           this._dontBlur = false;
>         ]]></body>
>       </method>
[Hey, I wonder whether Enn's focus changes makes this obsolete?]
Attachment #385359 - Flags: review?(neil) → review+
(In reply to comment #1)
> [Hey, I wonder whether Enn's focus changes makes this obsolete?]

Not sure... I thought _dontBlur actually made sense and didn't just workaround some focus weirdness.
http://hg.mozilla.org/mozilla-central/rev/882cfa39b73d
Status: ASSIGNED → RESOLVED
Closed: 15 years ago
Resolution: --- → FIXED
Target Milestone: --- → mozilla1.9.2a1
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Created:
Updated:
Size: