Stop RenderProgress timer on destruction.

Stop RenderProgress timer on destruction.

To handle the Oilpan case where the RenderObject stays
around until the next GC after the object has been
destroy()ed, stop the RenderProgress animation timer.

R=tkent@chromium.org
BUG=

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

[sof, 2014-08-19 07:02:04]

Please take a look.

Somewhat of a speculative null pointer fix, Oilpan bots are currently a bit red
due to this. Relevant stack trace:


https://storage.googleapis.com/chromium-layout-test-archives/WebKit_Win_Oilpa...

 (from
https://storage.googleapis.com/chromium-layout-test-archives/WebKit_Win_Oilpa...
)

[tkent, 2014-08-19 07:15:05]

In Oilpan, RenderObject is alive after destroy() until the next GC.
RenderProgress should override destroy(), and it should stop the timer.

[sof, 2014-08-19 07:29:52]

On 2014/08/19 07:15:05, tkent wrote:
> In Oilpan, RenderObject is alive after destroy() until the next GC.
> RenderProgress should override destroy(), and it should stop the timer.

Thanks for explaining the actual underlying cause; CL&description updated -
PTAL?

[tkent, 2014-08-19 07:41:41]

lgtm

[sof, 2014-08-19 07:49:48]

The CQ bit was checked by sigbjornf@opera.com

[commit-bot: I haz the power, 2014-08-19 07:50:35]

CQ is trying da patch. Follow status at
 https://chromium-status.appspot.com/cq/sigbjornf@opera.com/476543003/20001

[commit-bot: I haz the power, 2014-08-19 08:50:40]

FYI, CQ is re-trying this CL (attempt #1).
The failing builders are:
  win_blink_rel on tryserver.blink
(http://build.chromium.org/p/tryserver.blink/builders/win_blink_rel/builds/23380)

[commit-bot: I haz the power, 2014-08-19 09:21:50]

Committed patchset #2 (20001) as 180536

[haraken, 2014-08-19 11:37:24]

LGTM