Open Bug 1901798 (necko-ifndef-removal) Opened 2 years ago Updated 1 year ago

Remove #ifndef ANDROID blocks after the ongoing glean performance impact on android gets addressed

Categories

(Core :: Networking, task, P2)

Product:
Core
Component:
Networking
Type:
task

Tracking

()

People

(Reporter: sekim, Unassigned)

References

(Blocks 1 open bug,
URL
)

Details

(Keywords: leave-open, Whiteboard: [necko-triaged][necko-monitor])

Some #ifndef ANDROID (#[cfg(not(target_os = "android"))] in Rust) blocks have been added to prevent ongoing performance impacts on Android. Once this issue is addressed, we will need to remove these blocks to enable telemetry on Android.

The task is to simply remove

#ifndef ANDROID
...
#endif

blocks that the "Depends on" patches introduced.

Note that this bug should not be addressed until the ongoing Glean issues are resolved.

Blocks: necko-glean, necko-tech-debt
Keywords: leave-open
Depends on: 1897290, 1898018, 1898245, 1898248
Severity: -- → N/A
Depends on: 1851437
Depends on: 1898737
No longer depends on: 1898248
Priority: -- → P2
Whiteboard: [necko-triaged][necko-priority-monitor]
Depends on: 1813630
Depends on: 1905939
Depends on: 1898247
Depends on: 1907418
Depends on: 1908994
No longer depends on: 1851437
Depends on: 1917778
Depends on: 1906853
Whiteboard: [necko-triaged][necko-priority-monitor] → [necko-triaged][necko-monitor]
Depends on: 1919391
Alias: necko-ifndef-removal
No longer blocks: necko-glean
Depends on: 1926258
Depends on: 1906360
Depends on: 1926663
Depends on: 1922661
Depends on: 1909931
No longer depends on: 1909931
No longer depends on: 1926663

The leave-open keyword is there and there is no activity for 6 months.
:jesup, maybe it's time to close this bug?
For more information, please visit BugBot documentation.

Flags: needinfo?(rjesup)
Flags: needinfo?(rjesup)
You need to log in before you can comment on or make changes to this bug.