Closed
Bug 1181467
Opened 9 years ago
Closed 9 years ago
Can not copy/paste images to other applications
Categories
(Core :: DOM: Core & HTML, defect)
Core
DOM: Core & HTML
Tracking
()
RESOLVED
FIXED
mozilla42
Tracking | Status | |
---|---|---|
firefox42 | --- | fixed |
People
(Reporter: boris, Assigned: boris)
References
Details
Attachments
(1 file)
1.36 KB,
patch
|
enndeakin
:
review+
|
Details | Diff | Splinter Review |
Cannot copy image from the browser into other applications on my MacOS and Ubuntu(gtk) based on current nightly build. (I think windows also has this problem.)
Reason:
We set kNativeImageMime type whiling copying image on the browser
(https://dxr.mozilla.org/mozilla-central/source/dom/base/nsCopySupport.cpp#452)
, but ContentParent::RecvSetClipboard() didn't handle this case correctly, so nsClipboard::SetData() cannot get the imgIContainer successfully on gtk/cocoa/qt...
Assignee | ||
Updated•9 years ago
|
Assignee | ||
Updated•9 years ago
|
Assignee | ||
Updated•9 years ago
|
Component: DOM: Core & HTML → DOM
Assignee | ||
Updated•9 years ago
|
Summary: Fix mime type checking whiling copying image → Fix mime type checking while copying image
Assignee | ||
Comment 1•9 years ago
|
||
Handle kNativeImageMime case as other image types in RecvSetClipboard.
Assignee | ||
Updated•9 years ago
|
Attachment #8630908 -
Flags: review?(enndeakin)
Assignee | ||
Comment 2•9 years ago
|
||
Assignee | ||
Updated•9 years ago
|
Summary: Fix mime type checking while copying image → Can not copy/paste images onto other applications.
Assignee | ||
Updated•9 years ago
|
Summary: Can not copy/paste images onto other applications. → Can not copy/paste images to other applications.
Assignee | ||
Comment 3•9 years ago
|
||
Note:
Most platforms treat the transferable data of kNativeImageMime as imgIContainer.
1. cocoa:
https://dxr.mozilla.org/mozilla-central/source/widget/cocoa/nsClipboard.mm#448
2. gtk:
https://dxr.mozilla.org/mozilla-central/source/widget/gtk/nsClipboard.cpp#177
3. windows:
https://dxr.mozilla.org/mozilla-central/source/widget/windows/nsClipboard.cpp#210
4. qt:
https://dxr.mozilla.org/mozilla-central/source/widget/qt/nsClipboard.cpp#155
KNativeImageMime is moz internal type, so it is easy to be neglected :)
Assignee | ||
Updated•9 years ago
|
Summary: Can not copy/paste images to other applications. → Can not copy/paste images to other applications
Updated•9 years ago
|
Attachment #8630908 -
Flags: review?(enndeakin) → review+
Assignee | ||
Updated•9 years ago
|
Keywords: checkin-needed
Keywords: checkin-needed
Status: ASSIGNED → RESOLVED
Closed: 9 years ago
status-firefox42:
--- → fixed
Resolution: --- → FIXED
Target Milestone: --- → mozilla42
Updated•6 years ago
|
Component: DOM → DOM: Core & HTML
You need to log in
before you can comment on or make changes to this bug.
Description
•