Commit eb0c87bd authored by xhwang's avatar xhwang Committed by Commit bot

Fix media::MojoRendererTest.

Now we have real decoders abailable so MojoRendererImpl::Initialize() should
succeed.

Review URL: https://codereview.chromium.org/793973004

Cr-Commit-Position: refs/heads/master@{#308475}
parent e74f9bf8
...@@ -99,8 +99,7 @@ namespace media { ...@@ -99,8 +99,7 @@ namespace media {
class MojoRendererTest : public mojo::test::ApplicationTestBase { class MojoRendererTest : public mojo::test::ApplicationTestBase {
public: public:
MojoRendererTest() MojoRendererTest() : service_provider_(NULL) {}
: service_provider_(NULL) {}
~MojoRendererTest() override {} ~MojoRendererTest() override {}
protected: protected:
...@@ -157,10 +156,7 @@ TEST_F(MojoRendererTest, BasicInitialize) { ...@@ -157,10 +156,7 @@ TEST_F(MojoRendererTest, BasicInitialize) {
media::Renderer::PaintCB(), base::Closure(), media::Renderer::PaintCB(), base::Closure(),
base::Bind(&ErrorCallback, &expected_error)); base::Bind(&ErrorCallback, &expected_error));
base::MessageLoop::current()->Run(); base::MessageLoop::current()->Run();
EXPECT_EQ(PIPELINE_OK, expected_error);
// We expect an error during initialization because MojoRendererService
// doesn't initialize any decoders, which causes an error.
EXPECT_EQ(PIPELINE_ERROR_COULD_NOT_RENDER, expected_error);
} }
} // namespace media } // namespace media
Markdown is supported
0%
or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment