Closed
Bug 1441984
Opened 6 years ago
Closed 6 years ago
Hide OpeninPrivateBrowsing option in AS if PrivateBrowsingUtils.enabled is false
Categories
(Firefox :: New Tab Page, enhancement, P1)
Firefox
New Tab Page
Tracking
()
Tracking | Status | |
---|---|---|
firefox60 | --- | fixed |
People
(Reporter: k88hudson, Assigned: k88hudson)
References
Details
Attachments
(1 file)
This is needed in order for our context menus to hide the open in private browsing option, and maybe useful for snippets anyway. See Bug 1432355 for more details.
Assignee | ||
Updated•6 years ago
|
Assignee: nobody → khudson
Iteration: --- → 60.4 - Mar 12
Comment 1•6 years ago
|
||
Comment 2•6 years ago
|
||
Commits pushed to master at https://github.com/mozilla/activity-stream https://github.com/mozilla/activity-stream/commit/1623117fcb87682ffd4615be6a66f9c35c09becd Bug 1441984 - Only show OpenInPrivateBrowsing option if private browsing enabled https://github.com/mozilla/activity-stream/commit/f30bea2abda255e3c21c81b3c482a437491521f5 Merge pull request #4011 from k88hudson/bug1441984 Bug 1441984 - Only show OpenInPrivateBrowsing option if private browsing enabled
Assignee | ||
Updated•6 years ago
|
Summary: Add PrivateBrowsingUtils.enabled to snippets data → Hide OpeninPrivateBrowsing option in AS if PrivateBrowsingUtils.enabled is false
Assignee | ||
Updated•6 years ago
|
Status: NEW → RESOLVED
Closed: 6 years ago
Resolution: --- → FIXED
Assignee | ||
Updated•6 years ago
|
Priority: -- → P1
Assignee | ||
Updated•6 years ago
|
Severity: normal → enhancement
Comment 3•6 years ago
|
||
This bug depends on bug 1432355 to expose `PrivateBrowsingUtils.enabled` otherwise test added from bug 1410818 fails: TEST-UNEXPECTED-FAIL | browser/extensions/activity-stream/test/functional/mochitest/browser_topsites_contextMenu_options.js | Name option is correct - "Dismiss" == "Open in a New Private Window" -
Comment 4•6 years ago
|
||
https://hg.mozilla.org/mozilla-central/rev/723f97b68476
status-firefox60:
--- → fixed
Target Milestone: --- → Firefox 60
Updated•5 years ago
|
Component: Activity Streams: Newtab → New Tab Page
You need to log in
before you can comment on or make changes to this bug.
Description
•