[Extensions] Remove principalsPrivate API

[Extensions] Remove principalsPrivate API

BUG=595168
Committed: https://crrev.com/dc99ab106086dcb7cd12542ac669c9bd4b8fe198
Cr-Commit-Position: refs/heads/master@{#381591}

[Devlin, 2016-03-16 05:01:26]

Patchset #1 (id:1) has been deleted

[Devlin, 2016-03-16 05:01:39]

rdevlin.cronin@chromium.org changed reviewers:
+ asargent@chromium.org

[asargent_no_longer_on_chrome, 2016-03-16 22:17:25]

lgtm w/ 1 question

https://codereview.chromium.org/1806723003/diff/20001/chrome/test/data/extens...
File chrome/test/data/extensions/api_test/bindings/function_interceptions.html
(right):

https://codereview.chromium.org/1806723003/diff/20001/chrome/test/data/extens...
chrome/test/data/extensions/api_test/bindings/function_interceptions.html:5: var
objects = ['runtime', 'require', 'test', 'binding', 'Binding', '$set',
'exports', 'requireNative', 'requireAsync', 'privates'];
was this intended to be part of this CL?

[Devlin, 2016-03-16 22:58:01]

https://codereview.chromium.org/1806723003/diff/20001/chrome/test/data/extens...
File chrome/test/data/extensions/api_test/bindings/function_interceptions.html
(right):

https://codereview.chromium.org/1806723003/diff/20001/chrome/test/data/extens...
chrome/test/data/extensions/api_test/bindings/function_interceptions.html:5: var
objects = ['runtime', 'require', 'test', 'binding', 'Binding', '$set',
'exports', 'requireNative', 'requireAsync', 'privates'];
On 2016/03/16 22:17:24, Antony Sargent wrote:
> was this intended to be part of this CL?

Most definitely not. Thanks :)

[Devlin, 2016-03-16 22:58:04]

The CQ bit was checked by rdevlin.cronin@chromium.org

[Devlin, 2016-03-16 22:58:05]

The patchset sent to the CQ was uploaded after l-g-t-m from
asargent@chromium.org
Link to the patchset: https://codereview.chromium.org/1806723003/#ps40001
(title: "Remove extraneous mods")

[commit-bot: I haz the power, 2016-03-16 22:58:36]

CQ is trying da patch. Follow status at
 https://chromium-cq-status.appspot.com/patch-status/1806723003/40001
View timeline at
 https://chromium-cq-status.appspot.com/patch-timeline/1806723003/40001

[commit-bot: I haz the power, 2016-03-16 23:47:52]

Committed patchset #2 (id:40001)

[commit-bot: I haz the power, 2016-03-16 23:50:00]

Description was changed from

==========
[Extensions] Remove principalsPrivate API

BUG=595168
==========

to

==========
[Extensions] Remove principalsPrivate API

BUG=595168

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

[commit-bot: I haz the power, 2016-03-16 23:50:01]

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