cros: Remove unneeded includes from update_library.h plus some random cleanups.

cros: Remove unneeded includes from update_library.h plus some random cleanups.

BUG=None
TEST=None

R=stevenjb@chromium.org

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

[stevenjb, 2011-08-05 00:01:42]

http://codereview.chromium.org/7575001/diff/1/chrome/browser/chromeos/cros/up...
File chrome/browser/chromeos/cros/update_library.cc (right):

http://codereview.chromium.org/7575001/diff/1/chrome/browser/chromeos/cros/up...
chrome/browser/chromeos/cros/update_library.cc:19: Init();
We should make Init() virtual for this class also (or at least move the
EnsueLoaded() check to Init())

[tfarina, 2011-08-05 00:03:53]

http://codereview.chromium.org/7575001/diff/1/chrome/browser/chromeos/cros/up...
File chrome/browser/chromeos/cros/update_library.cc (right):

http://codereview.chromium.org/7575001/diff/1/chrome/browser/chromeos/cros/up...
chrome/browser/chromeos/cros/update_library.cc:19: Init();
On 2011/08/05 00:01:42, Steven Bennetts wrote:
> We should make Init() virtual for this class also (or at least move the
> EnsueLoaded() check to Init())

Sure, I'll do. It was in my TODO list. I was planning to do in a follow up patch
though.

[tfarina, 2011-08-05 00:22:18]

On 2011/08/05 00:03:53, tfarina wrote:
>
http://codereview.chromium.org/7575001/diff/1/chrome/browser/chromeos/cros/up...
> File chrome/browser/chromeos/cros/update_library.cc (right):
> 
>
http://codereview.chromium.org/7575001/diff/1/chrome/browser/chromeos/cros/up...
> chrome/browser/chromeos/cros/update_library.cc:19: Init();
> On 2011/08/05 00:01:42, Steven Bennetts wrote:
> > We should make Init() virtual for this class also (or at least move the
> > EnsueLoaded() check to Init())
> 
> Sure, I'll do. It was in my TODO list. I was planning to do in a follow up
patch
> though.

Do you mind if I make the Init() change in another patch?

[tfarina, 2011-08-05 04:10:22]

On Thu, Aug 4, 2011 at 9:24 PM,  <stevenjb@chromium.org> wrote:
> That's fine. LGTM.
>
>
> http://codereview.chromium.org/7575001/
>
Some how this didn't appear in rietveld. But I'm going to land as I
got it through email.


-- 
Thiago