599222 - Unresponsive script error in Sync.js during initial sync in Fennec

Status: RESOLVED WORKSFORME

(Firefox :: Sync, defect)

Description

[Matt Brubeck (:mbrubeck)]

Running a current opt build of Fennec on Android, I got two "Unresponsive script" dialogs for Sync.js, during the initial sync after connecting to my sync account.  mfinkle reported the seeing the same thing on a recent Fennec build.

Fennec sets dom.max_chrome_script_run_time to 30 seconds - we could increase this if necessary, but we should figure out the cause first.

Comment 1

[Naoki Hirata :nhirata (please use needinfo instead of cc)]

Attachment: Maemo 5 GTK Screenshot

Comment 2

[Dave Townsend [:mossop]]

I also see timeouts in Preferences.js and not just during the initial sync

Comment 3

[Philipp von Weitershausen [:philikon]]

I'd say that *where* the timeouts occur is rather arbitrary, but they would definitely be more likely during an initial sync since there's just a lot more going on there.

I don't know how a DOM script's run time is evaluated to be matched against 'dom.max_chrome_script_run_time' and how this evaluation copes with our funky Sync.js module (whose showing up in this bug is rather accidental I think).

Comment 4

[Aakash Desai [:aakashd]]

Doug tells me there's two bugs that will fix this issue. Which ones are they?

Comment 5

[Doug Turner (:dougt)]

two related bugs (not sure of the bug numbers):

1) logging needs to be disabled for mobile
2) get crypto off of the main thread

Comment 7

[Philipp von Weitershausen [:philikon]]

(In reply to comment #5)
> 1) logging needs to be disabled for mobile

Bug 599928

> 2) get crypto off of the main thread

Bug 570619

Comment 8

[Doug Turner (:dougt)]

This is really a meta bug.  I marked Bug 570619 as blocking b2.

Comment 10

[Aakash Desai [:aakashd]]

Can we close this bug now?

Comment 11

[Aakash Desai [:aakashd]]

Better question, is anyone still seeing this?

Comment 12

[Mike Connor [:mconnor]]

No reports in a long while, resolving WFM.  Other bugs will help with related stuff, but I think this specific case exists now.