1222238 - Hang in preferences pane on OS X

Status: RESOLVED FIXED

(Instantbird Graveyard :: Preferences, defect)

Description

[arlolra]

User Agent: Mozilla/5.0 (Macintosh; Intel Mac OS X 10_11_1) AppleWebKit/537.36 (KHTML, like Gecko) Chrome/45.0.2454.85 Safari/537.36

Steps to reproduce:

Open prefs.
Click the Themes tab.
Click Advanced tab.
Open Network > Settings.


Actual results:

Hangs.


Expected results:

Not hangs.

Comment 1

[arlolra]

This was introduced in Bug 1179943, the +1 there seems to be the cause.

Comment 2

[aleth [:aleth]]

Attachment: Avoid hang in proxy preferences pane

Comment 3

[arlolra]

Should you update viewlog.js with that as well?

Comment 4

[aleth [:aleth]]

Comment on attachment 8684473 [details] [diff] [review]
Avoid hang in proxy preferences pane

Review of attachment 8684473 [details] [diff] [review]:
-----------------------------------------------------------------

The hang seems to happen due to the getPendingMessages override (it goes away if the override is deleted after all the messages have been displayed). No idea what exactly causes the hang though.

This patch is a very slightly less hacky fix for bug 1179943 that avoids the problem.

Please check if this fixes it for you too (along with the original bug).

Comment 5

[aleth [:aleth]]

(In reply to arlolra from comment #3)
> Should you update viewlog.js with that as well?

The code there isn't the same. So unless you can produce a hang there too somehow, I don't think we should worry...

Comment 6

[arlolra]

Comment on attachment 8684473 [details] [diff] [review]
Avoid hang in proxy preferences pane

Review of attachment 8684473 [details] [diff] [review]:
-----------------------------------------------------------------

Yup, fixes the original bug and doesn't hang. Thanks!

Comment 7

[aleth [:aleth]]

https://hg.mozilla.org/comm-central/rev/25266a5c96bfaffb7084a36c91e3002c4f233309
Bug 1222238 - Avoid hang in proxy preferences pane. r=clokep a=aleth