1257488 - Fix test_offline_gzip.html to work on e10s

Status: RESOLVED FIXED

(Core :: Networking: Cache, defect)

Description

[Gabriele Svelto [:gsvelto]]

As per title.

Comment 1

[Gabriele Svelto [:gsvelto]]

Attachment: [PATCH] Fix test_gzip_offline.html to work with e10s enabled

This is my first attempt at fixing/rewriting a mochitest so I might have gotten some things wrong. It seems to work fine on both regular and e10s modes though so hopefully it's not too bad :)

Comment 2

[Honza Bambas (:mayhemer)]

Can you please describe more in detail what the patch exactly does and change?  Will help with review.

Comment 3

[Gabriele Svelto [:gsvelto]]

(In reply to Honza Bambas (:mayhemer) from comment #2)
> Can you please describe more in detail what the patch exactly does and
> change?  Will help with review.

Right, sorry about that. So the original test used the SpecialPowers API to grab hold of the browser window and allow the test to store local data by clicking on the appropriate button and by setting the offline-apps.allow_by_default preference. None of this action are allowed in e10s mode (IIRC they would happen in the content process while they need to happen in the parent one) and neither is the permission management done in the cleanup procedure. So I've modified the test to be a mochitest-browser test and split the operations appropriately between the parent and the content processes. Note that I've removed the part where we click the "Allow" dialog button because setting the offline-apps.allow_by_default pref makes it redundant.

Comment 4

[Honza Bambas (:mayhemer)]

Comment on attachment 8731672 [details] [diff] [review]
[PATCH] Fix test_gzip_offline.html to work with e10s enabled

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

My r+ depends on a green try run (I don't see any in the bug..)

Comment 5

[:Gijs (he/him)]

I just backed out my patch for bug 621158 because of this test, so if they land together they will likely work. I can do a trypush tomorrow if nobody has beaten me to it. :-)

Comment 6

[Gabriele Svelto [:gsvelto]]

Thanks Honza, the try run is here: https://treeherder.mozilla.org/#/jobs?repo=try&revision=76890c0b8394

I'll land as soon as it turns green.

Comment 7

[Pulsebot]

https://hg.mozilla.org/integration/fx-team/rev/116497ecbdab

Comment 8

[Carsten Book [:Tomcat]]

https://hg.mozilla.org/mozilla-central/rev/116497ecbdab