Remove knowledge of USB devices from permission prompt implementations.

Remove knowledge of USB devices from permission prompt implementations.

This change hides all the details of USB device objects inside of the
DevicePermissionsPrompt::Prompt object so that the Views and OS X
implementations do not need to be modified when this prompting logic is
extended to support HID devices as well.

BUG=457899
Committed: https://crrev.com/99ad34756e26e13a5252556a2df57397d5d8e20e
Cr-Commit-Position: refs/heads/master@{#325761}

[Reilly Grant (use Gerrit), 2015-04-16 22:50:41]

reillyg@chromium.org changed reviewers:
+ avi@chromium.org, pkasting@chromium.org

[Reilly Grant (use Gerrit), 2015-04-16 22:51:16]

Peter, please take a look at chrome/browser/ui/views.
Avi, please take a look at chrome/browser/ui/cocoa.

[Avi (use Gerrit), 2015-04-16 22:59:38]

Cocoa lgtm

[Peter Kasting, 2015-04-16 23:02:18]

c/b/ui/views LGTM

[Reilly Grant (use Gerrit), 2015-04-16 23:06:23]

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

[Reilly Grant (use Gerrit), 2015-04-16 23:06:41]

Devlin, please take a look at extensions/browser/api.

[Devlin, 2015-04-17 19:39:25]

lgtm

https://codereview.chromium.org/1097603003/diff/1/extensions/browser/api/devi...
File extensions/browser/api/device_permissions_prompt.h (right):

https://codereview.chromium.org/1097603003/diff/1/extensions/browser/api/devi...
extensions/browser/api/device_permissions_prompt.h:109: Delegate* delegate_;
init to null?

[Reilly Grant (use Gerrit), 2015-04-17 20:05:03]

https://codereview.chromium.org/1097603003/diff/1/extensions/browser/api/devi...
File extensions/browser/api/device_permissions_prompt.h (right):

https://codereview.chromium.org/1097603003/diff/1/extensions/browser/api/devi...
extensions/browser/api/device_permissions_prompt.h:109: Delegate* delegate_;
On 2015/04/17 19:39:25, Devlin wrote:
> init to null?

Done.

[Reilly Grant (use Gerrit), 2015-04-17 20:05:14]

The CQ bit was checked by reillyg@chromium.org

[Reilly Grant (use Gerrit), 2015-04-17 20:05:14]

The patchset sent to the CQ was uploaded after l-g-t-m from avi@chromium.org,
pkasting@chromium.org, rdevlin.cronin@chromium.org
Link to the patchset: https://codereview.chromium.org/1097603003/#ps20001
(title: "Initialize delegate_ to nullptr.")

[commit-bot: I haz the power, 2015-04-17 20:05:49]

CQ is trying da patch. Follow status at
 https://chromium-cq-status.appspot.com/patch-status/1097603003/20001

[commit-bot: I haz the power, 2015-04-17 21:00:54]

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

[commit-bot: I haz the power, 2015-04-17 21:00:55]

Try jobs failed on following builders:
  linux_android_rel_ng on tryserver.chromium.linux (JOB_FAILED,
http://build.chromium.org/p/tryserver.chromium.linux/builders/linux_android_r...)

[Reilly Grant (use Gerrit), 2015-04-17 22:59:45]

The CQ bit was checked by reillyg@chromium.org

[commit-bot: I haz the power, 2015-04-17 23:00:50]

CQ is trying da patch. Follow status at
 https://chromium-cq-status.appspot.com/patch-status/1097603003/20001

[commit-bot: I haz the power, 2015-04-18 02:11:54]

Committed patchset #2 (id:20001)

[commit-bot: I haz the power, 2015-04-18 02:13:22]

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