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

cc/resources/managed_tile_state.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/managed_tile_state.h"
 
 #include <limits>
 
 #include "cc/base/math_util.h"
 
@@ skipping 15 matching lines @@
           "EVENTUALLY_AND_ACTIVE_BIN"));
   case EVENTUALLY_BIN:
       return scoped_ptr<base::Value>(base::Value::CreateStringValue(
           "EVENTUALLY_BIN"));
   case NEVER_AND_ACTIVE_BIN:
       return scoped_ptr<base::Value>(base::Value::CreateStringValue(
           "NEVER_AND_ACTIVE_BIN"));
   case NEVER_BIN:
       return scoped_ptr<base::Value>(base::Value::CreateStringValue(
           "NEVER_BIN"));
-  default:
-      DCHECK(false) << "Unrecognized ManagedTileBin value " << bin;
-      return scoped_ptr<base::Value>(base::Value::CreateStringValue(
-          "<unknown ManagedTileBin value>"));
   }
+  DCHECK(false) << "Unrecognized ManagedTileBin value " << bin;
+  return scoped_ptr<base::Value>(base::Value::CreateStringValue(
+      "<unknown ManagedTileBin value>"));
 }
 
 scoped_ptr<base::Value> ManagedTileBinPriorityAsValue(

[vmpstr, 2013/08/22 23:33:34]

I don't think this function is used at all

     ManagedTileBinPriority bin_priority) {
   switch (bin_priority) {
   case HIGH_PRIORITY_BIN:
       return scoped_ptr<base::Value>(base::Value::CreateStringValue(
           "HIGH_PRIORITY_BIN"));
   case LOW_PRIORITY_BIN:
       return scoped_ptr<base::Value>(base::Value::CreateStringValue(
           "LOW_PRIORITY_BIN"));
-  default:
-      DCHECK(false) << "Unrecognized ManagedTileBinPriority value";
-      return scoped_ptr<base::Value>(base::Value::CreateStringValue(
-          "<unknown ManagedTileBinPriority value>"));
   }
+  DCHECK(false) << "Unrecognized ManagedTileBinPriority value";
+  return scoped_ptr<base::Value>(base::Value::CreateStringValue(
+      "<unknown ManagedTileBinPriority value>"));
 }
 
 ManagedTileState::ManagedTileState()
     : raster_mode(LOW_QUALITY_RASTER_MODE),
       gpu_memmgr_stats_bin(NEVER_BIN),
       resolution(NON_IDEAL_RESOLUTION),
       required_for_activation(false),
       time_to_needed_in_seconds(std::numeric_limits<float>::infinity()),
       distance_to_visible_in_pixels(std::numeric_limits<float>::infinity()),
       visible_and_ready_to_draw(false),
       scheduled_priority(0) {
-  for (int i = 0; i < NUM_TREES; ++i) {
+  for (int i = 0; i <= MAX_TREE; ++i) {
     tree_bin[i] = NEVER_BIN;
     bin[i] = NEVER_BIN;
   }
 }
 
 ManagedTileState::TileVersion::TileVersion()
     : mode_(RESOURCE_MODE),
       has_text_(false) {
 }
 
 ManagedTileState::TileVersion::~TileVersion() {
   DCHECK(!resource_);
 }
 
 bool ManagedTileState::TileVersion::IsReadyToDraw() const {
   switch (mode_) {
     case RESOURCE_MODE:
       return !!resource_;
     case SOLID_COLOR_MODE:
     case PICTURE_PILE_MODE:
       return true;
-    default:
-      NOTREACHED();
-      return false;
   }
+  NOTREACHED();
+  return false;
 }
 
 size_t ManagedTileState::TileVersion::GPUMemoryUsageInBytes() const {
   if (!resource_)
     return 0;
   return resource_->bytes();
 }
 
 ManagedTileState::~ManagedTileState() {
 }
@@ skipping 18 matching lines @@
   state->SetBoolean(
       "is_transparent",
       tile_versions[raster_mode].mode_ == TileVersion::SOLID_COLOR_MODE &&
           !SkColorGetA(tile_versions[raster_mode].solid_color_));
   state->SetInteger("scheduled_priority", scheduled_priority);
   return state.PassAs<base::Value>();
 }
 
 }  // namespace cc