Discard pinning an app with non-empty launcher id.

[Merge M57] Discard pinning an app with non-empty launcher id.

Currently we don't support persisting Chrome shelf pin for app with
custom launcher id. We should discard the code that stores pins for
such apps in app list sync service which is not compatible with its
design.

TEST=tests passed and manually on device various pin scenarios
BUG=689772
TBR=stevenjb@chromium.org
NOTRY=true
NOPRESUBMIT=true

Review-Url: https://codereview.chromium.org/2684853002
Cr-Commit-Position: refs/heads/master@{#449126}
(cherry picked from commit 613cdf8eeeaac4830b55c4b55b4377269faa2e35)

Review-Url: https://codereview.chromium.org/2691043002
Cr-Commit-Position: refs/branch-heads/2987@{#476}
Cr-Branched-From: ad51088c0e8776e8dcd963dbe752c4035ba6dab6-refs/heads/master@{#444943}
Committed: https://chromium.googlesource.com/chromium/src/+/1f9e48f79d14ac701260a68a60cddbeeb89e6f9a

[khmel, 2017-02-13 16:25:48]

Description was changed from

==========
Discard pinning an app with non-empty launcher id.

Currently we don't support persisting Chrome shelf pin for app with
custom launcher id. We should discard the code that stores pins for
such apps in app list sync service which is not compatible with its
design.

TEST=tests passed and manually on device various pin scenarios
BUG=689772

Review-Url: https://codereview.chromium.org/2684853002
Cr-Commit-Position: refs/heads/master@{#449126}
(cherry picked from commit 613cdf8eeeaac4830b55c4b55b4377269faa2e35)
==========

to

==========
[Merge M57] Discard pinning an app with non-empty launcher id.

Currently we don't support persisting Chrome shelf pin for app with
custom launcher id. We should discard the code that stores pins for
such apps in app list sync service which is not compatible with its
design.

TEST=tests passed and manually on device various pin scenarios
BUG=689772

TBR=stevenjb@chromium.org
NOTRY=true
NOPRESUBMIT=true

Review-Url: https://codereview.chromium.org/2684853002
Cr-Commit-Position: refs/heads/master@{#449126}
(cherry picked from commit 613cdf8eeeaac4830b55c4b55b4377269faa2e35)
==========

[khmel, 2017-02-13 16:52:44]

The CQ bit was checked by khmel@chromium.org

[commit-bot: I haz the power, 2017-02-13 16:53:27]

CQ is trying da patch. Follow status at
 
https://chromium-cq-status.appspot.com/v2/patch-status/codereview.chromium.or...

[commit-bot: I haz the power, 2017-02-13 16:56:39]

CQ is committing da patch.

Bot data: {"patchset_id": 1, "attempt_start_ts": 1487004764671260, "parent_rev":
"eb1631705160cba7c83a70393aba982719f60726", "commit_rev":
"9be7ffd436e6e6237cde747890e97b2803b80c64"}

[commit-bot: I haz the power, 2017-02-13 17:00:25]

CQ is committing da patch.

Bot data: {"patchset_id": 1, "attempt_start_ts": 1487004764671260, "parent_rev":
"eb1631705160cba7c83a70393aba982719f60726", "commit_rev":
"9be7ffd436e6e6237cde747890e97b2803b80c64"}

[commit-bot: I haz the power, 2017-02-13 17:03:17]

The CQ bit was unchecked by commit-bot@chromium.org

[commit-bot: I haz the power, 2017-02-13 17:03:19]

Prior attempt to commit was detected, but we were not able to check whether the
issue was successfully committed. Please check Git history manually and re-check
CQ or close this issue as needed.

[khmel, 2017-02-13 17:31:29]

The CQ bit was checked by khmel@chromium.org

[commit-bot: I haz the power, 2017-02-13 17:32:29]

CQ is trying da patch. Follow status at
 
https://chromium-cq-status.appspot.com/v2/patch-status/codereview.chromium.or...

[commit-bot: I haz the power, 2017-02-13 17:34:17]

CQ is committing da patch.

Bot data: {"patchset_id": 1, "attempt_start_ts": 1487007089732940, "parent_rev":
"9664f8e1c301f24157b0b17ac33466a0efa53d7c", "commit_rev":
"1f9e48f79d14ac701260a68a60cddbeeb89e6f9a"}

[commit-bot: I haz the power, 2017-02-13 17:34:51]

Description was changed from

==========
[Merge M57] Discard pinning an app with non-empty launcher id.

Currently we don't support persisting Chrome shelf pin for app with
custom launcher id. We should discard the code that stores pins for
such apps in app list sync service which is not compatible with its
design.

TEST=tests passed and manually on device various pin scenarios
BUG=689772

TBR=stevenjb@chromium.org
NOTRY=true
NOPRESUBMIT=true

Review-Url: https://codereview.chromium.org/2684853002
Cr-Commit-Position: refs/heads/master@{#449126}
(cherry picked from commit 613cdf8eeeaac4830b55c4b55b4377269faa2e35)
==========

to

==========
[Merge M57] Discard pinning an app with non-empty launcher id.

Currently we don't support persisting Chrome shelf pin for app with
custom launcher id. We should discard the code that stores pins for
such apps in app list sync service which is not compatible with its
design.

TEST=tests passed and manually on device various pin scenarios
BUG=689772

TBR=stevenjb@chromium.org
NOTRY=true
NOPRESUBMIT=true

Review-Url: https://codereview.chromium.org/2684853002
Cr-Commit-Position: refs/heads/master@{#449126}
(cherry picked from commit 613cdf8eeeaac4830b55c4b55b4377269faa2e35)

Review-Url: https://codereview.chromium.org/2691043002
Cr-Commit-Position: refs/branch-heads/2987@{#476}
Cr-Branched-From:
ad51088c0e8776e8dcd963dbe752c4035ba6dab6-refs/heads/master@{#444943}
Committed:
https://chromium.googlesource.com/chromium/src/+/1f9e48f79d14ac701260a68a60cd...
==========

[commit-bot: I haz the power, 2017-02-13 17:34:53]

Committed patchset #1 (id:1) as
https://chromium.googlesource.com/chromium/src/+/1f9e48f79d14ac701260a68a60cd...