<AWT Dev> Add mutter as a window manager.

Anthony Petrov anthony.petrov at oracle.com
Tue Apr 10 09:22:32 PDT 2012


On 4/10/2012 7:03 PM, Omair Majid wrote:
> On 04/09/2012 09:22 AM, Anthony Petrov wrote:
>> Mutter is the direct descendant of Metacity, so there's nothing wrong
>> with it inheriting some "inconvenient" behavior of its parent. Given
>> that Mutter is the standard WM for Gnome 3.0. I'm fine with the fix.
>>
>> A comment regarding the test:
>>
>>>   61         frame.pack();
>>>   62         frame.setSize(500, 500);
>> What's the point of this operations sequence? You should either simply
>> set the desired size, or rely on the pack() alone if the automatically
>> calculated size satisfies you. It just doesn't make sense to do both.
> 
> You are right. I have removed the pack() line.
> 
>> Also, the @bug line in the test should mention a real CR for this issue,
>> I think it is 7043963.
> 
> Ah, I didn't have this number. Fixed now.
> 
> Updated webrev at:
> http://cr.openjdk.java.net/~omajid/mutter-support/02/

Thanks!

> Given Artem's comments, though, I not sure what to do here.

If you can fix this issue in Mutter w/o touching AWT, then this would be 
the best option in my opinion.

--
best regards,
Anthony



More information about the awt-dev mailing list