Fix PipelineIntegrationTest so it can't hang on decoder errors.

Fix PipelineIntegrationTest so it can't hang on decoder errors.

BUG=109875
TEST=PipelineIntegrationTest.*


Committed: http://src.chromium.org/viewvc/chrome?view=rev&revision=119973

[acolwell GONE FROM CHROMIUM, 2012-01-31 18:10:37]

This fixes the problem, but it feels like there is a readability cost. I'm open
to ideas if you have a cleaner way to do this.

[Ami GONE FROM CHROMIUM, 2012-01-31 19:06:54]

http://codereview.chromium.org/9309004/diff/1/media/filters/pipeline_integrat...
File media/filters/pipeline_integration_test.cc (right):

http://codereview.chromium.org/9309004/diff/1/media/filters/pipeline_integrat...
media/filters/pipeline_integration_test.cc:108: error_status_(PIPELINE_OK) {
since it can hold OK, s/error_/pipeline_/

[Ami GONE FROM CHROMIUM, 2012-01-31 19:07:02]

LGTM mod that nit.

[acolwell GONE FROM CHROMIUM, 2012-01-31 19:16:00]

http://codereview.chromium.org/9309004/diff/1/media/filters/pipeline_integrat...
File media/filters/pipeline_integration_test.cc (right):

http://codereview.chromium.org/9309004/diff/1/media/filters/pipeline_integrat...
media/filters/pipeline_integration_test.cc:108: error_status_(PIPELINE_OK) {
On 2012/01/31 19:06:54, Ami Fischman wrote:
> since it can hold OK, s/error_/pipeline_/

Done.

[commit-bot: I haz the power, 2012-01-31 20:19:15]

CQ is trying da patch. Follow status at
https://chromium-status.appspot.com/cq/acolwell@chromium.org/9309004/4001

[commit-bot: I haz the power, 2012-01-31 22:28:14]

Change committed as 119973

[Brad Chen, 2012-01-31 23:21:24]

We're still seeing failures. See
http://build.chromium.org/p/chromium.memory.fyi/builders/Chromium%20OS%20UI%2...

On 2012/01/31 22:28:14, I haz the power (commit-bot) wrote:
> Change committed as 119973