Finalize scheduler before shutting down Blink.

Finalize scheduler before shutting down Blink.

Have the Blink unittest runner shutdown the scheduler before doing
same to Blink, as the former may well access objects in the latter.

R=kouhei,haraken
BUG=

Committed: https://crrev.com/12e3f839fe65a520ee0d1a0176110f8e05513ef0
Cr-Commit-Position: refs/heads/master@{#323453}

[kouhei (in TOK), 2015-04-02 07:07:56]

kouhei@chromium.org changed reviewers:
+ kouhei@chromium.org

[kouhei (in TOK), 2015-04-02 07:07:56]

The CQ bit was checked by kouhei@chromium.org

[kouhei (in TOK), 2015-04-02 07:07:56]

lgtm

[kouhei (in TOK), 2015-04-02 07:08:00]

The CQ bit was unchecked by kouhei@chromium.org

[haraken, 2015-04-02 07:26:51]

LGTM!

[haraken, 2015-04-02 07:27:02]

The CQ bit was checked by haraken@chromium.org

[commit-bot: I haz the power, 2015-04-02 07:27:14]

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

[commit-bot: I haz the power, 2015-04-02 07:37:04]

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

[commit-bot: I haz the power, 2015-04-02 07:37:05]

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

[haraken, 2015-04-02 07:42:34]

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

[haraken, 2015-04-02 07:42:35]

Jochen: PTAL? We need to land this to fix oilpan builds.

[jochen (gone - plz use gerrit), 2015-04-02 08:08:38]

jochen@chromium.org changed reviewers:
+ rmcilroy@chromium.org

[jochen (gone - plz use gerrit), 2015-04-02 08:08:39]

the bots aren't happy?

+ross for scheduler-foo

[haraken, 2015-04-02 08:22:38]

haraken@chromium.org changed reviewers:
+ alexclarke@chromium.org, skyostil@chromium.org

[haraken, 2015-04-02 08:22:39]

+sami, alex

[sof, 2015-04-02 08:23:46]

On 2015/04/02 08:08:39, jochen wrote:
> the bots aren't happy?
> 
> +ross for scheduler-foo

May not have a scheduler attached; added reqd null check.

[jochen (gone - plz use gerrit), 2015-04-02 08:28:25]

lgtm

[haraken, 2015-04-02 08:29:10]

The CQ bit was checked by haraken@chromium.org

[haraken, 2015-04-02 08:29:11]

The patchset sent to the CQ was uploaded after l-g-t-m from
haraken@chromium.org, kouhei@chromium.org
Link to the patchset: https://codereview.chromium.org/1054893002/#ps20001
(title: "optionally shutdown scheduler")

[commit-bot: I haz the power, 2015-04-02 08:29:22]

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

[rmcilroy, 2015-04-02 09:08:26]

On 2015/04/02 08:29:22, I haz the power (commit-bot) wrote:
> CQ is trying da patch. Follow status at
>  https://chromium-cq-status.appspot.com/patch-status/1054893002/20001

Lgtm, thanks!

[chromium-reviews, 2015-04-02 09:21:57]

Thanks for fixing this.

On Thu, Apr 2, 2015 at 10:08 AM, <rmcilroy@chromium.org> wrote:

> On 2015/04/02 08:29:22, I haz the power (commit-bot) wrote:
>
>> CQ is trying da patch. Follow status at
>>   https://chromium-cq-status.appspot.com/patch-status/1054893002/20001
>>
>
> Lgtm, thanks!
>
> https://codereview.chromium.org/1054893002/
>

To unsubscribe from this group and stop receiving emails from it, send an email
to chromium-reviews+unsubscribe@chromium.org.

[commit-bot: I haz the power, 2015-04-02 10:01:45]

Committed patchset #2 (id:20001)

[commit-bot: I haz the power, 2015-04-03 20:23:55]

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