Used oneof in blimp_message.proto

blimp/client/feature/navigation_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/navigation_feature.h"
 
 #include <map>
 #include <string>
 
 #include "blimp/common/create_blimp_message.h"
@@ skipping 84 matching lines @@
   navigation_message->set_type(NavigationMessage::GO_BACK);
 
   outgoing_message_processor_->ProcessMessage(std::move(blimp_message),
                                               net::CompletionCallback());
 }
 
 void NavigationFeature::ProcessMessage(
     std::unique_ptr<BlimpMessage> message,
     const net::CompletionCallback& callback) {
   DCHECK(!callback.is_null());
-  DCHECK(message->type() == BlimpMessage::NAVIGATION);
+  DCHECK(message->has_navigation());

[Kevin M, 2016/05/02 17:47:47]

Check case

[shaktisahu, 2016/05/16 20:19:02]

Done.

 
   int tab_id = message->target_tab_id();
-  DCHECK(message->has_navigation());
   const NavigationMessage& navigation_message = message->navigation();
 
   NavigationFeatureDelegate* delegate = FindDelegate(tab_id);
   DCHECK(delegate) << "NavigationFeatureDelegate not found for tab " << tab_id;
   switch (navigation_message.type()) {
     case NavigationMessage::NAVIGATION_STATE_CHANGED: {
       const NavigationStateChangeMessage& details =
           navigation_message.navigation_state_changed();
       if (details.has_url())
         delegate->OnUrlChanged(tab_id, GURL(details.url()));
@@ skipping 29 matching lines @@
 NavigationFeature::NavigationFeatureDelegate* NavigationFeature::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