Reverting r133134 - Make sure that base::MessagePumpForUI from different modules are isolated from …

Reverting r133134 - Make sure that base::MessagePumpForUI from different modules are isolated from each other and add protection from shatter attacks by placing |this| pointer to the used data associated with the message-only window (instead of blindly trusting the value of WPARAM).

We suspect that r133134 might be causing issues like 124091, so it needs to be reworked.

BUG=124091, 127933
TBR=alexeypa@chromium.org

Committed: https://src.chromium.org/viewvc/chrome?view=rev&revision=137542

[alexeypa (please no reviews), 2012-05-16 20:22:36]

Please take a look.

[Wez, 2012-05-16 21:01:34]

On 2012/05/16 20:22:36, alexeypa wrote:
> Please take a look.

Is this a straight revert of the original CL, in which case you can just
drover-land it?

[alexeypa (please no reviews), 2012-05-16 21:07:12]

On 2012/05/16 21:01:34, Wez wrote:
> On 2012/05/16 20:22:36, alexeypa wrote:
> > Please take a look.
> 
> Is this a straight revert of the original CL, in which case you can just
> drover-land it?

No, I had to resolve conflicts in base/message_pump_win.cc.

[Wez, 2012-05-16 21:13:55]

LGTM, FWIW

[jar (doing other things), 2012-05-16 22:13:29]

lgtm