Remove explict frame removal from fetch-request-css-base-url.html test

Remove explict frame removal from fetch-request-css-base-url.html test

This was causing occasional test failures on the WebKit Win7 Dbg
builder, as the frame variable was sometimes not set when the message
from the service worker was being handled. Since the frames created by
with_iframe() are auto-removed now, the explicit removal isn't necessary
anymore.

R=falken@chromium.org
BUG=648247
Committed: https://crrev.com/b4c7dfe047c13a76225f03badacd2e5a46f22982
Cr-Commit-Position: refs/heads/master@{#419736}

[iclelland, 2016-09-19 16:57:07]

The CQ bit was checked by iclelland@chromium.org to run a CQ dry run

[commit-bot: I haz the power, 2016-09-19 16:57:21]

Dry run: CQ is trying da patch. Follow status at
 
https://chromium-cq-status.appspot.com/v2/patch-status/codereview.chromium.or...

[commit-bot: I haz the power, 2016-09-19 17:50:08]

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

[commit-bot: I haz the power, 2016-09-19 17:50:09]

Dry run: This issue passed the CQ dry run.

[iclelland, 2016-09-19 17:52:06]

+r falken -- this should fix the Win7 flake on this test, but I wanted to run it
past you, since you added the frame.remove() call originally.

Can you take a look and let me know if my reasoning is good here?

Thanks!

[falken, 2016-09-20 04:06:13]

lgtm. Interesting bug! We should try to remove unnecessary frame.remove() in our
tests.

[iclelland, 2016-09-20 13:10:53]

The CQ bit was checked by iclelland@chromium.org

[commit-bot: I haz the power, 2016-09-20 13:11:04]

CQ is trying da patch. Follow status at
 
https://chromium-cq-status.appspot.com/v2/patch-status/codereview.chromium.or...

[commit-bot: I haz the power, 2016-09-20 13:15:54]

Committed patchset #1 (id:1)

[commit-bot: I haz the power, 2016-09-20 13:17:08]

Description was changed from

==========
Remove explict frame removal from fetch-request-css-base-url.html test

This was causing occasional test failures on the WebKit Win7 Dbg
builder, as the frame variable was sometimes not set when the message
from the service worker was being handled. Since the frames created by
with_iframe() are auto-removed now, the explicit removal isn't necessary
anymore.

R=falken@chromium.org
BUG=648247
==========

to

==========
Remove explict frame removal from fetch-request-css-base-url.html test

This was causing occasional test failures on the WebKit Win7 Dbg
builder, as the frame variable was sometimes not set when the message
from the service worker was being handled. Since the frames created by
with_iframe() are auto-removed now, the explicit removal isn't necessary
anymore.

R=falken@chromium.org
BUG=648247

Committed: https://crrev.com/b4c7dfe047c13a76225f03badacd2e5a46f22982
Cr-Commit-Position: refs/heads/master@{#419736}
==========

[commit-bot: I haz the power, 2016-09-20 13:17:09]

Patchset 1 (id:??) landed as
https://crrev.com/b4c7dfe047c13a76225f03badacd2e5a46f22982
Cr-Commit-Position: refs/heads/master@{#419736}