Drag and drop between app list and launcher - First patch

chrome/browser/ui/app_list/extension_app_item.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/ui/app_list/extension_app_item.h"
 
 #include "base/prefs/pref_service.h"
 #include "chrome/app/chrome_command_ids.h"
 #include "chrome/browser/extensions/context_menu_matcher.h"
 #include "chrome/browser/extensions/extension_prefs.h"
@@ skipping 187 matching lines @@
 }
 
 bool ExtensionAppItem::HasOverlay() const {
   return !is_platform_app_ && extension_id_ != extension_misc::kChromeAppId;
 }
 
 void ExtensionAppItem::Reload() {
   const Extension* extension = GetExtension();
   bool is_installing = !extension;
   SetIsInstalling(is_installing);
+  set_app_id(extension->id());

[koz (OOO until 15th September), 2013/05/03 01:19:51]

This should be set_app_id(extension_id_) as extension may be NULL here. ie: when
the app is installing there is no associated Extension* yet, but we do know its
id.

[Mr4D (OOO till 08-26), 2013/05/07 04:31:01]

Done.

   if (is_installing) {
     SetTitle(extension_name_);
     UpdateIcon();
     return;
   }
   SetTitle(extension->name());
   LoadImage(extension);
 }
 
 syncer::StringOrdinal ExtensionAppItem::GetPageOrdinal() const {
@@ skipping 358 matching lines @@
     context_menu_model_->AddItemWithStringId(DETAILS,
                                              IDS_NEW_TAB_APP_DETAILS);
     context_menu_model_->AddItemWithStringId(UNINSTALL,
                                              is_platform_app_ ?
                                                  IDS_APP_LIST_UNINSTALL_ITEM :
                                                  IDS_EXTENSIONS_UNINSTALL);
   }
 
   return context_menu_model_.get();
 }