Re: Issue 584924 in chromium: K9 Mail fails to display e-mail content with WebView 49.

[chro...@googlecode.com]

Updates:
Owner: bo...@chromium.org
Cc: mnag...@chromium.org

Comment #3 on issue 584924 by bo...@chromium.org: K9 Mail fails to display
e-mail content with WebView 49.
https://code.google.com/p/chromium/issues/detail?id=584924

Internal dup b/27038833. Offending change is
https://codereview.chromium.org/1497743005, and the cause is due to calling
loadDataWithBaseUrl where the base url is "http://" which probably doesn't
parse as a valid GURL.

--
You received this message because this project is configured to send all
issue notifications to this address.
You may adjust your notification preferences at:
https://code.google.com/hosting/settings

[chro...@googlecode.com]

Comment #4 on issue 584924 by reil...@chromium.org: K9 Mail fails to

display e-mail content with WebView 49.
https://code.google.com/p/chromium/issues/detail?id=584924

K9 Mail has worked around this issue in version 5.008.

[chro...@googlecode.com]

Comment #5 on issue 584924 by bo...@chromium.org: K9 Mail fails to display

e-mail content with WebView 49.
https://code.google.com/p/chromium/issues/detail?id=584924

Oh, nice. Second internal bug refers to a different app though.

[chro...@googlecode.com]

Comment #6 on issue 584924 by bo...@chromium.org: K9 Mail fails to display

e-mail content with WebView 49.
https://code.google.com/p/chromium/issues/detail?id=584924

That CL exposed an existing bug I think..

If the base url is invalid, the load never goes into
RenderFrameImpl::LoadDataURL, since by that time, the base url has been
completely thrown out. It just tries to do a normal load of what's in the
url field, which after that CL, is simply an *empty* data url.

[chro...@googlecode.com]

Updates:
Labels: -Merge-Request-49 Merge-Approved-49 Hotlist-Merge-Approved

Comment #9 on issue 584924 by tin...@google.com: K9 Mail fails to display

e-mail content with WebView 49.

https://code.google.com/p/chromium/issues/detail?id=584924#c9

Your change meets the bar and is auto-approved for M49 (branch: 2623)