[iOS Reading List] Skip invalid images in distillation.

[iOS Reading List] Skip invalid images in distillation.

If an image is invalid (e.g. src pointing to not existing URL), the
current behavior is to fail the whole distillation.
Just skip the image instead (as normal rendering would do).
Replace src by empty string to avoid network acces when displaying
offline page.

BUG=690829
Review-Url: https://codereview.chromium.org/2683393002
Cr-Commit-Position: refs/heads/master@{#449577}
Committed: https://chromium.googlesource.com/chromium/src/+/182bfc3b6cd2c7536577752dfbf032e42207b34f

[Olivier, 2017-02-10 10:12:32]

olivierrobin@chromium.org changed reviewers:
+ gambard@chromium.org

[gambard, 2017-02-10 10:15:18]

lgtm

[Olivier, 2017-02-10 10:16:16]

The CQ bit was checked by olivierrobin@chromium.org

[commit-bot: I haz the power, 2017-02-10 10:16:33]

CQ is trying da patch. Follow status at
 
https://chromium-cq-status.appspot.com/v2/patch-status/codereview.chromium.or...

[commit-bot: I haz the power, 2017-02-10 10:26:31]

CQ is committing da patch.

Bot data: {"patchset_id": 1, "attempt_start_ts": 1486721776539650, "parent_rev":
"cb2ff8ea37866225b2052e4fa0ab6a2945484889", "commit_rev":
"182bfc3b6cd2c7536577752dfbf032e42207b34f"}

[commit-bot: I haz the power, 2017-02-10 10:27:07]

Description was changed from

==========
[iOS Reading List] Skip invalid iages in distillation.

If an image is invalid (e.g. src pointing to not existing URL), the
current behavior is to fail the whole distillation.
Just skip the image instead (as normal rendering would do).
Replace src by empty string to avoid network acces when displaying
offline page.

BUG=690829
==========

to

==========
[iOS Reading List] Skip invalid iages in distillation.

If an image is invalid (e.g. src pointing to not existing URL), the
current behavior is to fail the whole distillation.
Just skip the image instead (as normal rendering would do).
Replace src by empty string to avoid network acces when displaying
offline page.

BUG=690829

Review-Url: https://codereview.chromium.org/2683393002
Cr-Commit-Position: refs/heads/master@{#449577}
Committed:
https://chromium.googlesource.com/chromium/src/+/182bfc3b6cd2c7536577752dfbf0...
==========

[commit-bot: I haz the power, 2017-02-10 10:27:08]

Committed patchset #1 (id:1) as
https://chromium.googlesource.com/chromium/src/+/182bfc3b6cd2c7536577752dfbf0...

[Olivier, 2017-02-10 10:41:07]

Description was changed from

==========
[iOS Reading List] Skip invalid iages in distillation.

If an image is invalid (e.g. src pointing to not existing URL), the
current behavior is to fail the whole distillation.
Just skip the image instead (as normal rendering would do).
Replace src by empty string to avoid network acces when displaying
offline page.

BUG=690829

Review-Url: https://codereview.chromium.org/2683393002
Cr-Commit-Position: refs/heads/master@{#449577}
Committed:
https://chromium.googlesource.com/chromium/src/+/182bfc3b6cd2c7536577752dfbf0...
==========

to

==========
[iOS Reading List] Skip invalid images in distillation.

If an image is invalid (e.g. src pointing to not existing URL), the
current behavior is to fail the whole distillation.
Just skip the image instead (as normal rendering would do).
Replace src by empty string to avoid network acces when displaying
offline page.

BUG=690829

Review-Url: https://codereview.chromium.org/2683393002
Cr-Commit-Position: refs/heads/master@{#449577}
Committed:
https://chromium.googlesource.com/chromium/src/+/182bfc3b6cd2c7536577752dfbf0...
==========