<AWT Dev> Add mutter as a window manager.
Omair Majid
omajid at redhat.com
Tue Apr 10 08:03:58 PDT 2012
Hi Anthony,
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/
Given Artem's comments, though, I not sure what to do here.
Thanks,
Omair
More information about the awt-dev
mailing list