Handle case when XSL stylesheet node is removed during load

Handle case when XSL stylesheet node is removed during load

When a XSL stylesheet node is removed from document while its content
is being loaded, the current code doesn't handle the case to clear the resource,
while when the node is added again the new resource is created again.

This causes reference count (StyleEngine::m_pendingStyleSheets) leakage.

BUG=427249
TEST=minimized clusterfuzz test

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

[kochi, 2014-11-13 06:32:39]

kochi@chromium.org changed reviewers:
+ tasak@google.com

[kochi, 2014-11-13 06:35:14]

Hi Takashi,

As we analyzed together, a case when XSL stylesheet
loading is interrupted by moving the node to another
document.

Please take a look.

[tasak, 2014-11-13 06:58:16]

lgtm

[kochi, 2014-11-13 07:39:31]

On 2014/11/13 06:58:16, tasak wrote:
> lgtm

Thanks.


Hayato-san,
Could you review as OWNERS?

[hayato, 2014-11-14 09:58:01]

LGTM

[kochi, 2014-11-14 09:58:13]

The CQ bit was checked by kochi@chromium.org

[commit-bot: I haz the power, 2014-11-14 09:58:27]

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

[commit-bot: I haz the power, 2014-11-14 11:23:36]

The CQ bit was unchecked by commit-bot@chromium.org

[commit-bot: I haz the power, 2014-11-14 11:23:37]

Try jobs failed on following builders:
  mac_blink_rel on tryserver.blink
(http://build.chromium.org/p/tryserver.blink/builders/mac_blink_rel/builds/30862)

[kochi, 2014-11-14 14:55:28]

The CQ bit was checked by kochi@chromium.org

[commit-bot: I haz the power, 2014-11-14 14:56:23]

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

[kochi, 2014-11-14 14:57:20]

The CQ bit was unchecked by kochi@chromium.org

[kochi, 2014-11-14 14:57:47]

On 2014/11/14 14:56:23, I haz the power (commit-bot) wrote:
> CQ is trying da patch. Follow status at
>  https://chromium-cq-status.appspot.com/patch-status/722093002/1

mac_blink_rel failure looks real.  I'll investigate.

[kochi, 2014-11-17 05:52:31]

Updated the layout test.

Takashi, could you take another look?

[tasak, 2014-11-18 06:31:25]

lgtm

[kochi, 2014-11-18 06:43:32]

The CQ bit was checked by kochi@chromium.org

[commit-bot: I haz the power, 2014-11-18 06:44:01]

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

[commit-bot: I haz the power, 2014-11-18 07:33:46]

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