cc: Remove unnecessary "default" cases from switch statements.

cc/resources/raster_mode.cc

 // Copyright 2013 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 "cc/resources/raster_mode.h"
 
 #include "base/logging.h"
 #include "base/memory/scoped_ptr.h"
 #include "base/values.h"
 
 namespace cc {
 
 scoped_ptr<base::Value> RasterModeAsValue(RasterMode raster_mode) {
   switch (raster_mode) {
     case HIGH_QUALITY_NO_LCD_RASTER_MODE:
       return scoped_ptr<base::Value>(
           base::Value::CreateStringValue("HIGH_QUALITY_NO_LCD_RASTER_MODE"));
     case HIGH_QUALITY_RASTER_MODE:
       return scoped_ptr<base::Value>(
           base::Value::CreateStringValue("HIGH_QUALITY_RASTER_MODE"));
     case LOW_QUALITY_RASTER_MODE:
       return scoped_ptr<base::Value>(
           base::Value::CreateStringValue("LOW_QUALITY_RASTER_MODE"));
-    case NUM_RASTER_MODES:
     default:

[vmpstr, 2013/08/26 15:38:30]

I think this can also be removed with NOTREACHED at the end of the function.

[reveman, 2013/08/26 17:16:57]

We need it as NUM_RASTER_MODES still exists.

[vmpstr, 2013/08/26 17:19:17]

Ah, my bad. I confused this with NUM_MODES that you removed.

       NOTREACHED() << "Unrecognized RasterMode value " << raster_mode;
       return scoped_ptr<base::Value>(
           base::Value::CreateStringValue("<unknown RasterMode value>"));
   }
 }
 
 }  // namespace cc