1533426 - browser/components/preferences/in-content/tests/siteData/browser_siteData2.js fails with LSNG enabled

Status: RESOLVED FIXED

(Core :: Storage: Quota Manager, defect)

Description

[Jan Varga [:janv]]

The test simulates user's interaction with pref UI and once OK is hit the clearing starts to happen, but the test checks usage on disk immediately without waiting for clearing to finish.

Comment 1

[Jan Varga [:janv]]

Attachment: race-fix

Comment 2

[Johann Hofmann [:johannh]]

Comment on attachment 9049225 [details] [diff] [review]
race-fix

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

This is okay for me to unblock LSNG landing, but please file a follow-up (or leave this bug open) to actually fix the test issues.

Comment 3

[Pulsebot]

Pushed by [e-mail]:
https://hg.mozilla.org/integration/mozilla-inbound/rev/9dde4b9b46e2
browser/components/preferences/in-content/tests/siteData/browser_siteData2.js fails with LSNG enabled; r=johannh

Comment 4

[Arthur Iakab [arthur_iakab]]

https://hg.mozilla.org/mozilla-central/rev/9dde4b9b46e2

Comment 5

[Johann Hofmann [:johannh]]

(In reply to Johann Hofmann [:johannh] from comment #2)

Comment on attachment 9049225 [details] [diff] [review]
race-fix

Review of attachment 9049225 [details] [diff] [review]:

This is okay for me to unblock LSNG landing, but please file a follow-up (or
leave this bug open) to actually fix the test issues.

Was this done?

Comment 6

[Jan Varga [:janv]]

Tom Tung is going to take a look at similar stuff in bug 1533681. He might investigate this bug after that.