Closed Bug 1340556 Opened 7 years ago Closed 7 years ago

Add tests for clipboardRead from the background page

Categories

(WebExtensions :: Compatibility, defect)

defect
Not set
normal

Tracking

(firefox54 fixed)

RESOLVED FIXED
mozilla54
Tracking Status
firefox54 --- fixed

People

(Reporter: zombie, Assigned: zombie)

Details

Attachments

(1 file)

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.
Assignee: nobody → tomica
Status: NEW → ASSIGNED
Summary: Add test that reading the clipboard works from the background page → Add/enable tests for clipboard API from the background page
Attachment #8839729 - Flags: review?(mixedpuppy)
Turns out clipboardWrite works only with OOP extensions, I didn't run the full suite (bug 1272869).
Summary: Add/enable tests for clipboard API from the background page → Add tests for clipboardRead from the background page
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?
Attachment #8839729 - Flags: review?(mixedpuppy) → review+
Keywords: checkin-needed
Pushed by ryanvm@gmail.com:
https://hg.mozilla.org/integration/autoland/rev/2f65dd833a4a
Add tests for clipboardRead from the background page r=mixedpuppy
Keywords: checkin-needed
https://hg.mozilla.org/mozilla-central/rev/2f65dd833a4a
Status: ASSIGNED → RESOLVED
Closed: 7 years ago
Resolution: --- → FIXED
Target Milestone: --- → mozilla54
Product: Toolkit → WebExtensions
You need to log in before you can comment on or make changes to this bug.