Closed
Bug 1155352
Opened 9 years ago
Closed 9 years ago
[e10s] Fix browser_middleMouse_noJSPaste.js
Categories
(Firefox :: General, defect)
Firefox
General
Tracking
()
People
(Reporter: enndeakin, Assigned: enndeakin)
References
Details
Attachments
(1 file)
4.43 KB,
patch
|
Gijs
:
review+
|
Details | Diff | Splinter Review |
No description provided.
Assignee | ||
Updated•9 years ago
|
Iteration: --- → 40.2 - 27 Apr
Points: --- → 1
Assignee | ||
Updated•9 years ago
|
Attachment #8593554 -
Flags: review?(gijskruitbosch+bugs)
Comment 1•9 years ago
|
||
Comment on attachment 8593554 [details] [diff] [review] Fix and re-enable browser_middleMouse_noJSPaste.js Review of attachment 8593554 [details] [diff] [review]: ----------------------------------------------------------------- r=me assuming green try ::: browser/base/content/test/general/browser_middleMouse_noJSPaste.js @@ +14,5 @@ > + SimpleTest.waitForClipboard(url, () => { > + Components.classes["@mozilla.org/widget/clipboardhelper;1"] > + .getService(Components.interfaces.nsIClipboardHelper) > + .copyString(url, document); > + }, resolve); Can we add reject as another callback here for the timed out / failed case (and an ok(false, ...) with a message to clarify what happened) ?
Attachment #8593554 -
Flags: review?(gijskruitbosch+bugs) → review+
Updated•9 years ago
|
tracking-e10s:
--- → +
https://hg.mozilla.org/mozilla-central/rev/fa9178ca5dc2
Status: ASSIGNED → RESOLVED
Closed: 9 years ago
Resolution: --- → FIXED
Target Milestone: --- → Firefox 40
You need to log in
before you can comment on or make changes to this bug.
Description
•