<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