Closed
Bug 1051774
Opened 10 years ago
Closed 10 years ago
Image position metadata isn't deactivated
Categories
(Firefox OS Graveyard :: Gaia::Camera, defect)
Tracking
(b2g-v1.4 affected, b2g-v2.0 affected, b2g-v2.0M wontfix, b2g-v2.1 affected, b2g-v2.2 fixed)
RESOLVED
FIXED
2.1 S4 (12sep)
People
(Reporter: markus, Assigned: mikeh)
References
Details
Attachments
(1 file, 1 obsolete file)
3.34 KB,
patch
|
aosmond
:
review+
|
Details | Diff | Splinter Review |
User Agent: Mozilla/5.0 (X11; Ubuntu; Linux x86_64; rv:31.0) Gecko/20100101 Firefox/31.0 (Beta/Release)
Build ID: 20140715215003
Steps to reproduce:
Took a picture with Position defined in CameraPictureOptions, then took another with no Position.
Actual results:
Both pictures got the same position metadata.
Expected results:
There shouldn't have been any postion metadata in the second picture.
Reporter | ||
Comment 1•10 years ago
|
||
The attached code in the pull request removes previously saved position if the position data is NaN when TakePicture is called.
Please review.
Attachment #8470825 -
Flags: review?(mhabicher)
Assignee | ||
Updated•10 years ago
|
Assignee: nobody → markus.nilsson
Assignee | ||
Updated•10 years ago
|
Status: UNCONFIRMED → ASSIGNED
Ever confirmed: true
Assignee | ||
Comment 2•10 years ago
|
||
Comment on attachment 8470825 [details] [review]
Github patch
This looks good, Markus, but it needs an automated test. You can either add it to one of the suitable existing tests, or create a new one (e.g. dom/camera/test/test_bug1051774.html).
Also, we don't take Gecko patches through Github. For a patch like this, it's not a big deal--we can easily extract the patch and submit it to Mercurial. But in the future, please refer to https://developer.mozilla.org/en-US/docs/Creating_a_patch_that_can_be_checked_in
Let me know if you need help with any of this.
Attachment #8470825 -
Flags: review?(mhabicher) → review-
Comment 3•10 years ago
|
||
Markus,
Thanks so much for filing this bug and contributing to this fix. Please address Mike's review comments and submit the patch for review again.
Thanks
Hema
Reporter | ||
Comment 4•10 years ago
|
||
Mike & Hema,
I haven't run mochi-tests before, so it takes a while for me to figure it out. Hopefully I'll have a new patch ready soon.
BR,
Markus
Reporter | ||
Comment 5•10 years ago
|
||
I can't figure out a way to create a test for this :(
If I've understood the test environment correctly, you can't run the camera tests with mochitest-plain since CONFIG['MOZ_WIDGET_TOOLKIT'] == 'gonk' needs to be true, so you'll have to run the tests on the emulator with mochitest-remote.
I modified http://www.nihilogic.dk/labs/exif/exif.js so that it could extract exif tags from a blob and then modified the mozCamera.takePicture() success function in gaia/apps/camera/js/lib/camera/camera.js to print them.
That worked fine on a device but when I added mochitests that did the same thing, the exif extraction failed. I then tried the modified camera app on the emulator and there it fails as well.
I then extracted one of the pictures from the emulator and as far as I can tell, the picture doesn't have any exif meta data; "exiftool -exif:all IMG_0001.jpg" returns nothing.
Any ideas on how to proceed?
Flags: needinfo?(mhabicher)
Assignee | ||
Comment 6•10 years ago
|
||
Yoink.
Assignee: markus.nilsson → mhabicher
Attachment #8470825 -
Attachment is obsolete: true
Attachment #8484224 -
Flags: review?(aosmond)
Flags: needinfo?(mhabicher)
Updated•10 years ago
|
Attachment #8484224 -
Flags: review?(aosmond) → review+
Assignee | ||
Updated•10 years ago
|
status-b2g-v1.4:
--- → wontfix
status-b2g-v2.0:
--- → affected
status-b2g-v2.0M:
--- → affected
status-b2g-v2.1:
--- → affected
status-b2g-v2.2:
--- → affected
Assignee | ||
Updated•10 years ago
|
Assignee | ||
Updated•10 years ago
|
Component: General → Gaia::Camera
OS: Linux → Gonk (Firefox OS)
Product: Core → Firefox OS
Hardware: x86_64 → ARM
Target Milestone: --- → 2.1 S4 (12sep)
Assignee | ||
Comment 7•10 years ago
|
||
I've landed this without a test because it's a privacy concern, but the emulator doesn't generate EXIF metadata so it's not trivial to check. We're looking at ways to improve our test framework so that we can catch problems like this in the future, but that's a while out.
For now:
https://hg.mozilla.org/integration/b2g-inbound/rev/664a6bd22f90
Status: ASSIGNED → RESOLVED
Closed: 10 years ago
Resolution: --- → FIXED
Updated•10 years ago
|
blocking-b2g: --- → 2.0M?
Updated•10 years ago
|
Updated•10 years ago
|
blocking-b2g: 2.0M? → ---
You need to log in
before you can comment on or make changes to this bug.
Description
•