Add CHECK()s to investigate Issue 737392

Desktop / Chromium - Hiroshige Hayashizaki [chromium.org] - 15 July 2017 03:16 EDT

ImageResourceContent::content_status_ is suspected to be reset to kNotStarted (i.e. 0) during ImageResourceContent::UpdateImage(), but therer is no code that sets content_status_ to kNotStarted explicitly.

This CL adds CHECKs() to- Check whether |this| is garbage collected unexpectedly during UpdateImage(), and
- Investigate where |content_status_| becomes kNotStarted precisely.

This is to be reverted before going to beta.

Bug: 737392 Change-Id: I2879bc54771a295a2dd77fed6808c1c08668e262 Reviewed-on: https://chromium-review.googlesource.com/572036 Commit-Queue: Hiroshige Hayashizaki

9517618 Add CHECK()s to investigate Issue 737392
.../core/loader/resource/ImageResourceContent.cpp | 19 ++++++++++++++++---
.../core/loader/resource/ImageResourceContent.h | 4 ++--
2 files changed, 18 insertions(+), 5 deletions(-)

Upstream: git.chromium.org


  • Share