Cleanup: Change a static initializer to a LazyInstance, and add some typedefs.

Cleanup: Change a static initializer to a LazyInstance, and add some typedefs.

BUG=none
TEST=none
Committed: http://src.chromium.org/viewvc/chrome?view=rev&revision=107824

[willchan no longer on Chromium, 2011-10-28 21:42:24]

LGTM

http://codereview.chromium.org/8413052/diff/1/base/mime_util_xdg.cc
File base/mime_util_xdg.cc (right):

http://codereview.chromium.org/8413052/diff/1/base/mime_util_xdg.cc#newcode32
base/mime_util_xdg.cc:32: static base::LazyInstance<base::Lock>
I wonder if you should use LeakyLazyInstance. Not sure if we need to delete this
guy.

[Lei Zhang, 2011-10-28 21:54:02]

http://codereview.chromium.org/8413052/diff/1/base/mime_util_xdg.cc
File base/mime_util_xdg.cc (right):

http://codereview.chromium.org/8413052/diff/1/base/mime_util_xdg.cc#newcode32
base/mime_util_xdg.cc:32: static base::LazyInstance<base::Lock>
On 2011/10/28 21:42:25, willchan wrote:
> I wonder if you should use LeakyLazyInstance. Not sure if we need to delete
this
> guy.

Would you say that's true of all locks? There's 7 our
base::LazyInstance<base::Lock> instantiations. We can change them all in a
separate CL.

[willchan no longer on Chromium, 2011-10-31 18:56:44]

On 2011/10/28 21:54:02, Lei Zhang wrote:
> http://codereview.chromium.org/8413052/diff/1/base/mime_util_xdg.cc
> File base/mime_util_xdg.cc (right):
> 
> http://codereview.chromium.org/8413052/diff/1/base/mime_util_xdg.cc#newcode32
> base/mime_util_xdg.cc:32: static base::LazyInstance<base::Lock>
> On 2011/10/28 21:42:25, willchan wrote:
> > I wonder if you should use LeakyLazyInstance. Not sure if we need to delete
> this
> > guy.
> 
> Would you say that's true of all locks? There's 7 our
> base::LazyInstance<base::Lock> instantiations. We can change them all in a
> separate CL.

Yes, I think it's true of all locks.