1340556 - Add tests for clipboardRead from the background page

Status: RESOLVED FIXED

(WebExtensions :: Compatibility, defect)

Description

[Tomislav Jovanovic :zombie]

Followup from bug 1312260.

I believe this is the most common way to use the clipboardRead permission, but I missed the test while fixing bug 1312260 because of the disabled test for clipboardWrite from the background page.

Comment 1

[Tomislav Jovanovic :zombie]

Attachment: Bug 1340556 - Add tests for clipboardRead from the background page

Review commit: https://reviewboard.mozilla.org/r/114308/diff/#index_header
See other reviews: https://reviewboard.mozilla.org/r/114308/

Comment 2

[Tomislav Jovanovic :zombie]

Turns out clipboardWrite works only with OOP extensions, I didn't run the full suite (bug 1272869).

Comment 3

[Shane Caraveo (:mixedpuppy)]

Comment on attachment 8839729 [details]
Bug 1340556 - Add tests for clipboardRead from the background page

https://reviewboard.mozilla.org/r/114308/#review115758

::: toolkit/components/extensions/test/mochitest/test_ext_clipboard.html:53
(Diff revision 1)
>    yield extension.awaitMessage("ready");
>  
>    yield extension.unload();
>  });
>  
>  /** Selecting text in a bg page is not possible, skip test until it's fixed.

I know it's unrelated, but can you dig up a bug # for this comment and add it?

Comment 4

[Tomislav Jovanovic :zombie]

Comment on attachment 8839729 [details]
Bug 1340556 - Add tests for clipboardRead from the background page

Review request updated; see interdiff: https://reviewboard.mozilla.org/r/114308/diff/1-2/

Comment 5

[Pulsebot]

Pushed by [e-mail]:
https://hg.mozilla.org/integration/autoland/rev/2f65dd833a4a
Add tests for clipboardRead from the background page r=mixedpuppy

Comment 6

[Wes Kocher (:KWierso) (Not reading bugmail; email directly if needed)]

https://hg.mozilla.org/mozilla-central/rev/2f65dd833a4a