1431470 - Expose bookmarks usage data for snippets

Status: RESOLVED FIXED

(Firefox :: Messaging System, enhancement, P1)

Description

[Jean Collings]

We would like to be able to target snippets to users based on their bookmarks usage. 

In Telemetry, under the Histogram section, there is an attribute ""PLACES_BOOKMARKS_COUNT" which tells us the number of bookmarks saved.

Kate please let me know what else you need - hoping to get this in Q1.

Comment 1

[Kate Hudson :k88hudson]

Attachment: Bug 1431470 - Add getTotalBookmarksCount to NewTabUtils.jsm

Review commit: https://reviewboard.mozilla.org/r/223626/diff/#index_header
See other reviews: https://reviewboard.mozilla.org/r/223626/

Comment 2

[Ursula Sarracini (:ursula)]

Comment on attachment 8954570 [details]
Bug 1431470 - Add getTotalBookmarksCount to NewTabUtils.jsm

https://reviewboard.mozilla.org/r/223626/#review229896

Looks good, just one nit...the other queries use camel case for `tagsFolderId` and `bookmarkType` rather than underscore. Let's just keep it consistent

Comment 3

[Pulsebot]

Pushed by [e-mail]:
https://hg.mozilla.org/integration/autoland/rev/85d750c79630
Add getTotalBookmarksCount to NewTabUtils.jsm r=ursula

Comment 4

[GitHub Bugzilla PR Linker]

Attachment: Link to GitHub pull-request: https://github.com/mozilla/activity-stream/pull/4010

Comment 5

[[github robot]]

Commits pushed to master at https://github.com/mozilla/activity-stream

https://github.com/mozilla/activity-stream/commit/8164c53149aab8f415eb2041ff3f703027988264
Fix Bug 1431470 - Add .getTotalBookmarksCount() to gSnippetsMap

https://github.com/mozilla/activity-stream/commit/9df2a5ee0dd7155f95cc1bfd2b5ff2db2d11ed64
Merge pull request #4010 from k88hudson/bug1431470

Fix Bug 1431470 - Add .getTotalBookmarksCount() to gSnippetsMap

Comment 6

[Tiberius Oros[:tiberius_oros]]

https://hg.mozilla.org/mozilla-central/rev/85d750c79630

Comment 7

[Ed Lee :Mardak]

https://hg.mozilla.org/mozilla-central/rev/723f97b68476