[Ozone] Add default case otherwise compiler complains about missing return

content/browser/compositor/overlay_candidate_validator_ozone.cc

 // Copyright 2014 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/browser/compositor/overlay_candidate_validator_ozone.h"
 
 #include "ui/gfx/ozone/overlay_candidates_ozone.h"
 
 namespace content {
 
 static gfx::SurfaceFactoryOzone::BufferFormat GetOzoneFormat(
     cc::ResourceFormat overlay_format) {
   switch (overlay_format) {
     case cc::RGBA_8888:
       return gfx::SurfaceFactoryOzone::RGBA_8888;
     case cc::RGBA_4444:
     case cc::BGRA_8888:
     case cc::LUMINANCE_8:
     case cc::RGB_565:
-    case cc::ETC1: {
+    case cc::ETC1:
+    default:

[danakj, 2014/04/03 14:25:34]

instead of default, which will take away compile-time checks for enum coverage,
how about just move the NOTREACHED/return out of the switch statement to below
it?

[dnicoara, 2014/04/03 14:34:34]

Done.

       NOTREACHED();
       return gfx::SurfaceFactoryOzone::UNKNOWN;
-    }
   }
 }
 
 OverlayCandidateValidatorOzone::OverlayCandidateValidatorOzone(
     gfx::AcceleratedWidget widget,
     gfx::OverlayCandidatesOzone* overlay_candidates)
     : widget_(widget), overlay_candidates_(overlay_candidates) {}
 
 OverlayCandidateValidatorOzone::~OverlayCandidateValidatorOzone() {}
 
@@ skipping 13 matching lines @@
 
   overlay_candidates_->CheckOverlaySupport(&ozone_surface_list);
   DCHECK_EQ(surfaces->size(), ozone_surface_list.size());
 
   for (size_t i = 0; i < surfaces->size(); i++) {
     surfaces->at(i).overlay_handled = ozone_surface_list.at(i).overlay_handled;
   }
 }
 
 }  // namespace content