988648 - Telemetry: Bookmarking/Saving

Status: VERIFIED FIXED

(Firefox for Android Graveyard :: General, defect, P1)

Description

[Chenxia Liu [:liuche]]

The goal of this probe is to measure what percentage of users bookmark/save things, how they do it, and how they revisit these pages.

Things to measure
* Pinning a site
  * Visiting that pinned site:
    * from Top Sites
    * from Awesomebar suggestions

* Bookmarking a page
  * Visiting that bookmarked page:
    * from Bookmarks
    * from Awesomebar suggestions
    * from Android home screen (for sites saved by "Add to Homescreen")

Comment 1

[Mark Finkle (:mfinkle) (use needinfo?)]

(In reply to Chenxia Liu [:liuche] from comment #0)
> The goal of this probe is to measure what percentage of users bookmark/save
> things, how they do it, and how they revisit these pages.
> 
> Things to measure
> * Pinning a site
>   * Visiting that pinned site:
>     * from Top Sites
>     * from Awesomebar suggestions

We already have telemetry for pinning and unpinning Top Sites. We also track opening URLs from Top Site thumbnails, but we don't really know if those thumbnails were pinned or not. Maybe we can check to see if the URL is "user-entered"? In any case, I'm not sure how important it is.

> * Bookmarking a page
>   * Visiting that bookmarked page:
>     * from Bookmarks
>     * from Awesomebar suggestions
>     * from Android home screen (for sites saved by "Add to Homescreen")

We don't currently track the save/unsave part of bookmarking. That's an easy fix. We do track opening URLs from the Bookmarks panel and tracking Awesomebar suggestions. We also track opening URLs from the homescreen.

I can make a patch for tracking save/unsave part of bookmarking.

Comment 2

[Mark Finkle (:mfinkle) (use needinfo?)]

Attachment: uitelemetry-bookmarking v0.1

Simple patch. Uses "bookmark" Extra which is consistent with our "reader" Extra for saving/unsaving to the Reading List.

Comment 3

[Mark Finkle (:mfinkle) (use needinfo?)]

https://hg.mozilla.org/integration/fx-team/rev/72928fbc2e04

Comment 4

[Mark Finkle (:mfinkle) (use needinfo?)]

Comment on attachment 8462946 [details] [diff] [review]
uitelemetry-bookmarking v0.1

Approval Request Comment
[Feature/regressing bug #]: None
[User impact if declined]: Let's get more data about bookmarking!
[Describe test coverage new/current, TBPL]: Landed
[Risks and why]: Low. Simple easy probes
[String/UUID change made/needed]: none

Comment 5

[Lawrence Mandel [:lmandel] (use needinfo)]

Comment on attachment 8462946 [details] [diff] [review]
uitelemetry-bookmarking v0.1

beta+ and aurora+

Comment 6

[Carsten Book [:Tomcat]]

https://hg.mozilla.org/mozilla-central/rev/72928fbc2e04

Comment 7

[Mark Finkle (:mfinkle) (use needinfo?)]

https://hg.mozilla.org/releases/mozilla-aurora/rev/e50a9d780b19
https://hg.mozilla.org/releases/mozilla-beta/rev/68d81380f84c

Comment 8

[Flaviu Cos]

Verified as fixed in builds:
-34.0a1 (2014-07-30);
-33.0a2 (2014-07-30);

Device: Asus Transformer Tab (Android 4.2.1).

Comment 9

[u421692]

Verified as fixed on Firefox 32 Beta 8 on LG Nexus 4(Android 4.4.4)