copy exif on share image

RESOLVED FIXED in 2.6 S7 - 2/12

Status

RESOLVED FIXED
3 years ago
3 years ago

People

(Reporter: hub, Assigned: hub)

Tracking

unspecified
2.6 S7 - 2/12
ARM
Gonk (Firefox OS)

Firefox Tracking Flags

(Not tracked)

Details

Attachments

(1 attachment)

(Assignee)

Description

3 years ago
Follow-up to bug 1178812 : we need to copy Exif for the share action. See frames.js:shareSingleItem()
(Assignee)

Updated

3 years ago
Depends on: 1178812
(Assignee)

Updated

3 years ago
Assignee: nobody → hub
Created attachment 8705807 [details] [review]
[gaia] hfiguiere:bug1237367-exif-copy-share > mozilla-b2g:master
(Assignee)

Updated

3 years ago
Attachment #8705807 - Flags: review?(dflanagan)
(Assignee)

Comment 2

3 years ago
Comment on attachment 8705807 [details] [review]
[gaia] hfiguiere:bug1237367-exif-copy-share > mozilla-b2g:master

Punam, if you don't mind reviewing this. Thanks !
Attachment #8705807 - Flags: review?(dflanagan) → review?(pdahiya)
Hi Hub

I am following below steps to test exif copy on single file share action.

1. Share file in gallery with messaging app

2. Send the message with the shared file.

3. Open the shared message and click on the attachment and save the attached image.

4. Saved image should show up in gallery

With the patch, I was expecting to see EXIF info with the saved image but I don't see it. Am I missing something?

I am using https://github.com/recurser/exif-orientation-examples/blob/master/Portrait_6.jpg to test the above flow. Thanks!
Flags: needinfo?(hub)
Comment on attachment 8705807 [details] [review]
[gaia] hfiguiere:bug1237367-exif-copy-share > mozilla-b2g:master

Issue reported in comment #3 resolved for me with re-flash of flame. EXIF data is seen on shared image. Thanks!
Attachment #8705807 - Flags: review?(pdahiya) → review+
(Assignee)

Comment 5

3 years ago
Merged
https://github.com/mozilla-b2g/gaia/commit/aee2763847b1b289f4876b04de78f06ee31e22c5
Status: NEW → RESOLVED
Last Resolved: 3 years ago
Flags: needinfo?(hub)
Resolution: --- → FIXED
(Assignee)

Updated

3 years ago
Target Milestone: --- → 2.6 S7 - 2/12
You need to log in before you can comment on or make changes to this bug.