Closed Bug 931054 Opened 9 years ago Closed 9 years ago
.js tries to access undefined preview metadata
Filmstrip hash to call parseJPEGMetadata on the preview to retrieve its metadata.
OS: Mac OS X → Gonk (Firefox OS)
Hardware: x86 → ARM
Summary: [Camera] filmstrip.js access undefined preview metadata → [Camera] filmstrip.js tries to access undefined preview metadata
Should be hd+ because it blocks an hd+ bug.
blocking-b2g: --- → hd?
Moving to hd+ since the bug that this blocks is a hd blocker
blocking-b2g: hd? → hd+
Comment on attachment 822412 [details] [review] Github Pull Request r+, but see my comments on github: 1) Use + instead of \ for long strings 2) File another hd+ bug for adding devicePixelRatio in the various places it will be needed. Once we get the embedded preview image size right, photos will look blurry on Helix unless we add devicePixelRatio support, so that new bug will have to block the same bug this one does.
Attachment #822412 - Flags: review?(dflanagan) → review+
https://github.com/mozilla-b2g/gaia/commit/1ec3346f270c7395a6b863403edeec2d527cf7c9 lint errors and bunch of other red on ci- backed out above.
Attaching a patch that applies on v1.1hd. I need some help. I don't have a device to test it. lecky, can you give it a try?
I tested the patch for v.1.1hd and it works fine to me. I'm sending this for review.
Attachment #825442 - Flags: review?(dflanagan)
Attached pull request specific to v.1.1.0hd. Need review for if
Setting the tracking flags, so this can be uplifted to v1.2 as well. Diego: are there merge conflicts or can we just let a sheriff uplift this to the hd branch? Also, comment 12 is incomplete... What do you want me to review for?
What's a sheriff uplift? The patch I created for master doesn't apply on v.1.1.0hd. The code has diverged.
I just need a review for the patch for v.1.1.0hd https://github.com/mozilla-b2g/gaia/pull/13264
Comment on attachment 825442 [details] [review] Pull request for v.1.1.0hd The patch looks good to me. AFAIK, you don't generally need to request reviews for uplifts. I assume you've tried it out on the hd branch... And if it works, then go ahead and land.
Attachment #825442 - Flags: review?(dflanagan) → review+
Status: NEW → RESOLVED
Closed: 9 years ago
Resolution: --- → FIXED
No it's merged on v.1.2 branch as well https://github.com/dmarcos/gaia/commit/1d161d820058f25e0e6c8f5a74e6c3fb4a010bb1
You need to log in before you can comment on or make changes to this bug.