Discard pinning an app with non-empty launcher id.

chrome/browser/ui/ash/chrome_launcher_prefs.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 CHROME_BROWSER_UI_ASH_CHROME_LAUNCHER_PREFS_H_
 #define CHROME_BROWSER_UI_ASH_CHROME_LAUNCHER_PREFS_H_
 
 #include <memory>
 #include <string>
 #include <vector>
 
 #include "ash/public/cpp/shelf_types.h"
 #include "base/macros.h"
 #include "components/sync_preferences/pref_service_syncable_observer.h"
 
 class LauncherControllerHelper;
 class PrefService;
 class Profile;
 
-namespace base {
-class DictionaryValue;
-}
-
 namespace sync_preferences {
 class PrefServiceSyncable;
 }
 
 namespace user_prefs {
 class PrefRegistrySyncable;
 }
 
 namespace ash {
 namespace launcher {
 
+class AppLauncherId;
+
 // Path within the dictionary entries in the prefs::kPinnedLauncherApps list
 // specifying the extension ID of the app to be pinned by that entry.
 extern const char kPinnedAppsPrefAppIDPath[];
 
 extern const char kPinnedAppsPrefPinnedByPolicy[];
 
 // Value used as a placeholder in the list of pinned applications.
 // This is NOT a valid extension identifier so pre-M31 versions ignore it.
 extern const char kPinnedAppsPlaceholder[];
 
 // Values used for prefs::kShelfAutoHideBehavior.
 extern const char kShelfAutoHideBehaviorAlways[];
 extern const char kShelfAutoHideBehaviorNever[];
 
 // Values used for prefs::kShelfAlignment.
 extern const char kShelfAlignmentBottom[];
 extern const char kShelfAlignmentLeft[];
 extern const char kShelfAlignmentRight[];
 
-// A unique chrome launcher id used to identify a shelf item. This class is a
-// wrapper for the chrome launcher identifier. |app_launcher_id_| includes the
-// |app_id| and the |launch_id|. The |app_id| is the application id associated
-// with a set of windows. The |launch_id| is an id that can be passed to an app
-// when launched in order to support multiple shelf items per app. This id is
-// used together with the |app_id| to uniquely identify each shelf item that
-// has the same |app_id|. The |app_id| must not be empty.
-class AppLauncherId {
- public:
-  AppLauncherId(const std::string& app_id, const std::string& launch_id);
-  // Creates an AppLauncherId with an empty |launch_id|.
-  explicit AppLauncherId(const std::string& app_id);
-  // Empty constructor for pre-allocating.
-  AppLauncherId();
-  ~AppLauncherId();
-
-  AppLauncherId(const AppLauncherId& app_launcher_id) = default;
-  AppLauncherId(AppLauncherId&& app_launcher_id) = default;
-  AppLauncherId& operator=(const AppLauncherId& other) = default;
-
-  std::string ToString() const;
-  const std::string& app_id() const { return app_id_; }
-  const std::string& launch_id() const { return launch_id_; }
-
-  bool operator<(const AppLauncherId& other) const;
-
- private:
-  // The application id associated with a set of windows.
-  std::string app_id_;
-  // An id that can be passed to an app when launched in order to support
-  // multiple shelf items per app.
-  std::string launch_id_;
-};
-
 void RegisterChromeLauncherUserPrefs(
     user_prefs::PrefRegistrySyncable* registry);
 
-std::unique_ptr<base::DictionaryValue> CreateAppDict(
-    const AppLauncherId& app_launcher_id);
-
 // Get or set the shelf auto hide behavior preference for a particular display.
 ShelfAutoHideBehavior GetShelfAutoHideBehaviorPref(PrefService* prefs,
                                                    int64_t display_id);
 void SetShelfAutoHideBehaviorPref(PrefService* prefs,
                                   int64_t display_id,
                                   ShelfAutoHideBehavior behavior);
 
 // Get or set the shelf alignment preference for a particular display.
 ShelfAlignment GetShelfAlignmentPref(PrefService* prefs, int64_t display_id);
 void SetShelfAlignmentPref(PrefService* prefs,
                            int64_t display_id,
                            ShelfAlignment alignment);
 
 // Get the list of pinned apps from preferences.
 std::vector<AppLauncherId> GetPinnedAppsFromPrefs(
     const PrefService* prefs,
     LauncherControllerHelper* helper);
 
-// Removes information about pin position from sync model for the app.
+// Removes information about pin position from sync model for the app. Note,
+// |app_launcher_id| with non-empty launch_id is not supported.
 void RemovePinPosition(Profile* profile, const AppLauncherId& app_launcher_id);
 
 // Updates information about pin position in sync model for the app
 // |app_launcher_id|. |app_launcher_id_before| optionally specifies an app that
 // exists right before the target app. |app_launcher_ids_after| optionally
 // specifies sorted by position apps that exist right after the target app.
+// Note, |app_launcher_id| with non-empty launch_id is not supported.
 void SetPinPosition(Profile* profile,
                     const AppLauncherId& app_launcher_id,
                     const AppLauncherId& app_launcher_id_before,
                     const std::vector<AppLauncherId>& app_launcher_ids_after);
 
 // Used to propagate remote preferences to local during the first run.
 class ChromeLauncherPrefsObserver
     : public sync_preferences::PrefServiceSyncableObserver {
  public:
   // Creates and returns an instance of ChromeLauncherPrefsObserver if the
@@ skipping 14 matching lines @@
   // Profile prefs. Not owned.
   sync_preferences::PrefServiceSyncable* prefs_;
 
   DISALLOW_COPY_AND_ASSIGN(ChromeLauncherPrefsObserver);
 };
 
 }  // namespace launcher
 }  // namespace ash
 
 #endif  // CHROME_BROWSER_UI_ASH_CHROME_LAUNCHER_PREFS_H_