Mac: Suppress asserts about resources being in use

Mac: Suppress asserts about resources being in use

This papers over the asserts that are occurring on the trybots by
changing the failing DCHECKs to not fail the reason for the failure
was IOSurfaceIsInUse returning YES unexpectedly.

This lets the asserts stay (mostly, completely on non-Mac platforms) in
place, but un-breaks the tryservers.

BUG=577121
CQ_INCLUDE_TRYBOTS=tryserver.blink:linux_blink_rel

Committed: https://crrev.com/60d756a6b3daaf8e579bc8464f8cf72f0cb090f3
Cr-Commit-Position: refs/heads/master@{#374786}

[ccameron, 2016-02-10 21:49:10]

Description was changed from

==========
Mac: Suppress asserts about resources being in use

This papers over the asserts that are occurring on the trybots by
changing the failing DCHECKs to not fail the reason for the failure
was IOSurfaceIsInUse returning YES unexpectedly.

This lets the asserts stay (mostly, completely on non-Mac platforms) in
place, but un-breaks the tryservers.

BUG=577121
==========

to

==========
Mac: Suppress asserts about resources being in use

This papers over the asserts that are occurring on the trybots by
changing the failing DCHECKs to not fail the reason for the failure
was IOSurfaceIsInUse returning YES unexpectedly.

This lets the asserts stay (mostly, completely on non-Mac platforms) in
place, but un-breaks the tryservers.

BUG=577121
CQ_INCLUDE_TRYBOTS=tryserver.blink:linux_blink_rel
==========

[ccameron, 2016-02-10 21:49:52]

ccameron@chromium.org changed reviewers:
+ dyen@chromium.org

[ccameron, 2016-02-10 21:49:53]

More spiralling out of control of this ...

[David Yen, 2016-02-10 21:59:31]

LGTM, do we have any ideas what could be going wrong here?

[ccameron, 2016-02-10 22:04:36]

Thanks. Still no -- never got a repro.

[ccameron, 2016-02-10 22:17:47]

The CQ bit was checked by ccameron@chromium.org

[ccameron, 2016-02-10 22:17:47]

The patchset sent to the CQ was uploaded after l-g-t-m from dyen@chromium.org
Link to the patchset: https://codereview.chromium.org/1688953002/#ps20001
(title: "Add missed function")

[commit-bot: I haz the power, 2016-02-10 22:18:07]

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

[commit-bot: I haz the power, 2016-02-10 23:43:06]

Committed patchset #2 (id:20001)

[commit-bot: I haz the power, 2016-02-16 22:32:42]

Description was changed from

==========
Mac: Suppress asserts about resources being in use

This papers over the asserts that are occurring on the trybots by
changing the failing DCHECKs to not fail the reason for the failure
was IOSurfaceIsInUse returning YES unexpectedly.

This lets the asserts stay (mostly, completely on non-Mac platforms) in
place, but un-breaks the tryservers.

BUG=577121
CQ_INCLUDE_TRYBOTS=tryserver.blink:linux_blink_rel
==========

to

==========
Mac: Suppress asserts about resources being in use

This papers over the asserts that are occurring on the trybots by
changing the failing DCHECKs to not fail the reason for the failure
was IOSurfaceIsInUse returning YES unexpectedly.

This lets the asserts stay (mostly, completely on non-Mac platforms) in
place, but un-breaks the tryservers.

BUG=577121
CQ_INCLUDE_TRYBOTS=tryserver.blink:linux_blink_rel

Committed: https://crrev.com/60d756a6b3daaf8e579bc8464f8cf72f0cb090f3
Cr-Commit-Position: refs/heads/master@{#374786}
==========

[commit-bot: I haz the power, 2016-02-16 22:32:43]

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