mash: Prerequisites for removing ShelfDelegate.

ash/shelf/shelf_application_menu_model.cc

 // Copyright 2017 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 "ash/shelf/shelf_application_menu_model.h"
 
 #include <stddef.h>
 
 #include <limits>
 #include <utility>
@@ skipping 39 matching lines @@
 bool ShelfApplicationMenuModel::IsCommandIdChecked(int command_id) const {
   return false;
 }
 
 bool ShelfApplicationMenuModel::IsCommandIdEnabled(int command_id) const {
   return command_id >= 0 && static_cast<size_t>(command_id) < items_.size();
 }
 
 void ShelfApplicationMenuModel::ExecuteCommand(int command_id,
                                                int event_flags) {
-  DCHECK(delegate_);
   DCHECK(IsCommandIdEnabled(command_id));
   // Have the delegate execute its own custom command id for the given item.
-  delegate_->ExecuteCommand(items_[command_id]->command_id, event_flags);
+  if (delegate_)
+    delegate_->ExecuteCommand(items_[command_id]->command_id, event_flags);
   RecordMenuItemSelectedMetrics(command_id, items_.size());
 }
 
 void ShelfApplicationMenuModel::RecordMenuItemSelectedMetrics(
     int command_id,
     int num_menu_items_enabled) {
   UMA_HISTOGRAM_COUNTS_100("Ash.Shelf.Menu.SelectedMenuItemIndex", command_id);
   UMA_HISTOGRAM_COUNTS_100("Ash.Shelf.Menu.NumItemsEnabledUponSelection",
                            num_menu_items_enabled);
 }
 
 }  // namespace ash