Whitelist the APIs for chromeos/onhub recovery untility extensions.

Whitelist the APIs for chromeos/onhub recovery untility extensions.

The following APIs are whitelisted:
- imageWriterPrivate
- chromeosInfoPrivate
- the feedback API
- fileSystem

BUG=642141
CQ_INCLUDE_TRYBOTS=master.tryserver.chromium.linux:closure_compilation

Committed: https://crrev.com/2f83bcda82ac58b93a793660e93fa1ce38ea6dac
Cr-Commit-Position: refs/heads/master@{#418487}

[Michael Tang, 2016-08-31 05:03:53]

Description was changed from

==========
Whitelist the APIs for chromeos/onhub recovery untility extensions.

The following APIs are whitelisted:
- imageWriterPrivate
- chromeosInfoPrivate
- the feedback API
- fileSystem

BUG=642141
==========

to

==========
Whitelist the APIs for chromeos/onhub recovery untility extensions.

The following APIs are whitelisted:
- imageWriterPrivate
- chromeosInfoPrivate
- the feedback API
- fileSystem

BUG=642141
CQ_INCLUDE_TRYBOTS=master.tryserver.chromium.linux:closure_compilation
==========

[Michael Tang, 2016-08-31 05:08:45]

Description was changed from

==========
Whitelist the APIs for chromeos/onhub recovery untility extensions.

The following APIs are whitelisted:
- imageWriterPrivate
- chromeosInfoPrivate
- the feedback API
- fileSystem

BUG=642141
CQ_INCLUDE_TRYBOTS=master.tryserver.chromium.linux:closure_compilation
==========

to

==========
Whitelist the APIs for chromeos/onhub recovery untility extensions.

The following APIs are whitelisted:
- imageWriterPrivate
- chromeosInfoPrivate
- the feedback API
- fileSystem

BUG=642141
CQ_INCLUDE_TRYBOTS=master.tryserver.chromium.linux:closure_compilation
==========

[Michael Tang, 2016-08-31 05:08:45]

ntang@chromium.org changed reviewers:
+ rkc@chromium.org, stephenlin@chromium.org

[Michael Tang, 2016-09-01 16:26:06]

ntang@chromium.org changed reviewers:
+ asargent@chromium.org

[Michael Tang, 2016-09-07 00:55:50]

ntang@chromium.org changed reviewers:
+ steel@chromium.org

[Rahul Chaturvedi, 2016-09-08 19:45:49]

So I understand that the extension needs to be whitelisted for
'chrome.fileSystem', because though that was available to the recovery app,
since it is now an extension, it needs to be whitelisted to have access. For the
rest of the APIs though, wasn't the recovery app already whitelisted?

[Michael Tang, 2016-09-09 16:56:32]

On 2016/09/08 19:45:49, Rahul Chaturvedi wrote:
> So I understand that the extension needs to be whitelisted for
> 'chrome.fileSystem', because though that was available to the recovery app,
> since it is now an extension, it needs to be whitelisted to have access. For
the
> rest of the APIs though, wasn't the recovery app already whitelisted?

For chrome.fileSystem, that is correct. For the rest of the APIs, they were
whitelisted only for "apps" and not for "extensions". So we need to whitelist
for the new extensiosn. 

Per our conversation offline, I added TODO to remove the whitelisting for "apps"
when they are deprecated.

[Rahul Chaturvedi, 2016-09-09 17:06:27]

https://codereview.chromium.org/2297113002/diff/20001/chrome/browser/resource...
File chrome/browser/resources/feedback/js/event_handler.js (right):

https://codereview.chromium.org/2297113002/diff/20001/chrome/browser/resource...
chrome/browser/resources/feedback/js/event_handler.js:75: // TODO (ntang)
Removes after deprecation.
TODO(ntang): Remove the following hashes by <insert date> at the latest.

https://codereview.chromium.org/2297113002/diff/20001/chrome/common/extension...
File chrome/common/extensions/api/_permission_features.json (right):

https://codereview.chromium.org/2297113002/diff/20001/chrome/common/extension...
chrome/common/extensions/api/_permission_features.json:126: // TODO(ntang)
Remove Chrome Recovery Utility App after deprecation.
Don't use app names - just use the format I specified in the other comment.

[Michael Tang, 2016-09-09 17:20:36]

The CQ bit was checked by ntang@chromium.org to run a CQ dry run

[commit-bot: I haz the power, 2016-09-09 17:20:59]

Dry run: 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-09-09 17:20:59]

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

[commit-bot: I haz the power, 2016-09-09 17:21:01]

Dry run: No L-G-T-M from a valid reviewer yet. 
CQ run can only be started by full committers or once the patch has
received an L-G-T-M from a full committer.
Even if an L-G-T-M may have been provided, it was from a non-committer,
_not_ a full super star committer.
See http://www.chromium.org/getting-involved/become-a-committer
Note that this has nothing to do with OWNERS files.

[Michael Tang, 2016-09-09 17:23:19]

On 2016/09/09 17:06:27, Rahul Chaturvedi wrote:
>
https://codereview.chromium.org/2297113002/diff/20001/chrome/browser/resource...
> File chrome/browser/resources/feedback/js/event_handler.js (right):
> 
>
https://codereview.chromium.org/2297113002/diff/20001/chrome/browser/resource...
> chrome/browser/resources/feedback/js/event_handler.js:75: // TODO (ntang)
> Removes after deprecation.
> TODO(ntang): Remove the following hashes by <insert date> at the latest.
> 
>
https://codereview.chromium.org/2297113002/diff/20001/chrome/common/extension...
> File chrome/common/extensions/api/_permission_features.json (right):
> 
>
https://codereview.chromium.org/2297113002/diff/20001/chrome/common/extension...
> chrome/common/extensions/api/_permission_features.json:126: // TODO(ntang)
> Remove Chrome Recovery Utility App after deprecation.
> Don't use app names - just use the format I specified in the other comment.

updated

[Rahul Chaturvedi, 2016-09-09 18:04:06]

12/31/2017 seems a bit excessive, but fine - we can resolve the actual date
offline.

lgtm

[asargent_no_longer_on_chrome, 2016-09-09 19:04:05]

lgtm

https://codereview.chromium.org/2297113002/diff/40001/chrome/browser/resource...
File chrome/browser/resources/feedback/js/event_handler.js (right):

https://codereview.chromium.org/2297113002/diff/40001/chrome/browser/resource...
chrome/browser/resources/feedback/js/event_handler.js:75: // TODO (ntang)
Removes the following hashes by 12/31/2017.
grammar nit: s/Removes/Remove/

Also, do you mean that we should remove *all* of the hashes following your
comment, or just that hashes that existed before, but we'd want to keep the last
2 you're adding in this CL? If the latter, you might want to revise the comment
to something like "Remove the following 7 hashes ..."

https://codereview.chromium.org/2297113002/diff/40001/chrome/common/extension...
File chrome/common/extensions/api/_permission_features.json (right):

https://codereview.chromium.org/2297113002/diff/40001/chrome/common/extension...
chrome/common/extensions/api/_permission_features.json:126: // TODO (ntang)
Removes the following hashes by 12/31/2017.
same nit for this TODO comment (and elsewhere in this file)

[Michael Tang, 2016-09-09 21:46:22]

On 2016/09/09 19:04:05, Antony Sargent wrote:
> lgtm
> 
>
https://codereview.chromium.org/2297113002/diff/40001/chrome/browser/resource...
> File chrome/browser/resources/feedback/js/event_handler.js (right):
> 
>
https://codereview.chromium.org/2297113002/diff/40001/chrome/browser/resource...
> chrome/browser/resources/feedback/js/event_handler.js:75: // TODO (ntang)
> Removes the following hashes by 12/31/2017.
> grammar nit: s/Removes/Remove/
> 
> Also, do you mean that we should remove *all* of the hashes following your
> comment, or just that hashes that existed before, but we'd want to keep the
last
> 2 you're adding in this CL? If the latter, you might want to revise the
comment
> to something like "Remove the following 7 hashes ..."
> 
>
https://codereview.chromium.org/2297113002/diff/40001/chrome/common/extension...
> File chrome/common/extensions/api/_permission_features.json (right):
> 
>
https://codereview.chromium.org/2297113002/diff/40001/chrome/common/extension...
> chrome/common/extensions/api/_permission_features.json:126: // TODO (ntang)
> Removes the following hashes by 12/31/2017.
> same nit for this TODO comment (and elsewhere in this file)

Updated

[Michael Tang, 2016-09-09 21:47:08]

The CQ bit was checked by ntang@chromium.org

[Michael Tang, 2016-09-09 21:47:08]

The patchset sent to the CQ was uploaded after l-g-t-m from steel@chromium.org,
asargent@chromium.org
Link to the patchset: https://codereview.chromium.org/2297113002/#ps60001
(title: "Whitelist the APIs for chromeos/onhub recovery untility extensions.")

[commit-bot: I haz the power, 2016-09-09 21:47:33]

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

[Michael Tang, 2016-09-09 21:48:26]

The CQ bit was checked by ntang@chromium.org

[commit-bot: I haz the power, 2016-09-09 21:48:54]

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-09-09 21:50:56]

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

[commit-bot: I haz the power, 2016-09-09 21:50:57]

Try jobs failed on following builders:
  ios-device on master.tryserver.chromium.mac (JOB_FAILED,
http://build.chromium.org/p/tryserver.chromium.mac/builders/ios-device/builds...)
  ios-simulator on master.tryserver.chromium.mac (JOB_FAILED,
http://build.chromium.org/p/tryserver.chromium.mac/builders/ios-simulator/bui...)

[Michael Tang, 2016-09-10 00:42:13]

The CQ bit was checked by ntang@chromium.org

[commit-bot: I haz the power, 2016-09-10 00:42:56]

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-09-10 00:45:47]

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

[commit-bot: I haz the power, 2016-09-10 00:45:49]

Try jobs failed on following builders:
  mac_chromium_compile_dbg_ng on master.tryserver.chromium.mac (JOB_FAILED,
http://build.chromium.org/p/tryserver.chromium.mac/builders/mac_chromium_comp...)
  mac_chromium_rel_ng on master.tryserver.chromium.mac (JOB_FAILED,
http://build.chromium.org/p/tryserver.chromium.mac/builders/mac_chromium_rel_...)

[Michael Tang, 2016-09-12 16:35:13]

The CQ bit was checked by ntang@chromium.org

[commit-bot: I haz the power, 2016-09-12 16:35:40]

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-09-12 16:38:02]

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

[commit-bot: I haz the power, 2016-09-12 16:38:04]

Try jobs failed on following builders:
  ios-device on master.tryserver.chromium.mac (JOB_FAILED,
http://build.chromium.org/p/tryserver.chromium.mac/builders/ios-device/builds...)
  mac_chromium_rel_ng on master.tryserver.chromium.mac (JOB_FAILED,
http://build.chromium.org/p/tryserver.chromium.mac/builders/mac_chromium_rel_...)

[Michael Tang, 2016-09-12 22:03:50]

The CQ bit was checked by ntang@chromium.org

[commit-bot: I haz the power, 2016-09-12 22:04:54]

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-09-12 22:08:29]

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

[commit-bot: I haz the power, 2016-09-12 22:08:30]

Try jobs failed on following builders:
  android_clang_dbg_recipe on master.tryserver.chromium.android (JOB_FAILED,
https://build.chromium.org/p/tryserver.chromium.android/builders/android_clan...)
  chromeos_amd64-generic_chromium_compile_only_ng on
master.tryserver.chromium.linux (JOB_FAILED,
http://build.chromium.org/p/tryserver.chromium.linux/builders/chromeos_amd64-...)
  ios-device on master.tryserver.chromium.mac (JOB_FAILED,
http://build.chromium.org/p/tryserver.chromium.mac/builders/ios-device/builds...)
  ios-simulator on master.tryserver.chromium.mac (JOB_FAILED,
http://build.chromium.org/p/tryserver.chromium.mac/builders/ios-simulator/bui...)
  mac_chromium_compile_dbg_ng on master.tryserver.chromium.mac (JOB_FAILED,
http://build.chromium.org/p/tryserver.chromium.mac/builders/mac_chromium_comp...)
  mac_chromium_rel_ng on master.tryserver.chromium.mac (JOB_FAILED,
http://build.chromium.org/p/tryserver.chromium.mac/builders/mac_chromium_rel_...)

[Michael Tang, 2016-09-12 22:09:08]

The CQ bit was checked by ntang@chromium.org

[Michael Tang, 2016-09-12 22:09:08]

The patchset sent to the CQ was uploaded after l-g-t-m from steel@chromium.org,
asargent@chromium.org
Link to the patchset: https://codereview.chromium.org/2297113002/#ps80001
(title: "Whitelist the APIs for chromeos/onhub recovery untility extensions.")

[commit-bot: I haz the power, 2016-09-12 22:09:53]

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-09-12 22:13:39]

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

[commit-bot: I haz the power, 2016-09-12 22:13:43]

Try jobs failed on following builders:
  android_clang_dbg_recipe on master.tryserver.chromium.android (JOB_FAILED,
https://build.chromium.org/p/tryserver.chromium.android/builders/android_clan...)
  chromium_presubmit on master.tryserver.chromium.linux (JOB_FAILED,
http://build.chromium.org/p/tryserver.chromium.linux/builders/chromium_presub...)
  ios-device on master.tryserver.chromium.mac (JOB_FAILED,
http://build.chromium.org/p/tryserver.chromium.mac/builders/ios-device/builds...)
  mac_chromium_compile_dbg_ng on master.tryserver.chromium.mac (JOB_FAILED,
http://build.chromium.org/p/tryserver.chromium.mac/builders/mac_chromium_comp...)
  mac_chromium_rel_ng on master.tryserver.chromium.mac (JOB_FAILED,
http://build.chromium.org/p/tryserver.chromium.mac/builders/mac_chromium_rel_...)

[ntang, 2016-09-13 04:04:55]

The CQ bit was checked by ntang@google.com

[commit-bot: I haz the power, 2016-09-13 04:05:04]

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-09-13 04:07:19]

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

[commit-bot: I haz the power, 2016-09-13 04:07:20]

Try jobs failed on following builders:
  mac_chromium_compile_dbg_ng on master.tryserver.chromium.mac (JOB_FAILED,
http://build.chromium.org/p/tryserver.chromium.mac/builders/mac_chromium_comp...)
  mac_chromium_rel_ng on master.tryserver.chromium.mac (JOB_FAILED,
http://build.chromium.org/p/tryserver.chromium.mac/builders/mac_chromium_rel_...)

[Michael Tang, 2016-09-13 04:16:53]

The CQ bit was checked by ntang@chromium.org

[Michael Tang, 2016-09-13 04:16:54]

The patchset sent to the CQ was uploaded after l-g-t-m from steel@chromium.org,
asargent@chromium.org
Link to the patchset: https://codereview.chromium.org/2297113002/#ps100001
(title: "Whitelist the APIs for chromeos/onhub recovery untility extensions.")

[commit-bot: I haz the power, 2016-09-13 04:17:05]

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-09-13 04:19:10]

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

[commit-bot: I haz the power, 2016-09-13 04:19:12]

Try jobs failed on following builders:
  ios-device on master.tryserver.chromium.mac (JOB_FAILED,
http://build.chromium.org/p/tryserver.chromium.mac/builders/ios-device/builds...)

[Michael Tang, 2016-09-13 22:38:54]

On 2016/09/09 19:04:05, Antony Sargent wrote:
> lgtm
> 
>
https://codereview.chromium.org/2297113002/diff/40001/chrome/browser/resource...
> File chrome/browser/resources/feedback/js/event_handler.js (right):
> 
>
https://codereview.chromium.org/2297113002/diff/40001/chrome/browser/resource...
> chrome/browser/resources/feedback/js/event_handler.js:75: // TODO (ntang)
> Removes the following hashes by 12/31/2017.
> grammar nit: s/Removes/Remove/
> 
> Also, do you mean that we should remove *all* of the hashes following your
> comment, or just that hashes that existed before, but we'd want to keep the
last
> 2 you're adding in this CL? If the latter, you might want to revise the
comment
> to something like "Remove the following 7 hashes ..."
> 
>
https://codereview.chromium.org/2297113002/diff/40001/chrome/common/extension...
> File chrome/common/extensions/api/_permission_features.json (right):
> 
>
https://codereview.chromium.org/2297113002/diff/40001/chrome/common/extension...
> chrome/common/extensions/api/_permission_features.json:126: // TODO (ntang)
> Removes the following hashes by 12/31/2017.
> same nit for this TODO comment (and elsewhere in this file)

Antony & Rahul, Do you know why commit queue always fail for this CL? I did the
rebase and re-upload a few times, still could not get it in? Any suggestion?

[Rahul Chaturvedi, 2016-09-13 22:54:31]

How are you doing the rebase/re-upload? It doesn't look like the code review
tool is seeing your rebase.

[Michael Tang, 2016-09-13 22:58:12]

On 2016/09/13 22:54:31, Rahul Chaturvedi wrote:
> How are you doing the rebase/re-upload? It doesn't look like the code review
> tool is seeing your rebase.

under chromium/src:
git rebase-update
git cl upload

[Michael Tang, 2016-09-13 22:58:45]

The CQ bit was checked by ntang@chromium.org

[Michael Tang, 2016-09-13 22:58:46]

The patchset sent to the CQ was uploaded after l-g-t-m from steel@chromium.org,
asargent@chromium.org
Link to the patchset: https://codereview.chromium.org/2297113002/#ps120001
(title: "Whitelist the APIs for chromeos/onhub recovery untility extensions.")

[asargent_no_longer_on_chrome, 2016-09-13 22:58:56]

On 2016/09/13 22:38:54, Michael Tang wrote:
> On 2016/09/09 19:04:05, Antony Sargent wrote:
> > lgtm
> > 
> >
>
https://codereview.chromium.org/2297113002/diff/40001/chrome/browser/resource...
> > File chrome/browser/resources/feedback/js/event_handler.js (right):
> > 
> >
>
https://codereview.chromium.org/2297113002/diff/40001/chrome/browser/resource...
> > chrome/browser/resources/feedback/js/event_handler.js:75: // TODO (ntang)
> > Removes the following hashes by 12/31/2017.
> > grammar nit: s/Removes/Remove/
> > 
> > Also, do you mean that we should remove *all* of the hashes following your
> > comment, or just that hashes that existed before, but we'd want to keep the
> last
> > 2 you're adding in this CL? If the latter, you might want to revise the
> comment
> > to something like "Remove the following 7 hashes ..."
> > 
> >
>
https://codereview.chromium.org/2297113002/diff/40001/chrome/common/extension...
> > File chrome/common/extensions/api/_permission_features.json (right):
> > 
> >
>
https://codereview.chromium.org/2297113002/diff/40001/chrome/common/extension...
> > chrome/common/extensions/api/_permission_features.json:126: // TODO (ntang)
> > Removes the following hashes by 12/31/2017.
> > same nit for this TODO comment (and elsewhere in this file)
> 
> Antony & Rahul, Do you know why commit queue always fail for this CL? I did
the
> rebase and re-upload a few times, still could not get it in? Any suggestion?

Hmm, you can see the patch failures for one of the builders at:

https://build.chromium.org/p/tryserver.chromium.linux/builders/linux_chromium...

Looks like a real error - are you sure you resolved the merge properly? If you
create a new test branch off of origin/master and then try doing 'git cl patch
2297113002' to patch in the changes from this CL to it, do they apply cleanly to
your test branch?

[commit-bot: I haz the power, 2016-09-13 22:59:26]

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-09-13 23:03:06]

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

[commit-bot: I haz the power, 2016-09-13 23:03:07]

Try jobs failed on following builders:
  linux_chromium_compile_dbg_ng on master.tryserver.chromium.linux (JOB_FAILED,
http://build.chromium.org/p/tryserver.chromium.linux/builders/linux_chromium_...)
  ios-device on master.tryserver.chromium.mac (JOB_FAILED,
http://build.chromium.org/p/tryserver.chromium.mac/builders/ios-device/builds...)
  ios-simulator on master.tryserver.chromium.mac (JOB_FAILED,
http://build.chromium.org/p/tryserver.chromium.mac/builders/ios-simulator/bui...)
  mac_chromium_compile_dbg_ng on master.tryserver.chromium.mac (JOB_FAILED,
http://build.chromium.org/p/tryserver.chromium.mac/builders/mac_chromium_comp...)
  mac_chromium_rel_ng on master.tryserver.chromium.mac (JOB_FAILED,
http://build.chromium.org/p/tryserver.chromium.mac/builders/mac_chromium_rel_...)

[Michael Tang, 2016-09-14 00:03:59]

The CQ bit was checked by ntang@chromium.org

[Michael Tang, 2016-09-14 00:03:59]

The patchset sent to the CQ was uploaded after l-g-t-m from steel@chromium.org,
asargent@chromium.org
Link to the patchset: https://codereview.chromium.org/2297113002/#ps140001
(title: "rebase and merge from origin/master")

[commit-bot: I haz the power, 2016-09-14 00:04:38]

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-09-14 02:23:45]

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

[commit-bot: I haz the power, 2016-09-14 02:23:47]

Try jobs failed on following builders:
  mac_chromium_rel_ng on master.tryserver.chromium.mac (JOB_FAILED,
http://build.chromium.org/p/tryserver.chromium.mac/builders/mac_chromium_rel_...)

[ntang, 2016-09-14 03:19:01]

The CQ bit was checked by ntang@google.com

[commit-bot: I haz the power, 2016-09-14 03:19:19]

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-09-14 04:34:26]

Description was changed from

==========
Whitelist the APIs for chromeos/onhub recovery untility extensions.

The following APIs are whitelisted:
- imageWriterPrivate
- chromeosInfoPrivate
- the feedback API
- fileSystem

BUG=642141
CQ_INCLUDE_TRYBOTS=master.tryserver.chromium.linux:closure_compilation
==========

to

==========
Whitelist the APIs for chromeos/onhub recovery untility extensions.

The following APIs are whitelisted:
- imageWriterPrivate
- chromeosInfoPrivate
- the feedback API
- fileSystem

BUG=642141
CQ_INCLUDE_TRYBOTS=master.tryserver.chromium.linux:closure_compilation
==========

[commit-bot: I haz the power, 2016-09-14 04:34:28]

Committed patchset #8 (id:140001)

[commit-bot: I haz the power, 2016-09-14 04:37:51]

Description was changed from

==========
Whitelist the APIs for chromeos/onhub recovery untility extensions.

The following APIs are whitelisted:
- imageWriterPrivate
- chromeosInfoPrivate
- the feedback API
- fileSystem

BUG=642141
CQ_INCLUDE_TRYBOTS=master.tryserver.chromium.linux:closure_compilation
==========

to

==========
Whitelist the APIs for chromeos/onhub recovery untility extensions.

The following APIs are whitelisted:
- imageWriterPrivate
- chromeosInfoPrivate
- the feedback API
- fileSystem

BUG=642141
CQ_INCLUDE_TRYBOTS=master.tryserver.chromium.linux:closure_compilation

Committed: https://crrev.com/2f83bcda82ac58b93a793660e93fa1ce38ea6dac
Cr-Commit-Position: refs/heads/master@{#418487}
==========

[commit-bot: I haz the power, 2016-09-14 04:37:52]

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