Make test_bug523771.html work with e10s

RESOLVED FIXED in Firefox 43

Status

()

RESOLVED FIXED
3 years ago
3 years ago

People

(Reporter: mrbkap, Assigned: mrbkap)

Tracking

(Blocks: 1 bug)

unspecified
mozilla43
Points:
---
Bug Flags:
in-testsuite +

Firefox Tracking Flags

(e10s+, firefox43 fixed)

Details

Attachments

(1 attachment)

(Assignee)

Description

3 years ago
This test attempts to create files in the profile directory, which is a no-no in e10s. Fortunately, we can easily use DOM files backed by DOM strings to fix the test.
(Assignee)

Comment 1

3 years ago
Created attachment 8648286 [details] [diff] [review]
Patch v1
Attachment #8648286 - Flags: review?(jld)
Comment on attachment 8648286 [details] [diff] [review]
Patch v1

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

r+ with a couple comments.

Nit: maybe the commit message could be a little more specific?

::: dom/html/test/test_bug523771.html
@@ +32,5 @@
>  SimpleTest.waitForExplicitFinish();
>  
>  function setFileInputs () {
> +  var f = createFileWithData(input1File.name, input1File.body, input1File.type);
> +  SpecialPowers.wrap(singleFileInput).mozSetFileArray([f], 1);

mozSetFileArray just takes a sequence<File>; it doesn't need the length passed separately.  (And I think mozSetFileNameArray doesn't need it either, these days; it's in the XPIDL but not the WebIDL.)

@@ +42,2 @@
>    }
> +  SpecialPowers.wrap(multiFileInput).mozSetFileArray(input2FileNames, input2FileNames.length);

And here.
Attachment #8648286 - Flags: review?(jld) → review+
https://hg.mozilla.org/mozilla-central/rev/4dc348b4f780
Status: NEW → RESOLVED
Last Resolved: 3 years ago
status-firefox43: --- → fixed
Flags: in-testsuite+
Resolution: --- → FIXED
Target Milestone: --- → mozilla43
You need to log in before you can comment on or make changes to this bug.