Properly document selection resetting in UrlbarInput::_loadURL
Categories
(Firefox :: Address Bar, task, P3)
Tracking
()
People
(Reporter: dao, Assigned: okonaraddi)
References
Details
Attachments
(1 file)
47 bytes,
text/x-phabricator-request
|
jcristau
:
approval-mozilla-beta+
|
Details | Review |
In bug 1533691, mak said that this code would be "necessary for spoof protection, so that the domain stays visible." The code comment however says: "Ensure the start of the URL is visible for usability reasons."
This is vague enough to be useless if not actively misleading.
Comment 1•6 years ago
|
||
Yeah, it's actually both usability and spoof prevention, the host is an important information that should stay visible as far as possible.
Reporter | ||
Comment 2•6 years ago
|
||
Gonna up the priority because P5 is almost wontfix and the current documentation is just not good. Also marking Task. (By definition tasks have low user impact, and making all tasks P5s seems a bit silly.)
Assignee | ||
Updated•6 years ago
|
Assignee | ||
Comment 3•6 years ago
|
||
Reporter | ||
Comment 5•6 years ago
|
||
Comment on attachment 9069791 [details]
Bug 1534552 - Properly document selection resetting in UrlbarInput::_loadURL r=adw!
Beta/Release Uplift Approval Request
- User impact if declined: No user impact, could potentially avoid conflicts with future quantumbar uplifts
- Is this code covered by automated tests?: No
- Has the fix been verified in Nightly?: No
- Needs manual test from QE?: No
- If yes, steps to reproduce:
- List of other uplifts needed: None
- Risk to taking this patch: Low
- Why is the change risky/not risky? (and alternatives if risky): Only fixing up a comment
- String changes made/needed:
Reporter | ||
Updated•6 years ago
|
Comment 6•6 years ago
|
||
bugherder |
Comment 7•6 years ago
|
||
Comment on attachment 9069791 [details]
Bug 1534552 - Properly document selection resetting in UrlbarInput::_loadURL r=adw!
updating a comment, effectively a no-op, approved for 68.0b8
Comment 8•6 years ago
|
||
bugherder uplift |
Description
•