Fix use-after-scope issue in WebFrameTest.

Fix use-after-scope issue in WebFrameTest.

Local variable client goes out of scope before WebViewHelper,
which indirectly access client during destruction.

The bug was found by AddressSanitizer with use-after-free check
enabled. It's currently being rolled out into Chrome, and this CL
is a part of a larger cleanup of existing failures.

BUG=649897
Committed: https://crrev.com/f21d6e5f8417163e52730a30b9e85fb6cfd6f251
Cr-Commit-Position: refs/heads/master@{#439982}

[krasin1, 2016-12-21 00:07:04]

krasin@chromium.org changed reviewers:
+ haraken@chromium.org

[haraken, 2016-12-21 00:08:12]

LGTM

[krasin1, 2016-12-21 00:19:06]

On 2016/12/21 00:08:12, haraken wrote:
> LGTM

Thank you!

[krasin1, 2016-12-21 00:19:08]

The CQ bit was checked by krasin@chromium.org

[commit-bot: I haz the power, 2016-12-21 00:19:50]

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

[krasin1, 2016-12-21 00:20:33]

The CQ bit was unchecked by krasin@chromium.org

[krasin1, 2016-12-21 00:31:35]

The CQ bit was checked by krasin@chromium.org

[krasin1, 2016-12-21 00:31:35]

The patchset sent to the CQ was uploaded after l-g-t-m from haraken@chromium.org
Link to the patchset: https://codereview.chromium.org/2591733003/#ps20001
(title: "Fix a few more similar cases")

[commit-bot: I haz the power, 2016-12-21 00:32:55]

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-12-21 02:21:12]

CQ is committing da patch.

Bot data: {"patchset_id": 20001, "attempt_start_ts": 1482280295314510,
"parent_rev": "2100497f610b8f549ded88fc521bb77867029b97", "commit_rev":
"ad1d221909d1ae93f5921d1a9f50404bdf246c4c"}

[commit-bot: I haz the power, 2016-12-21 02:21:38]

Description was changed from

==========
Fix use-after-scope issue in WebFrameTest.

Local variable client goes out of scope before WebViewHelper,
which indirectly access client during destruction.

The bug was found by AddressSanitizer with use-after-free check
enabled. It's currently being rolled out into Chrome, and this CL
is a part of a larger cleanup of existing failures.

BUG=649897
==========

to

==========
Fix use-after-scope issue in WebFrameTest.

Local variable client goes out of scope before WebViewHelper,
which indirectly access client during destruction.

The bug was found by AddressSanitizer with use-after-free check
enabled. It's currently being rolled out into Chrome, and this CL
is a part of a larger cleanup of existing failures.

BUG=649897

Review-Url: https://codereview.chromium.org/2591733003
==========

[commit-bot: I haz the power, 2016-12-21 02:21:39]

Committed patchset #2 (id:20001)

[commit-bot: I haz the power, 2016-12-21 02:23:45]

Description was changed from

==========
Fix use-after-scope issue in WebFrameTest.

Local variable client goes out of scope before WebViewHelper,
which indirectly access client during destruction.

The bug was found by AddressSanitizer with use-after-free check
enabled. It's currently being rolled out into Chrome, and this CL
is a part of a larger cleanup of existing failures.

BUG=649897

Review-Url: https://codereview.chromium.org/2591733003
==========

to

==========
Fix use-after-scope issue in WebFrameTest.

Local variable client goes out of scope before WebViewHelper,
which indirectly access client during destruction.

The bug was found by AddressSanitizer with use-after-free check
enabled. It's currently being rolled out into Chrome, and this CL
is a part of a larger cleanup of existing failures.

BUG=649897

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

[commit-bot: I haz the power, 2016-12-21 02:23:46]

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