Move ChromeShellPKCS11AuthenticationManager to Empty...

Move ChromeShellPKCS11AuthenticationManager to Empty...

Change the name and move it to a more generic place so that it can be used
by ChromeMobileApplication.

BUG=428860
Committed: https://crrev.com/9bf2589b5e25e120b7b15fac2d28208f5e037e41
Cr-Commit-Position: refs/heads/master@{#324381}

[Changwan Ryu, 2015-04-07 07:53:43]

changwan@chromium.org changed reviewers:
+ newt@chromium.org, yfriedman@chromium.org

[Yaron, 2015-04-07 13:45:14]

lgtm

[newt (away), 2015-04-07 17:47:45]

Is there a subpackage we could put this file in? It seems much too specialized
to belong in the root Java package.

[Changwan Ryu, 2015-04-08 02:53:14]

On 2015/04/07 17:47:45, newt wrote:
> Is there a subpackage we could put this file in? It seems much too specialized
> to belong in the root Java package.

Yaron, any suggestion for the name of this subpackage?

My candidates are smartcard / oemsupport / externalauth / oemauth.

[Yaron, 2015-04-08 13:12:54]

On 2015/04/08 02:53:14, Changwan Ryu wrote:
> On 2015/04/07 17:47:45, newt wrote:
> > Is there a subpackage we could put this file in? It seems much too
specialized
> > to belong in the root Java package.
> 
> Yaron, any suggestion for the name of this subpackage?
> 
> My candidates are smartcard / oemsupport / externalauth / oemauth.

I guess go with smartcard. Note that you'd probably want to move
PKCS11AuthenticationManager.java as well

[Changwan Ryu, 2015-04-09 00:44:17]

The CQ bit was checked by changwan@chromium.org

[Changwan Ryu, 2015-04-09 00:44:17]

The patchset sent to the CQ was uploaded after l-g-t-m from
yfriedman@chromium.org
Link to the patchset: https://codereview.chromium.org/1063033002/#ps40001
(title: " ")

[commit-bot: I haz the power, 2015-04-09 00:47:37]

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

[commit-bot: I haz the power, 2015-04-09 02:07:50]

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

[commit-bot: I haz the power, 2015-04-09 02:07:51]

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

[Changwan Ryu, 2015-04-09 03:05:38]

The CQ bit was checked by changwan@chromium.org

[commit-bot: I haz the power, 2015-04-09 03:06:25]

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

[commit-bot: I haz the power, 2015-04-09 03:58:40]

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

[commit-bot: I haz the power, 2015-04-09 03:58:40]

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

[Changwan Ryu, 2015-04-09 04:50:18]

The CQ bit was checked by changwan@chromium.org

[Changwan Ryu, 2015-04-09 04:50:18]

The patchset sent to the CQ was uploaded after l-g-t-m from
yfriedman@chromium.org
Link to the patchset: https://codereview.chromium.org/1063033002/#ps60001
(title: "fixed findbug error")

[commit-bot: I haz the power, 2015-04-09 04:50:38]

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

[commit-bot: I haz the power, 2015-04-09 05:18:55]

Committed patchset #4 (id:60001)

[commit-bot: I haz the power, 2015-04-09 05:19:39]

Patchset 4 (id:??) landed as
https://crrev.com/9bf2589b5e25e120b7b15fac2d28208f5e037e41
Cr-Commit-Position: refs/heads/master@{#324381}