Closed Bug 1346208 Opened 5 years ago Closed 5 years ago

Remove B2G-specific code from Telemetry C++ modules


(Toolkit :: Telemetry, enhancement, P4)




Tracking Status
firefox55 --- fixed


(Reporter: gfritzsche, Assigned: clintonfeng, Mentored)


(Whiteboard: [measurement:client] [lang=c++] [good first bug])


(1 file, 2 obsolete files)

B2G / Firefox OS / Gonk is not a supported platform anymore.

We should remove the B2G/Gonk logic from the Telemetry C++ files:
Im pretty sure I have done what you have asked. What I did is I removed "!defined(MOZ_WIDGET_GONK) &&" from the code which hopefully is right. Im currently trying to figure out how to submit a patch.
Attached patch TelemetryHistogram.cpp (obsolete) — Splinter Review
Attached patch Telemetry.cpp (obsolete) — Splinter Review
Hi Clinton,
it looks like you ended up submitting the full source file, not a patch.

See here for documentation on submitting patches:

Before asking for feedback, please make sure that, with your changes:
- the build succeeds: mach build
- the Telemetry unit tests run fine: mach test toolkit/components/telemetry/tests/unit
Assignee: nobody → clintonfeng
The build works and the tests run fine on my machine. I submitted a patch via mozreview and hopefully I have done it right this time.
Attachment #8846280 - Attachment is obsolete: true
Attachment #8846279 - Attachment is obsolete: true
Comment on attachment 8846369 [details]
Bug 1346208 - Remove B2G-specific code from Telemetry C++ modules

Thanks, this looks good!
I'll push it to our try server (dry run of our CI test runs) to make sure this is fine across platforms.
Attachment #8846369 - Flags: review?(gfritzsche) → review+
Pushed by
Remove B2G-specific code from Telemetry C++ modules r=gfritzsche
Closed: 5 years ago
Resolution: --- → FIXED
Target Milestone: --- → mozilla55
You need to log in before you can comment on or make changes to this bug.