Closed
Bug 729244
Opened 12 years ago
Closed 12 years ago
ESC doesn't revert the location bar value anymore
Categories
(Toolkit :: Autocomplete, defect)
Toolkit
Autocomplete
Tracking
()
RESOLVED
FIXED
mozilla13
People
(Reporter: dao, Assigned: dao)
References
Details
(Keywords: regression)
Attachments
(1 file)
745 bytes,
patch
|
Gavin
:
review+
|
Details | Diff | Splinter Review |
STR: - type something in the location bar - hit ESC expected: the current page's address should be restored actual result: nothing happens
Comment 1•12 years ago
|
||
Last good nightly: 2012-02-17 First bad nightly: 2012-02-18 Pushlog: http://hg.mozilla.org/mozilla-central/pushloghtml?fromchange=2271cb92cc05&tochange=550779e6bab4
Comment 2•12 years ago
|
||
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
Blocks: 495277
Keywords: regressionwindow-wanted
Comment 3•12 years ago
|
||
Hmm, that's odd. Calling gURLBar.onTextReverted() manually still works fine, but the call from the autocomplete controller doesn't seem to work.
Assignee | ||
Comment 4•12 years ago
|
||
Assignee | ||
Updated•12 years ago
|
Component: Location Bar → Autocomplete
Product: Firefox → Toolkit
QA Contact: location.bar → autocomplete
Assignee | ||
Comment 5•12 years ago
|
||
(In reply to Thomas Ahlblom from comment #2) > The first bad revision is: > changeset: 87042:f01f200f798a Thanks!
Updated•12 years ago
|
Attachment #599560 -
Flags: review?(gavin.sharp) → review+
Comment 6•12 years ago
|
||
We should get test coverage for this as well.
Assignee | ||
Comment 7•12 years ago
|
||
https://hg.mozilla.org/integration/mozilla-inbound/rev/6cbae3d86fb4
Flags: in-testsuite+
Target Milestone: --- → mozilla13
Comment 8•12 years ago
|
||
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.
Description
•