Closed Bug 914493 Opened 8 years ago Closed 8 years ago
[Email] iframe is not resized for the embedded images that were already downloaded while building body
STR: 1. Create an IMAP account. 2. Receive an email with large bmp image as inline image 3. Message box -> Open the Email -> click to down load the line image 4. Downloaded image shows with scroll bars. 5. Go back 6. Again open the same email Result: Image is not displayed completely Expected: Image should be displayed with proper scrolling. Note: 1.Issue reproduced with gaia v1-train.
Test image with which issue reproduces. Add the attached image as inline and send. Open the email from FxOS device.
Andrew, The attached patch has fix for resizing the iframe for embedded images, when we are building body. Please review and provide feedback. Thanks.
Pointer to Github pull-request
Comment on attachment 802158 [details] Pointer to Github pull request: https://github.com/mozilla-b2g/gaia/pull/12059 r=asuth. I'll land this once it's clear there are no Travis surprises. Currently skipping unit tests since the tests are about two orders of magnitude more complex than the fix and the fix is a logical consistency issue. (Use a callback in the other places we use a callback). For the 1.3 cycle I believe we are planning to fix the message display mechanism which is when I expect us to address the test deficit. psingapati, if you can mark the bug leo+, I can land this on v1-train for you too.
Attachment #802158 - Flags: review+
Thanks for r+. made leo+, please land it on v1-train. Thanks
Landed on gaia/master: https://github.com/mozilla-b2g/gaia/pull/12059 https://github.com/mozilla-b2g/gaia/commit/ee8f269603d0ddfd7a25e1eb10c0ce31b3389a37 Landed on gaia/v1-train: https://github.com/mozilla-b2g/gaia/pull/12060 https://github.com/mozilla-b2g/gaia/commit/946623575b3fbf6138fff5f77ddead3cadffacce
v1.1.0hd: 946623575b3fbf6138fff5f77ddead3cadffacce v1.1.0hd: f73489abba922237dcf7ff0c52df2a972bc59b78
You need to log in before you can comment on or make changes to this bug.