Update SkCoInitialize to better match spec.

SkAutoCoInitialize isn't as useful of an abstraction as it could be. If COM is already initialized, most users will think things failed. Instead, change things up so that it just says if COM is initialized, since we should never care what the actual threading model is.

[bsalomon, 2011-09-16 14:52:49]

LGTM

http://codereview.appspot.com/5036044/diff/1/src/ports/SkImageDecoder_WIC.cpp
File src/ports/SkImageDecoder_WIC.cpp (right):

http://codereview.appspot.com/5036044/diff/1/src/ports/SkImageDecoder_WIC.cpp...
src/ports/SkImageDecoder_WIC.cpp:29: if (!scopedCo.succeeded()) return false;
nit: style is to do:
if (!scopedCo.succeeded()) {
    return false;
}

http://codereview.appspot.com/5036044/diff/1/src/ports/SkImageDecoder_WIC.cpp...
src/ports/SkImageDecoder_WIC.cpp:195: if (!scopedCo.succeeded()) return false;
same nit

[bungeman, 2011-09-16 15:37:59]

Committed revision 2274.