Fix USB device permissions tests

Fix USB device permissions tests

BUG=629223
R=tbarzic, reillyg@chromium.org, meacer@chromium.org

Committed: https://crrev.com/438efe78ab6e2caa5d5a63967aea4b6544831dc8
Cr-Commit-Position: refs/heads/master@{#434312}

[Lof, 2016-11-23 13:31:44]

Tests passed with warning: 
https://cs.chromium.org/chromium/src/extensions/common/manifest_handlers/perm...

[tbarzic, 2016-11-23 20:32:05]

lgtm

[Reilly Grant (use Gerrit), 2016-11-23 20:40:46]

lgtm but do we understand why these tests passed before?

[tbarzic, 2016-11-23 20:54:33]

On 2016/11/23 at 20:40:46, reillyg wrote:
> lgtm but do we understand why these tests passed before?

Yes - the permission data that was actually tested was not retrieved from the
app manifest data - it was created from the value passed into CreateTestApp.
The app itself is only used to determine whether
allow_usb_devices_permission_interface_class behavior feature is available (when
creating check params).

[Lof, 2016-11-23 21:40:01]

The CQ bit was checked by lof84@yandex-team.ru

[commit-bot: I haz the power, 2016-11-23 21:40:42]

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-11-23 23:42:24]

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

[commit-bot: I haz the power, 2016-11-23 23:42:25]

Try jobs failed on following builders:
  linux_chromium_clobber_rel_ng on master.tryserver.chromium.linux
(JOB_TIMED_OUT, no build URL)
  linux_chromium_compile_dbg_ng on master.tryserver.chromium.linux
(JOB_TIMED_OUT, no build URL)

[Lof, 2016-11-24 04:43:29]

The CQ bit was checked by lof84@yandex-team.ru

[commit-bot: I haz the power, 2016-11-24 04:43:51]

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

[Lof, 2016-11-24 06:42:06]

The CQ bit was unchecked by lof84@yandex-team.ru

[Lof, 2016-11-24 06:52:46]

The CQ bit was checked by lof84@yandex-team.ru

[commit-bot: I haz the power, 2016-11-24 06:53:16]

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-11-24 08:54:25]

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

[commit-bot: I haz the power, 2016-11-24 08:54:26]

Try jobs failed on following builders:
  chromium_presubmit on master.tryserver.chromium.linux (JOB_TIMED_OUT, no build
URL)

[Lof, 2016-11-24 10:05:32]

The CQ bit was checked by lof84@yandex-team.ru

[commit-bot: I haz the power, 2016-11-24 10:05:51]

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-11-24 12:05:15]

CQ is committing da patch.

Bot data: {"patchset_id": 1, "attempt_start_ts": 1479981932858640, "parent_rev":
"92eeea7822f42323c3ced8c23fbfe130adaeb1cb", "commit_rev":
"f7e443ae8b06f4a9c9a81e75bcf8559585e42090"}

[commit-bot: I haz the power, 2016-11-24 12:05:44]

Committed patchset #1 (id:1)

[commit-bot: I haz the power, 2016-11-24 12:08:17]

Description was changed from

==========
Fix USB device permissions tests

BUG=629223
R=tbarzic, reillyg@chromium.org, meacer@chromium.org
==========

to

==========
Fix USB device permissions tests

BUG=629223
R=tbarzic, reillyg@chromium.org, meacer@chromium.org

Committed: https://crrev.com/438efe78ab6e2caa5d5a63967aea4b6544831dc8
Cr-Commit-Position: refs/heads/master@{#434312}
==========

[commit-bot: I haz the power, 2016-11-24 12:08:18]

Patchset 1 (id:??) landed as
https://crrev.com/438efe78ab6e2caa5d5a63967aea4b6544831dc8
Cr-Commit-Position: refs/heads/master@{#434312}