Last Comment Bug 772968 - Specify a different APP_ID for fennec webapp runtime
: Specify a different APP_ID for fennec webapp runtime
Product: Firefox for Android
Classification: Client Software
Component: Web Apps (show other bugs)
: unspecified
: ARM Android
P1 normal (vote)
: Firefox 17
Assigned To: Nobody; OK to take it and work on it
: Aaron Train [:aaronmt]
: Myk Melez [:myk] [@mykmelez]
Depends on:
Blocks: 780972
  Show dependency treegraph
Reported: 2012-07-11 11:39 PDT by Pedro Alves
Modified: 2012-08-09 15:26 PDT (History)
16 users (show)
See Also:
Crash Signature:
QA Whiteboard:
Iteration: ---
Points: ---
Has Regression Range: ---
Has STR: ---

Patch (1.01 KB, patch)
2012-07-24 17:11 PDT, Wesley Johnston (:wesj)
mark.finkle: review+
lukasblakk+bugs: approval‑mozilla‑aurora+
Details | Diff | Splinter Review

Description User image Pedro Alves 2012-07-11 11:39:11 PDT
Currently webapp runtime on fennec is indistinguishable from fennec, as it uses the same APP_ID.

Would be useful to give it a different one. Desktop webapp rt uses ''. Something like '' would be nice (or whatever unique app_id)
Comment 1 User image Myk Melez [:myk] [@mykmelez] 2012-07-11 14:30:29 PDT
(Minor correction: the desktop runtime actually uses the app ID

cc:ing some of the Fennec folks who are working on the Android runtime.

Note that we discussed whether or not to distinguish runtime reports from Firefox reports on desktop, and we ultimately decided to distinguish them, even if some (perhaps most) will have identical root causes, because others will not, and the relative frequencies of even identical crashes will vary because of the different usage patterns of apps vs. the web as a whole (so topcrashes will differ).

Changing our minds about that is relatively easy, provided the runtime has a different app ID, because we can reprocess reports and munge products together (or vice-versa).  But the runtime does need to have a different app ID for us to have the option to distinguish its reports, hence this bug.
Comment 2 User image Mark Finkle (:mfinkle) (use needinfo?) 2012-07-12 09:17:39 PDT
As long as Fennec _is_ the webapp runtime on Android, we can't change the core APP ID.

We might be able to play with the URL sent to the blocklist (or whatever URL metrics uses) to substitute a different APP ID.
Comment 3 User image Pedro Alves 2012-07-12 09:20:40 PDT
From the metrics end, changing the app_id to something unique or changing the blocklist ping to something like:<insert_unique_stuff_here>/%APP_VERSION%/%PRODUCT%/%BUILD_ID%/%BUILD_TARGET%/%LOCALE%/%CHANNEL%/%OS_VERSION%/%DISTRIBUTION%/%DISTRIBUTION_VERSION%/%PING_COUNT%/%TOTAL_PING_COUNT%/%DAYS_SINCE_LAST_PING%/

would be exactly the same

(is url is a modification of the one extracted from firefox desktop: , I'm assuming fennec is the same)
Comment 4 User image Wesley Johnston (:wesj) 2012-07-24 17:11:53 PDT
Created attachment 645596 [details] [diff] [review]

Sending this review to mfinkle because I want to make sure he's ok with it.
Comment 5 User image Mark Finkle (:mfinkle) (use needinfo?) 2012-07-28 17:08:48 PDT
Comment on attachment 645596 [details] [diff] [review]

This method should work OK. I worry that "mobile" is too general. This is the webapprt for Android only. Other mobile platforms will have different runtimes.

Maybe I am over thinking things.

nit: '' -> ""

* add the '-'
* 'com" -> 'org'
* use double quotes
Comment 6 User image Robert Kaiser 2012-07-29 15:13:02 PDT
That would mean that we'd also need to add both an ADI mechanism for the mobile webapp runtime in separate, just as we did for desktop, and that we'd also need to add it as a separate product in Socorro for monitoring crashes.
Comment 8 User image Wesley Johnston (:wesj) 2012-08-02 09:58:38 PDT
Comment on attachment 645596 [details] [diff] [review]

[Approval Request Comment]
Bug caused by (feature/regressing bug #): No regression
User impact if declined: none
Testing completed (on m-c, etc.): landed on mc today 8/3
Risk to taking this patch (and alternatives if risky): very low risk. Just changes a pref on webapps. needed to push webapps in 16
String or UUID changes made by this patch: None.
Comment 9 User image [:jberkus] Josh Berkus 2012-08-02 10:25:46 PDT
So has this been changed in crash reporting?

And if so, what AppID is it reporting?

Daniel, will metrics have any ADU for Fennec WebRT?
Comment 10 User image Jason Smith [:jsmith] 2012-08-02 10:28:20 PDT
*** Bug 764955 has been marked as a duplicate of this bug. ***
Comment 11 User image Wesley Johnston (:wesj) 2012-08-02 10:29:35 PDT
I'm not familiar with the crash reporter code. An mxr search showed %APP_ID% only being used for addon updates (addons are disabled in webapps) and in the addon blocklist. I'll have to dig more to know for sure that one of them is being used for crash reporting somehow (unless you can point me to some code?)
Comment 12 User image Daniel Einspanjer [:dre] [:deinspanjer] 2012-08-02 11:17:50 PDT
s/ADU/ADI.  ADI is Mozilla's measure of Active Daily Installs.  Currently, this measurement is derived from the add-on blocklist check requests that come in to  The Crash Stats team (Socorro) uses the ADI to calculate the ratio of crash reports to install-base.

If the Fennec WebappRT doesn't make a blocklist check, we will not have any visibility into the number of active daily installs for it.

If this is the case, we need to escalate this issue and figure out quickly how to fix or work around it.
Comment 13 User image Robert Kaiser 2012-08-02 11:19:23 PDT
(In reply to Wesley Johnston (:wesj) from comment #11)
> I'm not familiar with the crash reporter code.

Bug 745980 apparently implemented crash reporting for the desktop WebRT, I guess that code can help.
Comment 14 User image Wesley Johnston (:wesj) 2012-08-02 13:02:48 PDT
(In reply to Daniel Einspanjer :dre [:deinspanjer] from comment #12)
> If the Fennec WebappRT doesn't make a blocklist check, we will not have any
> visibility into the number of active daily installs for it.

I need to double check that disabling addons doesn't disable the blocklist check, but we're currently doing the same thing that desktop does, so I imagine we're fine.
Comment 15 User image Pedro Alves 2012-08-02 13:12:59 PDT
I can confirm that we are getting the correct information from desktop
Comment 16 User image mroldan 2012-08-02 13:36:25 PDT
We gave the name 'WebappRuntime' to the desktop webapprt ('')

So we're going to give the name 'WebappRuntime Mobile' to this new one ('') unless someone has a better suggestion.

Comment 17 User image Pedro Alves 2012-08-02 13:46:50 PDT
Remove the whitespace, please. It's easier for the socorro team
Comment 18 User image Ryan VanderMeulen [:RyanVM] 2012-08-02 19:08:50 PDT
Comment 19 User image Lukas Blakk [:lsblakk] use ?needinfo 2012-08-03 12:04:42 PDT
Comment on attachment 645596 [details] [diff] [review]

Just for WebRT so shouldn't affect Fennec itself - approving for Aurora.
Comment 20 User image Wesley Johnston (:wesj) 2012-08-09 15:25:51 PDT

Note You need to log in before you can comment on or make changes to this bug.