Closed
Bug 944801
Opened 11 years ago
Closed 10 years ago
Collect UITelemetry on how many links are opened from the bookmarks menu button
Categories
(Firefox :: Toolbars and Customization, defect)
Tracking
()
RESOLVED
FIXED
Firefox 29
People
(Reporter: mconley, Assigned: mconley)
References
(Depends on 1 open bug)
Details
(Whiteboard: [fixed-in-holly][good first verify])
Attachments
(2 files)
995 bytes,
patch
|
Gijs
:
review+
lsblakk
:
approval-mozilla-aurora+
lsblakk
:
approval-mozilla-beta+
|
Details | Diff | Splinter Review |
936 bytes,
patch
|
Gijs
:
review+
|
Details | Diff | Splinter Review |
In order to measure bookmarks menu button usage, we want to count how many links are opened from it.
Assignee | ||
Updated•11 years ago
|
Whiteboard: [Australis:P1] → [Australis:P1][Non-Australis-Only]
Assignee | ||
Comment 1•11 years ago
|
||
BMB_bookmarksPopup:child should match any bookmark opened under the bookmarks menu button.
Attachment #8346104 -
Flags: review?(gijskruitbosch+bugs)
Updated•11 years ago
|
Attachment #8346104 -
Flags: review?(gijskruitbosch+bugs) → review+
Assignee | ||
Comment 2•11 years ago
|
||
Thanks Gijs! Landed on Holly as https://hg.mozilla.org/projects/holly/rev/b55bba3d7f25
Whiteboard: [Australis:P1][Non-Australis-Only] → [Australis:P1][Non-Australis-Only][fixed-in-holly]
Assignee | ||
Comment 3•11 years ago
|
||
Comment on attachment 8346104 [details] [diff] [review] Patch v1 - for non-Australis [Approval Request Comment] Bug caused by (feature/regressing bug #): None. This gives BrowserUITelemetry the capability to count how many times bookmarks are opened from the bookmarks menu button. User impact if declined: None. Testing completed (on m-c, etc.): Only manual testing. Risk to taking this patch (and alternatives if risky): None. String or IDL/UUID changes made by this patch: None.
Attachment #8346104 -
Flags: approval-mozilla-beta?
Attachment #8346104 -
Flags: approval-mozilla-aurora?
Assignee | ||
Comment 4•11 years ago
|
||
Removing Australis:P1 whiteboard tag because these already block an Australis:P1 and being redundant isn't helpful.
Whiteboard: [Australis:P1][Non-Australis-Only][fixed-in-holly] → [Non-Australis-Only][fixed-in-holly]
Updated•11 years ago
|
Attachment #8346104 -
Flags: approval-mozilla-beta?
Attachment #8346104 -
Flags: approval-mozilla-beta+
Attachment #8346104 -
Flags: approval-mozilla-aurora?
Attachment #8346104 -
Flags: approval-mozilla-aurora+
Assignee | ||
Comment 5•11 years ago
|
||
Landed in mozilla-beta as https://hg.mozilla.org/releases/mozilla-beta/rev/262bd97fdd8c
status-firefox27:
--- → fixed
Assignee | ||
Comment 6•11 years ago
|
||
Landed in mozilla-aurora as https://hg.mozilla.org/releases/mozilla-aurora/rev/38cc1844cd2f
status-firefox28:
--- → fixed
Whiteboard: [Non-Australis-Only][fixed-in-holly] → [Non-Australis-Only][fixed-in-holly][good first verify]
Assignee | ||
Updated•10 years ago
|
Whiteboard: [Non-Australis-Only][fixed-in-holly][good first verify] → [fixed-in-holly][good first verify]
Assignee | ||
Updated•10 years ago
|
Attachment #8346104 -
Attachment description: Patch v1 → Patch v1 - for non-Australis
Assignee | ||
Comment 7•10 years ago
|
||
So it turns out this was remarkably easy. One caveat is that sometimes mouseups are fired for the "arrow" of the panel when after clicking on the dropmarker. It's possible Enn's patch in bug 941409 will fix this, but if not, we're going to have to address this somehow. Worst case scenario, we'll need to subtract menu button clicks from the BMB_bookmarksPopup click count. I'll file a separate bug for that.
Assignee | ||
Comment 8•10 years ago
|
||
Comment on attachment 8361676 [details] [diff] [review] Patch v1 - for Australis Almost... there...
Attachment #8361676 -
Flags: review?(gijskruitbosch+bugs)
Updated•10 years ago
|
Attachment #8361676 -
Flags: review?(gijskruitbosch+bugs) → review+
Assignee | ||
Comment 9•10 years ago
|
||
Thanks! remote: https://hg.mozilla.org/integration/fx-team/rev/736fa1afd493
Comment 10•10 years ago
|
||
https://hg.mozilla.org/mozilla-central/rev/736fa1afd493
Status: NEW → RESOLVED
Closed: 10 years ago
Resolution: --- → FIXED
Target Milestone: --- → Firefox 29
Comment 11•10 years ago
|
||
[bugday-20140122] - Could you please provide the exact STR(Steps to Reproduce) to check for fixed
Flags: needinfo?(mconley)
Assignee | ||
Comment 12•10 years ago
|
||
(In reply to svbabukumar from comment #11) > [bugday-20140122] - Could you please provide the exact STR(Steps to > Reproduce) to check for fixed Sure, here goes: 1) In Nightly, open about:telemetry and ensure Telemetry is enabled. If not, enable and restart the browser. 2) Click on the right side of the Bookmarks menu button to bring up the list of bookmarks. 3) Click on a bookmark. 4) Repeat steps 2 and 3 any number of times 5) Open about:telemetry, and expand the "Simple Measurements" section. 6) There should be a row labeled "UITelemetry". On the right half of that row is a big JSON blob. You're looking for and entry in that blob called "BMB_bookmarksPopup". The entry should be a key, mapped to an object that will hold the count of how many times a bookmark has been clicked from the popup with each mouse button. So, for example, if you clicked on 15 bookmarks in the popup with the left mouse button, and right-clicked one of those bookmarks once, you'd see: BMB_bookmarksPopup: { left: 15, right: 1 } And that's it! Let me know if you have more questions.
Flags: needinfo?(mconley)
You need to log in
before you can comment on or make changes to this bug.
Description
•