Add a non-const APIPermissionSet to Extension to be manipulated during initialization.

Add a non-const APIPermissionSet to Extension to be manipulated during initialization.

This makes it possible for ManifestHandlers to read and modify the initial API permissions.

BUG=159265
Committed: https://src.chromium.org/viewvc/chrome?view=rev&revision=182796

[Yoyo Zhou, 2013-02-14 00:40:47]

cc: Devlin, Joe FYI

[Matt Perry, 2013-02-14 00:47:00]

LGTM. Though couldn't you also just pass api_permissions_ to
ManifestHandler::Parse?

[Yoyo Zhou, 2013-02-14 00:49:51]

On 2013/02/14 00:47:00, Matt Perry wrote:
> LGTM. Though couldn't you also just pass api_permissions_ to
> ManifestHandler::Parse?

I could, but it's somewhat uglier when (right now) none of the existing handlers
do anything with api_permissions.

[commit-bot: I haz the power, 2013-02-14 01:18:59]

CQ is trying da patch. Follow status at
https://chromium-status.appspot.com/cq/yoz@chromium.org/12256031/10002

[commit-bot: I haz the power, 2013-02-14 01:43:57]

Retried try job too often on linux_chromeos for step(s) aura_unittests,
base_unittests, browser_tests, cacheinvalidation_unittests, check_deps,
chromeos_unittests, components_unittests, content_browsertests,
content_unittests, crypto_unittests, dbus_unittests, device_unittests,
gpu_unittests, interactive_ui_tests, ipc_tests, jingle_unittests,
media_unittests, net_unittests, ppapi_unittests, printing_unittests,
sandbox_linux_unittests, sql_unittests, sync_unit_tests, unit_tests
http://build.chromium.org/p/tryserver.chromium/buildstatus?builder=linux_chro...

[commit-bot: I haz the power, 2013-02-14 02:36:39]

CQ is trying da patch. Follow status at
https://chromium-status.appspot.com/cq/yoz@chromium.org/12256031/10002

[commit-bot: I haz the power, 2013-02-14 05:38:18]

Retried try job too often on mac_rel for step(s) browser_tests
http://build.chromium.org/p/tryserver.chromium/buildstatus?builder=mac_rel&nu...

[commit-bot: I haz the power, 2013-02-14 16:16:25]

CQ is trying da patch. Follow status at
https://chromium-status.appspot.com/cq/yoz@chromium.org/12256031/10002

[commit-bot: I haz the power, 2013-02-14 22:27:33]

CQ is trying da patch. Follow status at
https://chromium-status.appspot.com/cq/yoz@chromium.org/12256031/10002

[commit-bot: I haz the power, 2013-02-15 12:24:39]

Retried try job too often on win7_aura for step(s) browser_tests
http://build.chromium.org/p/tryserver.chromium/buildstatus?builder=win7_aura&...

[commit-bot: I haz the power, 2013-02-15 19:48:43]

CQ is trying da patch. Follow status at
https://chromium-status.appspot.com/cq/yoz@chromium.org/12256031/10002

[commit-bot: I haz the power, 2013-02-15 19:50:30]

Change committed as 182796