Fix use-after-scope in SharedStyleFinderTest.

Fix use-after-scope in SharedStyleFinderTest.

SharedStyleFinderTest::matchesRuleSet created a
ElementResolveContext instance and passed a reference
to SharedStyleFinder. Then the scope of the context
is ended, and finder retains a reference to an address
on stack that ends up being reused.

It was detected by 'ThinLTO Linux ToT' bot, but it
seems to be just a coincedence.

BUG=645295
Committed: https://crrev.com/25782de6522c605860d6e0fc265b164f3d567442
Cr-Commit-Position: refs/heads/master@{#420879}

[krasin1, 2016-09-24 00:55:36]

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

[krasin1, 2016-09-24 00:55:42]

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

[commit-bot: I haz the power, 2016-09-24 00:56:08]

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-24 02:36:25]

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

[commit-bot: I haz the power, 2016-09-24 02:36:26]

Dry run: This issue passed the CQ dry run.

[meade_UTC10, 2016-09-26 03:12:32]

lgtm

[krasin1, 2016-09-26 03:20:08]

The CQ bit was checked by krasin@chromium.org

[commit-bot: I haz the power, 2016-09-26 03:20:18]

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-26 03:28:07]

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

[commit-bot: I haz the power, 2016-09-26 03:28:08]

Try jobs failed on following builders:
  chromium_presubmit on master.tryserver.chromium.linux (JOB_FAILED,
http://build.chromium.org/p/tryserver.chromium.linux/builders/chromium_presub...)

[krasin1, 2016-09-26 03:31:51]

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

[krasin1, 2016-09-26 03:31:52]

Hi Elliott,

please, approve this one-liner.

[rune, 2016-09-26 06:48:59]

rune@opera.com changed reviewers:
+ rune@opera.com

[rune, 2016-09-26 06:48:59]

lgtm

[krasin1, 2016-09-26 06:55:46]

On 2016/09/26 06:48:59, rune wrote:
> lgtm

Thank you!

[krasin1, 2016-09-26 06:55:52]

The CQ bit was checked by krasin@chromium.org

[commit-bot: I haz the power, 2016-09-26 06:56:08]

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-26 10:40:14]

Committed patchset #1 (id:1)

[commit-bot: I haz the power, 2016-09-26 10:42:25]

Description was changed from

==========
Fix use-after-scope in SharedStyleFinderTest.

SharedStyleFinderTest::matchesRuleSet created a
ElementResolveContext instance and passed a reference
to SharedStyleFinder. Then the scope of the context
is ended, and finder retains a reference to an address
on stack that ends up being reused.

It was detected by 'ThinLTO Linux ToT' bot, but it
seems to be just a coincedence.

BUG=645295
==========

to

==========
Fix use-after-scope in SharedStyleFinderTest.

SharedStyleFinderTest::matchesRuleSet created a
ElementResolveContext instance and passed a reference
to SharedStyleFinder. Then the scope of the context
is ended, and finder retains a reference to an address
on stack that ends up being reused.

It was detected by 'ThinLTO Linux ToT' bot, but it
seems to be just a coincedence.

BUG=645295

Committed: https://crrev.com/25782de6522c605860d6e0fc265b164f3d567442
Cr-Commit-Position: refs/heads/master@{#420879}
==========

[commit-bot: I haz the power, 2016-09-26 10:42:26]

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