Remove static initializer for TLS object. This moves the TLS to the

Remove static initializer for TLS object. This moves the TLS to the
PluginGlobals. I did not do a general sultion so that anybody can add to the
globals TLS object because this may be the only place we need this.

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

[brettw, 2012-01-27 22:11:53]

This is to remove a static initializer I added in my MessageLoop patch.

[yzshen1, 2012-01-27 22:32:19]

lgtm

[commit-bot: I haz the power, 2012-02-04 22:44:50]

CQ is trying da patch. Follow status at
https://chromium-status.appspot.com/cq/brettw@chromium.org/9296017/7001

[commit-bot: I haz the power, 2012-02-04 22:44:51]

Can't apply patch for file ppapi/proxy/ppb_message_loop_proxy.cc.
While running patch -p0 --forward --force;
patching file ppapi/proxy/ppb_message_loop_proxy.cc
Hunk #2 FAILED at 26.
Hunk #6 succeeded at 226 (offset 1 line).
1 out of 6 hunks FAILED -- saving rejects to file
ppapi/proxy/ppb_message_loop_proxy.cc.rej

[commit-bot: I haz the power, 2012-02-04 23:18:52]

CQ is trying da patch. Follow status at
https://chromium-status.appspot.com/cq/brettw@chromium.org/9296017/18001

[commit-bot: I haz the power, 2012-02-05 03:07:40]

Change committed as 120500

[Nico, 2012-02-05 03:13:37]

http://codereview.chromium.org/9296017/diff/18001/ppapi/proxy/ppb_message_loo...
File ppapi/proxy/ppb_message_loop_proxy.cc (left):

http://codereview.chromium.org/9296017/diff/18001/ppapi/proxy/ppb_message_loo...
ppapi/proxy/ppb_message_loop_proxy.cc:29: static
base::ThreadLocalStorage::StaticSlot tls_slot = TLS_INITIALIZER;
I think this stopped needing a static initializer after http://crrev.com/119679.

[brettw, 2012-02-05 03:33:21]

http://codereview.chromium.org/9296017/diff/18001/ppapi/proxy/ppb_message_loo...
File ppapi/proxy/ppb_message_loop_proxy.cc (left):

http://codereview.chromium.org/9296017/diff/18001/ppapi/proxy/ppb_message_loo...
ppapi/proxy/ppb_message_loop_proxy.cc:29: static
base::ThreadLocalStorage::StaticSlot tls_slot = TLS_INITIALIZER;
On 2012/02/05 03:13:37, Nico wrote:
> I think this stopped needing a static initializer after
http://crrev.com/119679.

Yeah, I did this patch actually before you wrote yours, and I kept it because I
like the new design better since it avoids the statics altogether.