Closed Bug 729244 Opened 12 years ago Closed 12 years ago

ESC doesn't revert the location bar value anymore

Categories

(Toolkit :: Autocomplete, defect)

defect
Not set
normal

Tracking

()

RESOLVED FIXED
mozilla13

People

(Reporter: dao, Assigned: dao)

References

Details

(Keywords: regression)

Attachments

(1 file)

STR:
- type something in the location bar
- hit ESC

expected: the current page's address should be restored

actual result: nothing happens
Last good nightly: 2012-02-17
First bad nightly: 2012-02-18

Pushlog:
http://hg.mozilla.org/mozilla-central/pushloghtml?fromchange=2271cb92cc05&tochange=550779e6bab4
The first bad revision is:
changeset:   87042:f01f200f798a
user:        Felix Fung <ffung@mozilla.com>
date:        Thu Feb 16 14:56:06 2012 -0500
summary:     Bug 495277 - Reduce calls to new Function in Autocomplete.xml. r=gavin

https://hg.mozilla.org/mozilla-central/rev/f01f200f798a
Hmm, that's odd. Calling gURLBar.onTextReverted() manually still works fine, but the call from the autocomplete controller doesn't seem to work.
Attached patch patchSplinter Review
Assignee: nobody → dao
Status: NEW → ASSIGNED
Attachment #599560 - Flags: review?(gavin.sharp)
Component: Location Bar → Autocomplete
Product: Firefox → Toolkit
QA Contact: location.bar → autocomplete
(In reply to Thomas Ahlblom from comment #2)
> The first bad revision is:
> changeset:   87042:f01f200f798a

Thanks!
Attachment #599560 - Flags: review?(gavin.sharp) → review+
We should get test coverage for this as well.
https://hg.mozilla.org/integration/mozilla-inbound/rev/6cbae3d86fb4
Flags: in-testsuite+
Target Milestone: --- → mozilla13
https://hg.mozilla.org/mozilla-central/rev/6cbae3d86fb4
Status: ASSIGNED → RESOLVED
Closed: 12 years ago
Resolution: --- → FIXED
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Created:
Updated:
Size: