s/NOTREACHED/LOG(FATAL)/ to die even in release builds.

BUG=104579
TEST=built Debug & Release on desktop and verified test passes (after failing in Release on desktop before the change to RemoveFilter).


Review URL: http://codereview.chromium.org/8934002

git-svn-id: svn://svn.chromium.org/chrome/trunk/src@114193 0039d316-1c4b-4281-b951-d872f2087c98

media/base/composite_filter.cc

@@ -82,7 +82,7 @@ void CompositeFilter::RemoveFilter(scoped_refptr<Filter> filter) {
     filter->clear_host();
     return;
   }
-  NOTREACHED() << "Filter missing.";
+  LOG(FATAL) << "Filter missing.";
 }
 
 void CompositeFilter::set_host(FilterHost* host) {

media/base/composite_filter_unittest.cc

@@ -396,8 +396,7 @@ TEST_F(CompositeFilterTest, TestAddRemoveFilter) {
 class CompositeFilterDeathTest : public CompositeFilterTest {};
 
 // Test failure of RemoveFilter() on an unknown filter.
-// TODO(fischman): figure out why this is flaky.  http://crbug.com/104579
-TEST_F(CompositeFilterDeathTest, DISABLED_TestRemoveUnknownFilter) {
+TEST_F(CompositeFilterDeathTest, TestRemoveUnknownFilter) {
   composite_->set_host(mock_filter_host_.get());
   // Remove unknown filter.
   scoped_refptr<StrictMock<MockFilter> > filter = new StrictMock<MockFilter>();