Closed
Bug 740076
Opened 13 years ago
Closed 13 years ago
autoFill should be disabled by the autocomplete preference
Categories
(Firefox :: Address Bar, defect)
Tracking
()
VERIFIED
FIXED
Firefox 15
People
(Reporter: Virtual, Assigned: mak)
References
Details
(Keywords: nightly-community, regression)
Attachments
(4 files)
(deleted),
image/png
|
Details | |
(deleted),
image/png
|
Details | |
(deleted),
image/png
|
Details | |
(deleted),
patch
|
dietrich
:
review+
akeybl
:
approval-mozilla-aurora+
|
Details | Diff | Splinter Review |
Location bar suggest some sites from bookmarks, even if all settings are disabled for autofilling in options...
Comment 4•13 years ago
|
||
Confirmed on Mozilla/5.0 (Windows NT 5.1; rv:14.0) Gecko/20120328 Firefox/14.0a1 ID:20120328031211
Version: unspecified → Trunk
Reporter | ||
Updated•13 years ago
|
Keywords: regression,
regressionwindow-wanted
Version: Trunk → 14 Branch
Reporter | ||
Updated•13 years ago
|
tracking-firefox14:
--- → ?
Updated•13 years ago
|
Reporter | ||
Comment 5•13 years ago
|
||
FYI - This is regression from version 13. When I will have more free time I will try to find it.
Comment 6•13 years ago
|
||
I imagine this is because the inline autocomplete search doesn't obey the "autocomplete" prefs.
Status: UNCONFIRMED → NEW
tracking-firefox13:
--- → +
tracking-firefox14:
+ → ---
Ever confirmed: true
Keywords: regressionwindow-wanted
OS: Windows 7 → All
Assignee | ||
Comment 7•13 years ago
|
||
(In reply to Gavin Sharp (use gavin@gavinsharp.com for email) from comment #6)
> I imagine this is because the inline autocomplete search doesn't obey the
> "autocomplete" prefs.
right.
Reporter | ||
Comment 8•13 years ago
|
||
BUMP, Will it make into Firefox 13 ?
I'm asking, because Firefox 12 was released and this bug is hella annoying as I always use location bar for google search and this issue make it completely unavailable ;p
So please, don't forget about this.
Comment 9•13 years ago
|
||
(In reply to Virtual_ManPL from comment #8)
> BUMP, Will it make into Firefox 13 ?
Inline auto-complete won't ship until FF14, so tracking for that release.
tracking-firefox13:
+ → ---
tracking-firefox14:
--- → +
Comment 10•13 years ago
|
||
mak, would you be the right person to look into this for FF14, or would ddahl be?
Assignee: nobody → mak77
Assignee | ||
Comment 11•13 years ago
|
||
I may look at it, if we consider it a blocker for enabling the feature, currently we have no UI to enable/disable the feature, the prefs UI above controls the popup behavior, though that's completely unclear to the end user.
Assignee | ||
Comment 12•13 years ago
|
||
I will split out some complication from here.
Basically, the blocking issue is that having autocomplete UI pref set to "Suggest nothing" actually is confusing, cause autoFill keeps suggesting urls. This should be easy to patch and test, and safe to backport to Aurora.
The issue on top, that I'd move to a separate bug, is that autoFill doesn't obey when selecting only-history or only-bookmarks options. This is quite more complicated to get right, especially from a performances point of view. Since these are more technical options and autoFill acts already is some sort of exotic way (looking detached from the popup results and only filling typed entries) this issue looks far less problematic to live with.
Assignee | ||
Comment 13•13 years ago
|
||
And as a side note to issue 2, we also have bug 530209, that suggests we want to revise the ui there, to clarify it.
Comment 14•13 years ago
|
||
(In reply to Marco Bonardo [:mak] from comment #12)
I concur wholeheartedly!
Assignee | ||
Updated•13 years ago
|
Summary: Location bar suggest sth from bookmarks even if all settings are disabled for autofilling in options → autoFill should be disabled by the autocomplete preference
Assignee | ||
Comment 15•13 years ago
|
||
Attachment #621001 -
Flags: review?(dietrich)
Assignee | ||
Updated•13 years ago
|
Severity: major → normal
Flags: in-testsuite+
Comment 16•13 years ago
|
||
Comment on attachment 621001 [details] [diff] [review]
patch v1.0
Review of attachment 621001 [details] [diff] [review]:
-----------------------------------------------------------------
::: toolkit/components/places/tests/inline/test_autocomplete_functional.js
@@ +34,5 @@
> "vis",
> "visit2.mozilla.org/",
> function ()
> {
> + Services.prefs.setBoolPref("browser.urlbar.autoFill", true);
optional fix: i wish this could be done in the right test, as a cleanup func, instead of spread across to an unrelated test.
Attachment #621001 -
Flags: review?(dietrich) → review+
Assignee | ||
Comment 17•13 years ago
|
||
(In reply to Dietrich Ayala (:dietrich) from comment #16)
> optional fix: i wish this could be done in the right test, as a cleanup
> func, instead of spread across to an unrelated test.
will move it to the head, so that each test starts in a clean env with known prefs setup.
Assignee | ||
Comment 18•13 years ago
|
||
Target Milestone: --- → Firefox 15
Comment 19•13 years ago
|
||
Status: NEW → RESOLVED
Closed: 13 years ago
Resolution: --- → FIXED
Reporter | ||
Comment 20•13 years ago
|
||
Verified with latest hourly.
Works properly now.
Thank you very much!
Status: RESOLVED → VERIFIED
Assignee | ||
Comment 21•13 years ago
|
||
Comment on attachment 621001 [details] [diff] [review]
patch v1.0
[Approval Request Comment]
Regression caused by (bug #): No regression, this is a blocker for keeping the feature enabled in Firefox 14
User impact if declined: Disable the feature
Testing completed (on m-c, etc.): m-c
Risk to taking this patch (and alternatives if risky): quite low, small patch leveraging existing code paths
String changes made by this patch: none
Attachment #621001 -
Flags: approval-mozilla-aurora?
Comment 22•13 years ago
|
||
Comment on attachment 621001 [details] [diff] [review]
patch v1.0
[Triage Comment]
Approved for Aurora 14 since this will partially dictate whether the feature is left enabled on that release.
Attachment #621001 -
Flags: approval-mozilla-aurora? → approval-mozilla-aurora+
Assignee | ||
Comment 23•13 years ago
|
||
status-firefox14:
--- → fixed
Reporter | ||
Updated•7 years ago
|
Keywords: nightly-community
Reporter | ||
Updated•7 years ago
|
QA Contact: Virtual
You need to log in
before you can comment on or make changes to this bug.
Description
•