Discard pinning an app with non-empty launcher id.

chrome/browser/ui/ash/app_launcher_id.h

Index: chrome/browser/ui/ash/app_launcher_id.h
diff --git a/chrome/browser/ui/ash/app_launcher_id.h b/chrome/browser/ui/ash/app_launcher_id.h
new file mode 100644
index 0000000000000000000000000000000000000000..dcdcc6ff715be4b35d01797f99214d2716d1783a
--- /dev/null
+++ b/chrome/browser/ui/ash/app_launcher_id.h
@@ -0,0 +1,47 @@
+// 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.
+
+#ifndef CHROME_BROWSER_UI_ASH_APP_LAUNCHER_ID_H_
+#define CHROME_BROWSER_UI_ASH_APP_LAUNCHER_ID_H_
+
+#include <string>
+
+namespace ash {
+namespace launcher {

[James Cook, 2017/02/09 01:26:53]

Is there any particular reason you put this in "namespace launcher"?  That
namespace is only used in ChromeLauncherPrefs and probably shouldn't exist at
all. Taking this out of namespace launcher would make some identifiers shorter.

[khmel, 2017/02/09 01:30:22]

There is no particular reason for this (kept from previous impl). (IIUC this
change is due rebase)

+
+// 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;
+
+  const std::string& app_id() const { return app_id_; }
+  const std::string& launch_id() const { return launch_id_; }
+
+ 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_;
+};
+
+}  // namespace launcher
+}  // namespace ash
+
+#endif  // CHROME_BROWSER_UI_ASH_APP_LAUNCHER_ID_H_