Closed Bug 1705271 Opened 4 years ago Closed 2 years ago

Remove PushServiceAndroidGCM.jsm

Categories

(Core :: DOM: Notifications, task, P3)

Unspecified
Android
task

Tracking

()

RESOLVED FIXED
108 Branch
Tracking Status
firefox-esr102 --- wontfix
firefox106 --- wontfix
firefox107 --- wontfix
firefox108 --- fixed

People

(Reporter: m_kato, Assigned: m_kato)

References

(Blocks 1 open bug)

Details

Attachments

(1 file)

PushServiceAndroidGCM.jsm is for Fennec only and GeckoView has another implementations for Web Push as GeckoViewPush.jsm.

So PushServiceAndroidGCM.jsm is unused now.

Shouldn't we consider this to be more a task than a defect?

Flags: needinfo?(m_kato)
Severity: -- → S4
Type: defect → task
Priority: -- → P3

(In reply to Jens Stutte [:jstutte] from comment #1)

Shouldn't we consider this to be more a task than a defect?

Although asuth already changes to it, this should be task

Flags: needinfo?(m_kato)

PushServiceAndroidGCM is Fennec only module and GeckoView uses
GeckoViewPush.jsm for push service. So we should get rid of
PushServiceAndroidGCM.

Assignee: nobody → m_kato
Status: NEW → ASSIGNED
Pushed by m_kato@ga2.so-net.ne.jp: https://hg.mozilla.org/integration/autoland/rev/7e777801b7cd Get rid of PushServiceAndroidGCM.sys.mjs. r=saschanaz
Status: ASSIGNED → RESOLVED
Closed: 2 years ago
Resolution: --- → FIXED
Target Milestone: --- → 108 Branch
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Created:
Updated:
Size: