1246661 - Enable test_clipboard_events.html in e10s

Status: RESOLVED FIXED

(Core :: DOM: Core & HTML, defect)

Description

[Neil Deakin]

Attachment: test_clipboard_events

Comment 1

[Blake Kaplan (:mrbkap) (inactive)]

Comment on attachment 8716976 [details] [diff] [review]
test_clipboard_events

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

I have a couple of questions/comments, but r=me if you don't want to do them.

::: dom/tests/mochitest/general/test_clipboard_events.html
@@ +56,5 @@
> +  test_input_copy_button_dataTransfer,
> +  test_eventspref_disabled
> +  ];
> +
> +function doTests()

I don't know if it's worth it for you, but I've been using SpawnTask.js and spawn_task(generator_function) to avoid re-implementing this wheel. That would also mean that you could do:

function* doTests() {
  // init

  for (test of testFunctions) {
    yield test();
  }

  // cleanup.
}

which is cleaner.

@@ +525,5 @@
>  
>  function test_eventspref_disabled() {
>    // Disable clipboard events
> +  return new Promise(resolve => {
> +    SpecialPowers.pushPrefEnv({"set": [['dom.event.clipboardevents.enabled', false]]}, doPrefDisabledTest);

I wonder if we should add some promise-based pref functions to SpecialPowers.

Comment 2

[Andrew McCreight [:mccr8]]

Are this and bug 1246662 ready to land?

Comment 3

[Neil Deakin]

https://hg.mozilla.org/integration/mozilla-inbound/rev/2338e962df6da35910a112d0a0f6d2645a86bc78
Bug 1246661, enable test_clipboard_events.html in e10s, r=mrbkap

Comment 4

[Carsten Book [:Tomcat]]

https://hg.mozilla.org/mozilla-central/rev/2338e962df6d