1

Start LoginDetectionService on Android

 2 years ago
source link: https://bugzilla.mozilla.org/show_bug.cgi?id=1741610
Go to the source link to view the article. You can view the picture content, updated content and better typesetting reading experience. If the link is broken, please click the button below to view the snapshot at that time.
Closed Bug 1741610 Opened 2 months ago Closed 1 month ago

Start LoginDetectionService on Android

Categories

(Toolkit :: Password Manager, task, P1)

Toolkit ▾
Password Manager ▾

Unspecified

Android

Tracking

(bug RESOLVED as FIXED)

RESOLVED FIXED

97 Branch

Tracking Status firefox-esr91 --- wontfix firefox95 --- wontfix firefox96 --- wontfix firefox97 --- fixed

People

(Reporter: farre, Assigned: farre)

References

(Blocks 1 open bug)

Details

(Whiteboard: [fission:android:m3])

No description provided.
Assignee: nobody → afarre
Status: NEW → ASSIGNED
Severity: -- → N/A
Priority: -- → P1
Whiteboard: [fission:android:m3]

There's a r+ patch which didn't land and no activity in this bug for 2 weeks.
:farre, could you have a look please?
For more information, please visit auto_nag documentation.

Flags: needinfo?(dlee)
Flags: needinfo?(afarre)

ah, I guess :farre is on leave and hasn't had the chance to deal with the patch. I'll land this patch.

Flags: needinfo?(dlee)

Backed out for causing assertion failures on Android

Backout link: https://hg.mozilla.org/integration/autoland/rev/b5acafe1af063fcd72dacc96ff35a6d1fa1e4326

Push with failures: https://treeherder.mozilla.org/jobs?repo=autoland&selectedTaskRun=b6FHnSqlTr2vUjsbniIdxw.0&resultStatus=testfailed%2Cbusted%2Cexception%2Cretry%2Cusercancel&revision=ba1330ed02c7e581b329b0d708029df4297882c4

Failure log: https://treeherder.mozilla.org/logviewer?job_id=361341137&repo=autoland&lineNumber=1641 // https://treeherder.mozilla.org/logviewer?job_id=361341546&repo=autoland&lineNumber=7228 // https://treeherder.mozilla.org/logviewer?job_id=361349054&repo=autoland&lineNumber=1806

Failure lines: TEST-UNEXPECTED-FAIL | /fetch/api/response/json.any.worker.html | assertion count 1 is more than expected 0 assertions //

org.mozilla.geckoview.test.MediaDelegateTest#testDeviceRecordingEventAudioAndVideo | org.mozilla.geckoview.test.rule.GeckoSessionTestRule$RejectedPromiseException...

PROCESS-CRASH | netwerk/test/unit_ipc/test_cache_jar_wrap.js | application crashed [@ nsAppStartupNotifier::NotifyObservers(char const*)]

Flags: needinfo?(dlee)

I'm back from leave and will be looking into this promptly.

Flags: needinfo?(afarre)

This was easy enough. You can't call EventDispatcher.instance.registerListener from a content process. I'll move the registration down to where we check process type. That should do it. I'll run try and try to land again.

Status: ASSIGNED → RESOLVED
Closed: 1 month ago
Resolution: --- → FIXED
Target Milestone: --- → 97 Branch
Flags: needinfo?(dlee)
You need to log in before you can comment on or make changes to this bug.

report

Recommend

About Joyk


Aggregate valuable and interesting links.
Joyk means Joy of geeK