Mac: Support compositor in fullscreen windows.

Mac: Support compositor in fullscreen windows.

It worked just fine, but since the window wasn't opaque, the underlayed surface didn't show through.

BUG=53391
TEST=See bug

[viettrungluu, 2010-08-26 03:12:53]

http://codereview.chromium.org/3199019/diff/1/3
File chrome/browser/cocoa/fullscreen_window.mm (right):

http://codereview.chromium.org/3199019/diff/1/3#newcode106
chrome/browser/cocoa/fullscreen_window.mm:106: - (void)underlaySurfaceAdded {
That's some pretty bad cutting-and-pasting going on there. Maybe we should
rename ChromeEventProcessingWindow, or add something else into (the middle of)
the class hierarchy, where things like this can go?

[Nico, 2010-08-26 03:18:11]

On Wed, Aug 25, 2010 at 8:12 PM,  <viettrungluu@chromium.org> wrote:
>
> http://codereview.chromium.org/3199019/diff/1/3
> File chrome/browser/cocoa/fullscreen_window.mm (right):
>
> http://codereview.chromium.org/3199019/diff/1/3#newcode106
> chrome/browser/cocoa/fullscreen_window.mm:106: -
> (void)underlaySurfaceAdded {
> That's some pretty bad cutting-and-pasting going on there. Maybe we
> should rename ChromeEventProcessingWindow,

Can't do that, we don't want this for the extension bubbles.

> or add something else into
> (the middle of) the class hierarchy, where things like this can go?

That seems like overkill to me.

>
> http://codereview.chromium.org/3199019/show
>

[Nico, 2010-08-26 03:20:42]

On Wed, Aug 25, 2010 at 8:18 PM, Nico Weber <thakis@chromium.org> wrote:
> On Wed, Aug 25, 2010 at 8:12 PM,  <viettrungluu@chromium.org> wrote:
>>
>> http://codereview.chromium.org/3199019/diff/1/3
>> File chrome/browser/cocoa/fullscreen_window.mm (right):
>>
>> http://codereview.chromium.org/3199019/diff/1/3#newcode106
>> chrome/browser/cocoa/fullscreen_window.mm:106: -
>> (void)underlaySurfaceAdded {
>> That's some pretty bad cutting-and-pasting going on there. Maybe we
>> should rename ChromeEventProcessingWindow,
>
> Can't do that, we don't want this for the extension bubbles.
>
>> or add something else into
>> (the middle of) the class hierarchy, where things like this can go?
>
> That seems like overkill to me.

Sounds like rohitrao wants to rehierarchize fullscreenwindow anyway,
he agreed to merge this CL in. Hooray!

>
>>
>> http://codereview.chromium.org/3199019/show
>>
>