Return immediately after NOTREACHED()'s to avoid logspam.

content/common/gpu/media/omx_video_decode_accelerator.cc

 // Copyright (c) 2012 The Chromium Authors. All rights reserved.
 // Use of this source code is governed by a BSD-style license that can be
 // found in the LICENSE file.
 
 #include "content/common/gpu/media/omx_video_decode_accelerator.h"
 
 #include "base/bind.h"
 #include "base/debug/trace_event.h"
 #include "base/logging.h"
 #include "base/stl_util.h"
@@ skipping 862 matching lines @@
     case INITIALIZING:
       switch (reached) {
         case OMX_StateIdle:
           OnReachedIdleInInitializing();
           return;
         case OMX_StateExecuting:
           OnReachedExecutingInInitializing();
           return;
         default:
           NOTREACHED() << "Unexpected state in INITIALIZING: " << reached;
+          return;
       }
     case RESETTING:
       switch (reached) {
         case OMX_StatePause:
           OnReachedPauseInResetting();
           return;
         case OMX_StateExecuting:
           OnReachedExecutingInResetting();
           return;
         default:
           NOTREACHED() << "Unexpected state in RESETTING: " << reached;
+          return;
       }
     case DESTROYING:
       switch (reached) {
         case OMX_StatePause:
         case OMX_StateExecuting:
           // Because Destroy() can interrupt an in-progress Reset(),
           // we might arrive at these states after current_state_change_ was
           // overwritten with DESTROYING.  That's fine though - we already have
           // the state transition for Destroy() queued up at the component, so
           // we treat this as a no-op.
           return;
         case OMX_StateIdle:
           OnReachedIdleInDestroying();
           return;
         case OMX_StateLoaded:
           OnReachedLoadedInDestroying();
           return;
         default:
           NOTREACHED() << "Unexpected state in DESTROYING: " << reached;
+          return;
       }
     case ERRORING:
       switch (reached) {
         case OMX_StateInvalid:
           OnReachedInvalidInErroring();
           return;
         default:
           NOTREACHED() << "Unexpected state in ERRORING: " << reached;
+          return;
       }
     default:
       NOTREACHED() << "Unexpected state in " << current_state_change_
                    << ": " << reached;
   }
 }
 
 void OmxVideoDecodeAccelerator::EventHandlerCompleteTask(OMX_EVENTTYPE event,
                                                          OMX_U32 data1,
                                                          OMX_U32 data2) {
@@ skipping 168 matching lines @@
 
 bool OmxVideoDecodeAccelerator::SendCommandToPort(
     OMX_COMMANDTYPE cmd, int port_index) {
   DCHECK_EQ(message_loop_, MessageLoop::current());
   OMX_ERRORTYPE result = OMX_SendCommand(component_handle_,
                                          cmd, port_index, 0);
   RETURN_ON_OMX_FAILURE(result, "SendCommand() failed" << cmd,
                         PLATFORM_FAILURE, false);
   return true;
 }