Display an app's short name in the app launcher

ui/app_list/app_list_item_model.h

 // 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.
 
 #ifndef UI_APP_LIST_APP_LIST_ITEM_MODEL_H_
 #define UI_APP_LIST_APP_LIST_ITEM_MODEL_H_
 
 #include <string>
 
 #include "base/basictypes.h"
@@ skipping 13 matching lines @@
 // and action to be executed when the AppListItemView is activated.
 class APP_LIST_EXPORT AppListItemModel {
  public:
   AppListItemModel();
   virtual ~AppListItemModel();
 
   void SetIcon(const gfx::ImageSkia& icon, bool has_shadow);
   const gfx::ImageSkia& icon() const { return icon_; }
   bool has_shadow() const { return has_shadow_; }
 
   void SetTitle(const std::string& title);

[tapted, 2013/08/30 01:51:29]

What do people think about changing this to

void SetTitle(const std::string& title, const std::string& full_name); ?

(or SetName(..))

I think they are always set together, and this would eliminate some inconsistent
states. E.g. Currently SetTitle is calling UpdateTooltip, but accessing the
full_name, which won't have been set yet. You then also won't need to update the
observer interface.

[tmdiep, 2013/08/30 02:48:13]

Agreed, I think this is a good idea.

   const std::string& title() const { return title_; }
 
+  void SetFullName(const std::string& full_name);
+  const std::string& full_name() const { return full_name_; }
+
   void SetHighlighted(bool highlighted);
   bool highlighted() const { return highlighted_; }
 
   void SetIsInstalling(bool is_installing);
   bool is_installing() const { return is_installing_; }
 
   void SetPercentDownloaded(int percent_downloaded);
   int percent_downloaded() const { return percent_downloaded_; }
 
   void set_app_id(const std::string& app_id) { app_id_ = app_id; }
   const std::string& app_id() { return app_id_; }
 
   void AddObserver(AppListItemModelObserver* observer);
   void RemoveObserver(AppListItemModelObserver* observer);
 
   // Returns the context menu model for this item, or NULL if there is currently
   // no menu for the item (e.g. during install).
   // Note the returned menu model is owned by this item.
   virtual ui::MenuModel* GetContextMenuModel();
 
  private:
   gfx::ImageSkia icon_;
   bool has_shadow_;
   std::string title_;
+  std::string full_name_;
   bool highlighted_;
   bool is_installing_;
   int percent_downloaded_;
   std::string app_id_;
 
   ObserverList<AppListItemModelObserver> observers_;
 
   DISALLOW_COPY_AND_ASSIGN(AppListItemModel);
 };
 
 }  // namespace app_list
 
 #endif  // UI_APP_LIST_APP_LIST_ITEM_MODEL_H_