Add support for pinning platform apps

chrome/browser/ui/views/ash/launcher/chrome_launcher_controller.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/views/ash/launcher/chrome_launcher_controller.h"
 
 #include <set>
 #include <vector>
 
 #include "ash/launcher/launcher_model.h"
@@ skipping 173 matching lines @@
 
 void ChromeLauncherController::LauncherItemClosed(ash::LauncherID id) {
   DCHECK(id_to_item_map_.find(id) != id_to_item_map_.end());
   id_to_item_map_.erase(id);
   model_->RemoveItemAt(model_->ItemIndexByID(id));
 }
 
 void ChromeLauncherController::Unpin(ash::LauncherID id) {
   DCHECK(id_to_item_map_.find(id) != id_to_item_map_.end());
   DCHECK(!id_to_item_map_[id].controller);
-  LauncherItemClosed(id);
+
+  if (ShellWindowRegistry::Get(profile_)->GetShellWindowsForApp(
+      id_to_item_map_[id].app_id).size() > 0) {
+    int index = model_->ItemIndexByID(id);
+    ash::LauncherItem item = model_->items()[index];
+    item.type = ash::TYPE_PLATFORM_APP;
+    model_->Set(index, item);
+  } else {
+    LauncherItemClosed(id);
+  }
+  if (CanPin())
+    PersistPinnedState();
+}
+
+void ChromeLauncherController::Pin(ash::LauncherID id) {
+  DCHECK(id_to_item_map_.find(id) != id_to_item_map_.end());
+  DCHECK(!id_to_item_map_[id].controller);
+
+  int index = model_->ItemIndexByID(id);
+  ash::LauncherItem item = model_->items()[index];
+
+  if (item.type != ash::TYPE_PLATFORM_APP)
+    return;
+
+  item.type = ash::TYPE_APP_SHORTCUT;
+  model_->Set(index, item);

[sky, 2012/06/13 21:20:53]

Are we picking up the type right for these when they get restored?

[DaveMoore, 2012/06/13 22:30:16]

I believe so. We only persist shortcuts, not the apps themselves.

On 2012/06/13 21:20:53, sky wrote:

+
   if (CanPin())
     PersistPinnedState();
 }
 
 bool ChromeLauncherController::IsPinned(ash::LauncherID id) {
-  DCHECK(id_to_item_map_.find(id) != id_to_item_map_.end());
-  return id_to_item_map_[id].is_pinned();
+  int index = model_->ItemIndexByID(id);
+  ash::LauncherItemType type = model_->items()[index].type;
+  return type == ash::TYPE_APP_SHORTCUT;
 }
 
 void ChromeLauncherController::TogglePinned(ash::LauncherID id) {
   if (id_to_item_map_.find(id) == id_to_item_map_.end())
     return;  // May happen if item closed with menu open.
 
-  // Only currently support unpinning.
   if (IsPinned(id))
     Unpin(id);
+  else
+    Pin(id);
 }
 
 bool ChromeLauncherController::IsPinnable(ash::LauncherID id) const {
   int index = model_->ItemIndexByID(id);
-  return (index != -1 &&
-          model_->items()[index].type == ash::TYPE_APP_SHORTCUT &&
+  if (index == -1)
+    return false;
+
+  ash::LauncherItemType type = model_->items()[index].type;
+  return ((type == ash::TYPE_APP_SHORTCUT || type == ash::TYPE_PLATFORM_APP) &&
           CanPin());
 }
 
 void ChromeLauncherController::Open(ash::LauncherID id, int event_flags) {
   if (id_to_item_map_.find(id) == id_to_item_map_.end())
     return;  // In case invoked from menu and item closed while menu up.
 
   BrowserLauncherItemController* controller = id_to_item_map_[id].controller;
   if (controller) {
     controller->window()->Show();
@@ skipping 543 matching lines @@
   }
   model_->AddAt(index, item);
 
   if (!controller || controller->type() !=
       BrowserLauncherItemController::TYPE_EXTENSION_PANEL) {
     if (item.status != ash::STATUS_IS_PENDING)
       app_icon_loader_->FetchImage(app_id);
   }
   return id;
 }