Fix non-Oilpan following r379558.

Fix non-Oilpan following r379558.

R=
BUG=580169
Committed: https://crrev.com/49a3f94e52b4b61c06d9ba759973412bb99d26b1
Cr-Commit-Position: refs/heads/master@{#379786}

[sof, 2016-03-08 07:07:49]

Description was changed from

==========
Fix non-Oilpan following r379558.

TBR=
BUG=580169
==========

to

==========
Fix non-Oilpan following r379558.

R=
BUG=580169
==========

[sof, 2016-03-08 07:08:21]

sigbjornf@opera.com changed reviewers:
+ haraken@chromium.org, rbyers@chromium.org

[sof, 2016-03-08 07:08:22]

please take a look.

i'm not very happy about this.

[haraken, 2016-03-08 07:15:19]

LGTM

[sof, 2016-03-08 07:31:53]

The CQ bit was checked by sigbjornf@opera.com

[commit-bot: I haz the power, 2016-03-08 07:32:06]

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

[commit-bot: I haz the power, 2016-03-08 08:23:31]

Description was changed from

==========
Fix non-Oilpan following r379558.

R=
BUG=580169
==========

to

==========
Fix non-Oilpan following r379558.

R=
BUG=580169
==========

[commit-bot: I haz the power, 2016-03-08 08:23:32]

Committed patchset #2 (id:20001)

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

Description was changed from

==========
Fix non-Oilpan following r379558.

R=
BUG=580169
==========

to

==========
Fix non-Oilpan following r379558.

R=
BUG=580169

Committed: https://crrev.com/49a3f94e52b4b61c06d9ba759973412bb99d26b1
Cr-Commit-Position: refs/heads/master@{#379786}
==========

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

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

[Rick Byers, 2016-03-08 20:11:12]

Thanks for the fix, sorry for not catching this in review.  Maybe we should add
a single non-oilpan bot to the CQ?

[sof, 2016-03-08 21:38:22]

On 2016/03/08 20:11:12, Rick Byers wrote:
> Thanks for the fix, sorry for not catching this in review.  Maybe we should
add
> a single non-oilpan bot to the CQ?

Mistakes can easily happen if CQ doesn't tell you, but repeated ones are harder
to accept & tidy up after.

We (Opera) haven't seen much pro-active stability work for (non)Oilpan for a
long time, so don't expect things to change until ToT supports (Oilpan `xor`
!Oilpan.)

[haraken, 2016-03-08 23:03:36]

On 2016/03/08 20:11:12, Rick Byers wrote:
> Thanks for the fix, sorry for not catching this in review.  Maybe we should
add
> a single non-oilpan bot to the CQ?

We've tried that many times but couldn't (because of the amount of work on the
infra team).