1318218 - Remove PermissionsTable

Status: RESOLVED FIXED

(Core :: DOM: Core & HTML, defect)

Description

[(no longer active)]

This also removes the WebIDE UI for viewing the permissions table.

Comment 1

[(no longer active)]

Attachment: Remove PermissionsTable

Comment 2

[J. Ryan Stinnett [:jryans] (Use needinfo, replies may be slow)]

Comment on attachment 8811564 [details] [diff] [review]
Remove PermissionsTable

Review of attachment 8811564 [details] [diff] [review]:
-----------------------------------------------------------------

Thanks for doing this clean up!

::: devtools/server/actors/device.js
@@ +56,5 @@
>      let dataURL = canvas.toDataURL("image/png");
>      return new LongStringActor(this.conn, dataURL);
>    },
>  
>    getRawPermissionsTable: function () {

Seems like this whole method can be removed, doesn't seem to have much purpose anymore.

You should remove the `getRawPermissionsTable` part from http://searchfox.org/mozilla-central/source/devtools/shared/specs/device.js#15 (which marks the methods that go over the wire to devices).

Comment 3

[Pulsebot]

Pushed by [e-mail]:
https://hg.mozilla.org/integration/mozilla-inbound/rev/592927e727fd
Remove PermissionsTable; r=baku,jryans

Comment 5

[Ryan VanderMeulen [:RyanVM]]

https://hg.mozilla.org/mozilla-central/rev/592927e727fd

Comment 6

[areya]

https://hg.mozilla.org/mozilla-central/rev/592927e727fd