[Bug awt/28571] addNotify() on a Frame w/ Canvas destroys the Canvas pref size

[mark at gcc dot gnu dot org]

------- Comment #3 from mark at gcc dot gnu dot org  2006-08-02 23:02 -------
Reverting the following patch makes the testcase work.

2006-07-13  Roman Kennke  <address@hidden>

        * java/awt/Component.java
        (DEFAULT_MAX_SIZE): New static constant.
        (preferredSize): Return copy of the actual value computed
        by new helper method.
        (preferredSizeImpl): New helper method. Adds locking and
        correct handling of cached value.
        (minimumSize): Return copy of the actual value computed
        by new helper method.
        (minimumSizeImpl): New helper method. Adds locking and
        correct handling of cached value.
        (getMaximumSize):  Return copy of the actual value computed
        by new helper method.
        (maximumSizeImpl): New helper method. Adds locking and
        correct handling of cached value.
        (invalidate): Correct handling of cached layout information.
        Added locking.
        * java/awt/Container.java
        (preferredSize): Minimized locking. Corrected handling of cached
        values. Return copy of real value.
        (minimumSize): Minimized locking. Corrected handling of cached
        values. Return copy of real value.
        (getMaximumSize): Minimized locking. Corrected handling of cached
        values. Return copy of real value.

Could you take a look Roman?


-- 

mark at gcc dot gnu dot org changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
         AssignedTo|unassigned at gcc dot gnu   |roman at kennke dot org
                   |dot org                     |
             Status|NEW                         |ASSIGNED


http://gcc.gnu.org/bugzilla/show_bug.cgi?id=28571