Remove unneeded NSS headers.

Remove unneeded NSS headers and the base::EnsureNSSInit() call.

R=cmasone
BUG=none
TEST=no compilation errors.

Committed: http://src.chromium.org/viewvc/chrome?view=rev&revision=69340

[wtc, 2010-12-14 21:32:48]

I see no NSS types or functions in this file, so I believe
this file no longer needs to include NSS headers.

Compilation succeeded on the linux_chromeos trybot:
http://build.chromium.org/p/tryserver.chromium/builders/linux_chromeos/builds...

http://codereview.chromium.org/5763006/diff/1/chrome/browser/chromeos/login/o...
File chrome/browser/chromeos/login/owner_key_utils.cc (right):

http://codereview.chromium.org/5763006/diff/1/chrome/browser/chromeos/login/o...
chrome/browser/chromeos/login/owner_key_utils.cc:98: base::EnsureNSSInit();
I suspect this base::EnsureNSSInit() call can also be
deleted.  If NSS needs to be initialized, that should be
ensured by the class that calls NSS functions directly
(I guess that would be RSAPrivateKey).

http://codereview.chromium.org/5763006/diff/1/chrome/browser/chromeos/login/o...
chrome/browser/chromeos/login/owner_key_utils.cc:151: // Get the file size (must
fit in a 32 bit int for NSS).
Just wondering: is this because NSS functions use the
'unsigned int' type for buffer lengths?

[Chris Masone, 2010-12-14 22:21:52]

LGTM, w/nit

http://codereview.chromium.org/5763006/diff/1/chrome/browser/chromeos/login/o...
File chrome/browser/chromeos/login/owner_key_utils.cc (right):

http://codereview.chromium.org/5763006/diff/1/chrome/browser/chromeos/login/o...
chrome/browser/chromeos/login/owner_key_utils.cc:98: base::EnsureNSSInit();
On 2010/12/14 21:32:49, wtc wrote:
> I suspect this base::EnsureNSSInit() call can also be
> deleted.  If NSS needs to be initialized, that should be
> ensured by the class that calls NSS functions directly
> (I guess that would be RSAPrivateKey).

That makes sense to me; I wouldn't mind if you took it out while you're here :-)

http://codereview.chromium.org/5763006/diff/1/chrome/browser/chromeos/login/o...
chrome/browser/chromeos/login/owner_key_utils.cc:151: // Get the file size (must
fit in a 32 bit int for NSS).
On 2010/12/14 21:32:49, wtc wrote:
> Just wondering: is this because NSS functions use the
> 'unsigned int' type for buffer lengths?

yes

[wtc, 2010-12-15 18:11:55]

OK, I removed the base::EnsureNSSInit() call.  I will
check this in if it passes the linux_chromeos trybot.

[Chris Masone, 2010-12-15 21:41:18]

On 2010/12/15 18:11:55, wtc wrote:
> OK, I removed the base::EnsureNSSInit() call.  I will
> check this in if it passes the linux_chromeos trybot.

assuming it does, lgtm