Cleanup and optimizing switch using fall through.

chrome/browser/browsing_data/cookies_tree_model.cc

 // Copyright (c) 2012 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 "chrome/browser/browsing_data/cookies_tree_model.h"
 
 #include <algorithm>
 #include <functional>
 #include <map>
 #include <vector>
@@ skipping 115 matching lines @@
   replacements.ClearPort();
   if (url.SchemeIsCryptographic())
     replacements.SetScheme("http", url::Component(0, 4));
 
   return url.GetOrigin().ReplaceComponents(replacements);
 }
 
 #if defined(ENABLE_EXTENSIONS)
 bool TypeIsProtected(CookieTreeNode::DetailedInfo::NodeType type) {
   switch (type) {
-    case CookieTreeNode::DetailedInfo::TYPE_COOKIE:
-      return false;
+    // Fall through each below cases to return true.

[Bernhard Bauer, 2015/10/19 15:27:44]

Leave an empty line before each comment to increase readability, please.

[Deepak, 2015/10/20 02:49:25]

Done. But just after the switch statement and before comment empty line get
removed by 
"git cl format".

     case CookieTreeNode::DetailedInfo::TYPE_DATABASE:
-      return true;
     case CookieTreeNode::DetailedInfo::TYPE_LOCAL_STORAGE:
-      return true;
     case CookieTreeNode::DetailedInfo::TYPE_SESSION_STORAGE:
-      return true;
     case CookieTreeNode::DetailedInfo::TYPE_APPCACHE:
-      return true;
     case CookieTreeNode::DetailedInfo::TYPE_INDEXED_DB:
-      return true;
     case CookieTreeNode::DetailedInfo::TYPE_FILE_SYSTEM:
-      return true;
-    case CookieTreeNode::DetailedInfo::TYPE_QUOTA:
-      return false;
-    case CookieTreeNode::DetailedInfo::TYPE_CHANNEL_ID:
-      return false;
     case CookieTreeNode::DetailedInfo::TYPE_SERVICE_WORKER:
-      return true;
     case CookieTreeNode::DetailedInfo::TYPE_CACHE_STORAGE:
       return true;
+    // Fall through each below cases to return false.
+    case CookieTreeNode::DetailedInfo::TYPE_COOKIE:
+    case CookieTreeNode::DetailedInfo::TYPE_QUOTA:
+    case CookieTreeNode::DetailedInfo::TYPE_CHANNEL_ID:
     case CookieTreeNode::DetailedInfo::TYPE_FLASH_LSO:
       return false;
     default:
       break;
   }
   return false;
 }
 #endif
 
 // This function returns the local data container associated with a leaf tree
@@ skipping 126 matching lines @@
 
 void CookieTreeNode::DeleteStoredObjects() {
   std::for_each(children().begin(),
                 children().end(),
                 std::mem_fun(&CookieTreeNode::DeleteStoredObjects));
 }
 
 CookiesTreeModel* CookieTreeNode::GetModel() const {
   if (parent())
     return parent()->GetModel();
-  else
-    return nullptr;
+  return nullptr;
 }
 
 ///////////////////////////////////////////////////////////////////////////////
 // CookieTreeCookieNode, public:
 
 CookieTreeCookieNode::CookieTreeCookieNode(
     std::list<net::CanonicalCookie>::iterator cookie)
     : CookieTreeNode(base::UTF8ToUTF16(cookie->Name())),
       cookie_(cookie) {
 }
@@ skipping 708 matching lines @@
 }
 
 // Returns the index of the icon to use for |node|. Return -1 to use the
 // default icon. The index is relative to the list of icons returned from
 // GetIcons.
 int CookiesTreeModel::GetIconIndex(ui::TreeModelNode* node) {
   CookieTreeNode* ct_node = static_cast<CookieTreeNode*>(node);
   switch (ct_node->GetDetailedInfo().node_type) {
     case CookieTreeNode::DetailedInfo::TYPE_HOST:
       return ORIGIN;
+    // Fall through each below cases to return COOKIE.
     case CookieTreeNode::DetailedInfo::TYPE_COOKIE:
+    case CookieTreeNode::DetailedInfo::TYPE_CHANNEL_ID:
       return COOKIE;
+    // Fall through each below cases to return DATABASE.
     case CookieTreeNode::DetailedInfo::TYPE_DATABASE:
+    case CookieTreeNode::DetailedInfo::TYPE_LOCAL_STORAGE:
+    case CookieTreeNode::DetailedInfo::TYPE_SESSION_STORAGE:
+    case CookieTreeNode::DetailedInfo::TYPE_APPCACHE:
+    case CookieTreeNode::DetailedInfo::TYPE_INDEXED_DB:
+    case CookieTreeNode::DetailedInfo::TYPE_FILE_SYSTEM:
+    case CookieTreeNode::DetailedInfo::TYPE_SERVICE_WORKER:
+    case CookieTreeNode::DetailedInfo::TYPE_CACHE_STORAGE:
       return DATABASE;
-    case CookieTreeNode::DetailedInfo::TYPE_LOCAL_STORAGE:
-      return DATABASE;  // close enough
-    case CookieTreeNode::DetailedInfo::TYPE_SESSION_STORAGE:
-      return DATABASE;  // ditto
-    case CookieTreeNode::DetailedInfo::TYPE_APPCACHE:
-      return DATABASE;  // ditto
-    case CookieTreeNode::DetailedInfo::TYPE_INDEXED_DB:
-      return DATABASE;  // ditto
-    case CookieTreeNode::DetailedInfo::TYPE_FILE_SYSTEM:
-      return DATABASE;  // ditto
     case CookieTreeNode::DetailedInfo::TYPE_QUOTA:
       return -1;
-    case CookieTreeNode::DetailedInfo::TYPE_CHANNEL_ID:
-      return COOKIE;  // It's kinda like a cookie?
-    case CookieTreeNode::DetailedInfo::TYPE_SERVICE_WORKER:
-      return DATABASE;  // Just like appcache
-    case CookieTreeNode::DetailedInfo::TYPE_CACHE_STORAGE:
-      return DATABASE;  // ditto
     default:
       break;
   }
   return -1;
 }
 
 void CookiesTreeModel::DeleteAllStoredObjects() {
   NotifyObserverBeginBatch();
   CookieTreeNode* root = GetRoot();
   root->DeleteStoredObjects();
@@ skipping 489 matching lines @@
 void CookiesTreeModel::MaybeNotifyBatchesEnded() {
   // Only notify the observers if this is the outermost call to EndBatch() if
   // called in a nested manner.
   if (batches_ended_ == batches_started_ &&
       batches_seen_ == batches_expected_) {
     FOR_EACH_OBSERVER(Observer,
                       cookies_observer_list_,
                       TreeModelEndBatch(this));
   }
 }