Remove the pending container size requests list

Remove the pending container size requests list

Trust ImageResource::setContainerSizeForLayoutObject to be called
through ImageResource::notifyObservers when the image has loaded.
Given that, we will already get notifications about the up-to-date
containerSize and containerZoom and there is no need for keeping
around a list of pending container sizes and zooms to send when the
image is created.

As to why we can trust ImageResource::notifyObservers to call back
into ImageResource::setContainerSizeForLayoutObject,
https://codereview.chromium.org/22482004 added a
updateInnerContentRect() call taking care of calling
ImageResource::setContainerSizeForLayoutObject when the image's paint
has been invalidated.

Committed: https://src.chromium.org/viewvc/blink?view=rev&revision=194439

[davve, 2015-04-24 12:47:12]

davve@opera.com changed reviewers:
+ mstensho@opera.com, pdr@chromium.org

[davve, 2015-04-24 12:47:13]

This is a timing sensitive area, but from three try-bot runs no tests
seem to react to the removal of this code so let's try removal. Worst
case, we have to revert and will have a test in place that proves its
worth.

[pdr., 2015-04-25 02:23:21]

On 2015/04/24 at 12:47:13, davve wrote:
> This is a timing sensitive area, but from three try-bot runs no tests
> seem to react to the removal of this code so let's try removal. Worst
> case, we have to revert and will have a test in place that proves its
> worth.

So glad to see this code go. Thanks!

LGTM

[pdr., 2015-04-25 02:23:39]

The CQ bit was checked by pdr@chromium.org

[commit-bot: I haz the power, 2015-04-25 02:24:21]

CQ is trying da patch. Follow status at
 https://chromium-cq-status.appspot.com/patch-status/1102803002/40001

[commit-bot: I haz the power, 2015-04-25 16:42:37]

Committed patchset #3 (id:40001) as
https://src.chromium.org/viewvc/blink?view=rev&revision=194439

[sof, 2015-04-25 18:44:13]

https://codereview.chromium.org/1102803002/diff/40001/Source/core/fetch/Image...
File Source/core/fetch/ImageResource.h (left):

https://codereview.chromium.org/1102803002/diff/40001/Source/core/fetch/Image...
Source/core/fetch/ImageResource.h:132: virtual void
switchClientsToRevalidatedResource() override;
If the CL sticks, this method can now be de-virtual'ized (and made private) over
Resource?

[davve, 2015-04-25 20:53:24]

https://codereview.chromium.org/1102803002/diff/40001/Source/core/fetch/Image...
File Source/core/fetch/ImageResource.h (left):

https://codereview.chromium.org/1102803002/diff/40001/Source/core/fetch/Image...
Source/core/fetch/ImageResource.h:132: virtual void
switchClientsToRevalidatedResource() override;
On 2015/04/25 18:44:13, sof wrote:
> If the CL sticks, this method can now be de-virtual'ized (and made private)
over
> Resource?

Good catch! Uploaded as https://codereview.chromium.org/1109663002/