Blimp: change logger to use switch/case for types.

blimp/client/feature/render_widget_feature.cc

 // Copyright 2015 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 "blimp/client/feature/render_widget_feature.h"
 
 #include "base/numerics/safe_conversions.h"
 #include "blimp/common/create_blimp_message.h"
 #include "blimp/common/proto/blimp_message.pb.h"
 #include "blimp/common/proto/compositor.pb.h"
@@ skipping 82 matching lines @@
 
   switch (message->feature_case()) {
     case BlimpMessage::kRenderWidget:
       ProcessRenderWidgetMessage(delegate, message->render_widget());
       break;
     case BlimpMessage::kCompositor:
       ProcessCompositorMessage(delegate, message->compositor());
       break;
     default:
       NOTREACHED();
+      break;
   }
 
   callback.Run(net::OK);
 }
 
 void RenderWidgetFeature::ProcessRenderWidgetMessage(
     RenderWidgetFeatureDelegate* delegate,
     const RenderWidgetMessage& message) {
   int render_widget_id = message.render_widget_id();
 
   switch (message.type()) {
     case RenderWidgetMessage::CREATED:
       delegate->OnRenderWidgetCreated(render_widget_id);
       break;
     case RenderWidgetMessage::INITIALIZE:
       delegate->OnRenderWidgetInitialized(render_widget_id);
       break;
     case RenderWidgetMessage::DELETED:
       delegate->OnRenderWidgetDeleted(render_widget_id);
       break;
+    case RenderWidgetMessage::UNKNOWN:
+      NOTREACHED();
+      break;
   }
 }
 
 void RenderWidgetFeature::ProcessCompositorMessage(
     RenderWidgetFeatureDelegate* delegate,
     const CompositorMessage& message) {
   int render_widget_id = message.render_widget_id();
 
   std::unique_ptr<cc::proto::CompositorMessage> payload(
       new cc::proto::CompositorMessage);
   if (payload->ParseFromString(message.payload())) {
     delegate->OnCompositorMessageReceived(render_widget_id,
                                           std::move(payload));
   }
 }
 
 RenderWidgetFeature::RenderWidgetFeatureDelegate*
 RenderWidgetFeature::FindDelegate(const int tab_id) {
   DelegateMap::const_iterator it = delegates_.find(tab_id);
   if (it != delegates_.end())
     return it->second;
   return nullptr;
 }
 
 }  // namespace client
 }  // namespace blimp