We're changing some APIs around in bug 828829 that will require some modifications to the "data choices" preferences pane to make it work in the new world.

Current version of the patch. Holding off until the API is settled in bug 828829. glandium: please approve the configure.in change. Before, we had a number of places in XUL code where we effectively did an "if MOZ_TELEMETRY_REPORTING | MOZ_SERVICES_HEALTHREPORT | MOZ_CRASHREPORTER". I felt this was sufficiently repeated and would be creating engineering debt and potential for incomplete updates when this set was modified. So, I created a new AC_DEFINE for it. Unification FTW. Please bikeshed about naming. Essentially we're trying to capture "does this build of application X have the ability to automatically submit any metrics data to Mozilla?"

dolske gets /browser review. rnewman gets FHR specific behavior.

Comment on attachment 701280 [details] [diff] [review] Refactor pref pane, v2 Review of attachment 701280 [details] [diff] [review]: ----------------------------------------------------------------- r+ on these diffs. Not sure where some of this code comes from (it's not in m-c, nor bug 828829), but at long as it got reviewed somewhere that's fine. The changes here are basic.

I changed the open_preferences() helper to wait on "Initialized" otherwise there is a race condition on "load" between init_all() in the preferences pane and the test runner. Locally things passed just fine! Hopefully it holds on all platforms on TBPL.

Comment on attachment 701334 [details] [diff] [review] Refactor pref pane, v3 [Approval Request Comment] See approval request in bug 804745.

Comment on attachment 701334 [details] [diff] [review] Refactor pref pane, v3 https://bugzilla.mozilla.org/show_bug.cgi?id=804745#c37

Pushed to Aurora: http://hg.mozilla.org/releases/mozilla-aurora/rev/695dd2dc9fe8