plugins/update-widgets-crash.html victim element goes missing

plugins/update-widgets-crash.html victim element goes missing

The victim element to be removed is non-existant sometimes after the bugs
listed resolved. Workaround for now, by testing that the "victim" element
exists to avoid an exception, which causes extra test console output, and
fails the test. See issue 476590 #46 for a bot failure example.

TBR=haraken@chromium.org
BUG=457054, 476590
NOTRY=true

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

[Noel Gordon, 2015-05-01 13:37:31]

noel@chromium.org changed reviewers:
+ bbudge@chromium.org, haraken@chromium.org

[Noel Gordon, 2015-05-01 13:39:10]

The CQ bit was checked by noel@chromium.org

[commit-bot: I haz the power, 2015-05-01 13:39:24]

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

[Noel Gordon, 2015-05-01 13:41:06]

Might need if (victim && victim.parentNode) but we'll see.  Can't be any more
broken / racy than it seems to be on the bots right now.

[commit-bot: I haz the power, 2015-05-01 13:43:08]

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

[haraken, 2015-05-01 13:49:11]

LGTM

[Noel Gordon, 2015-05-01 15:19:46]

Shame it didn't quite fix it.  Reverting here, and will remove the original test
failure via TestExpectations.

[Noel Gordon, 2015-05-01 15:21:03]

A revert of this CL (patchset #1 id:1) has been created in
https://codereview.chromium.org/1120963002/ by noel@chromium.org.

The reason for reverting is: Didn't quite fix the issue: same failure appeared
on the build bots.  Reverting here to fix via TestExpectations instead..