Deleting the copresence API.

Deleting the copresence API.

BUG=496899
CQ_INCLUDE_TRYBOTS=tryserver.chromium.linux:closure_compilation

Committed: https://crrev.com/cc16886e9facbcb54fa86bbef0dc0e893e36a4fc
Cr-Commit-Position: refs/heads/master@{#404407}

[Charlie, 2016-07-07 15:39:47]

Description was changed from

==========
Deleting the copresence API.

BUG=496899
==========

to

==========
Deleting the copresence API.

BUG=496899
CQ_INCLUDE_TRYBOTS=tryserver.chromium.linux:closure_compilation
==========

[Charlie, 2016-07-07 15:52:59]

ckehoe@chromium.org changed reviewers:
+ rockot@chromium.org, xiyuan@chromium.org

[Charlie, 2016-07-07 15:53:00]

We are finally removing the copresence API from Chrome. It has been replaced by
an all-JavaScript implementation.

This is the first of 3 CLs, one each to remove the copresence, audio_modem, and
copresence_private APIs.

[Ken Rockot(use gerrit already), 2016-07-07 16:09:32]

rs lgtm

[xiyuan, 2016-07-07 16:16:13]

lgtm

https://codereview.chromium.org/2130803002/diff/1/extensions/common/permissio...
File extensions/common/permissions/api_permission.h (right):

https://codereview.chromium.org/2130803002/diff/1/extensions/common/permissio...
extensions/common/permissions/api_permission.h:82: kCopresence,  // Deleted
nit: kCopresence -> kDeletedCopresence so that it is clear this is deleted
without getting back here to see the comment.

[Charlie, 2016-07-07 20:06:16]

ckehoe@chromium.org changed reviewers:
+ brettw@chromium.org, isherman@chromium.org

[Charlie, 2016-07-07 20:06:18]

https://codereview.chromium.org/2130803002/diff/1/extensions/common/permissio...
File extensions/common/permissions/api_permission.h (right):

https://codereview.chromium.org/2130803002/diff/1/extensions/common/permissio...
extensions/common/permissions/api_permission.h:82: kCopresence,  // Deleted
On 2016/07/07 16:16:13, xiyuan wrote:
> nit: kCopresence -> kDeletedCopresence so that it is clear this is deleted
> without getting back here to see the comment.

Done.

[Ilya Sherman, 2016-07-07 20:15:11]

Please also run the scripts to update histograms.xml

[Charlie, 2016-07-07 20:21:10]

On 2016/07/07 20:15:11, Ilya Sherman wrote:
> Please also run the scripts to update histograms.xml

Done.

[Ilya Sherman, 2016-07-07 22:57:30]

histograms.xml and enum changes lgtm

[Charlie, 2016-07-08 03:07:02]

ckehoe@chromium.org changed reviewers:
+ jochen@chromium.org

[jochen (gone - plz use gerrit), 2016-07-08 12:54:36]

lgtm

[Charlie, 2016-07-08 14:17:04]

The CQ bit was checked by ckehoe@chromium.org

[Charlie, 2016-07-08 14:17:05]

The patchset sent to the CQ was uploaded after l-g-t-m from rockot@chromium.org,
xiyuan@chromium.org
Link to the patchset: https://codereview.chromium.org/2130803002/#ps40001
(title: ".")

[commit-bot: I haz the power, 2016-07-08 14:17:57]

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, 2016-07-08 16:42:45]

Committed patchset #3 (id:40001)

[commit-bot: I haz the power, 2016-07-08 16:45:00]

Description was changed from

==========
Deleting the copresence API.

BUG=496899
CQ_INCLUDE_TRYBOTS=tryserver.chromium.linux:closure_compilation
==========

to

==========
Deleting the copresence API.

BUG=496899
CQ_INCLUDE_TRYBOTS=tryserver.chromium.linux:closure_compilation

Committed: https://crrev.com/cc16886e9facbcb54fa86bbef0dc0e893e36a4fc
Cr-Commit-Position: refs/heads/master@{#404407}
==========

[commit-bot: I haz the power, 2016-07-08 16:45:02]

Patchset 3 (id:??) landed as
https://crrev.com/cc16886e9facbcb54fa86bbef0dc0e893e36a4fc
Cr-Commit-Position: refs/heads/master@{#404407}