Closed Bug 1200843 Opened 9 years ago Closed 9 years ago

Make the mixed-content tests pass

Categories

(Core :: DOM: Service Workers, defect)

defect
Not set
normal

Tracking

()

RESOLVED FIXED
mozilla43
Tracking Status
firefox43 --- fixed

People

(Reporter: khuey, Assigned: khuey)

References

Details

Attachments

(1 file, 1 obsolete file)

Attached patch 0008-mixed-content-tests.patch (obsolete) — Splinter Review
      No description provided.
Attachment #8655665 - Flags: review?(josh)
Consider the console.logs I scattered removed.
Comment on attachment 8655671 [details] [diff] [review]
0008-Bug-1200843-Make-fetch-mixed-content-tests-pass.-r-j.patch

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

::: testing/web-platform/mozilla/tests/service-workers/service-worker/resources/fetch-mixed-content-iframe.html
@@ +35,5 @@
>  function on_state_change(event) {
>    if (event.target.state != 'activated')
>      return;
>    var frame = document.createElement('iframe');
> +  console.log("loading frame " + SCOPE);

Remove this?
Attachment #8655671 - Flags: review?(josh) → review+
https://hg.mozilla.org/mozilla-central/rev/e9bce7012a9d
Status: ASSIGNED → RESOLVED
Closed: 9 years ago
Resolution: --- → FIXED
Target Milestone: --- → mozilla43
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Created:
Updated:
Size: