You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@harmony.apache.org by ay...@apache.org on 2007/03/09 16:05:08 UTC
svn commit: r516424 - in /harmony/enhanced/classlib/trunk/modules/swing/src:
main/java/common/javax/swing/table/DefaultTableCellRenderer.java
test/api/java/common/javax/swing/table/DefaultTableCellRendererTest.java
Author: ayza
Date: Fri Mar 9 07:05:05 2007
New Revision: 516424
URL: http://svn.apache.org/viewvc?view=rev&rev=516424
Log:
Applying patch from HARMONY-1721 ([classlib][swing] javax.swing.table.DefaultTableCellRenderer.getBackground() returns some default value while RI returns null)
Modified:
harmony/enhanced/classlib/trunk/modules/swing/src/main/java/common/javax/swing/table/DefaultTableCellRenderer.java
harmony/enhanced/classlib/trunk/modules/swing/src/test/api/java/common/javax/swing/table/DefaultTableCellRendererTest.java
Modified: harmony/enhanced/classlib/trunk/modules/swing/src/main/java/common/javax/swing/table/DefaultTableCellRenderer.java
URL: http://svn.apache.org/viewvc/harmony/enhanced/classlib/trunk/modules/swing/src/main/java/common/javax/swing/table/DefaultTableCellRenderer.java?view=diff&rev=516424&r1=516423&r2=516424
==============================================================================
--- harmony/enhanced/classlib/trunk/modules/swing/src/main/java/common/javax/swing/table/DefaultTableCellRenderer.java (original)
+++ harmony/enhanced/classlib/trunk/modules/swing/src/main/java/common/javax/swing/table/DefaultTableCellRenderer.java Fri Mar 9 07:05:05 2007
@@ -53,8 +53,10 @@
super.setBackground(c);
}
- public void updateUI() {
+ public void updateUI() {
super.updateUI();
+ setBackground(null);
+ setForeground(null);
focusBorder = UIManager.getBorder("Table.focusCellHighlightBorder");
focusCellBackground = UIManager.getColor("Table.focusCellBackground");
focusCellForeground = UIManager.getColor("Table.focusCellForeground");
Modified: harmony/enhanced/classlib/trunk/modules/swing/src/test/api/java/common/javax/swing/table/DefaultTableCellRendererTest.java
URL: http://svn.apache.org/viewvc/harmony/enhanced/classlib/trunk/modules/swing/src/test/api/java/common/javax/swing/table/DefaultTableCellRendererTest.java?view=diff&rev=516424&r1=516423&r2=516424
==============================================================================
--- harmony/enhanced/classlib/trunk/modules/swing/src/test/api/java/common/javax/swing/table/DefaultTableCellRendererTest.java (original)
+++ harmony/enhanced/classlib/trunk/modules/swing/src/test/api/java/common/javax/swing/table/DefaultTableCellRendererTest.java Fri Mar 9 07:05:05 2007
@@ -156,6 +156,11 @@
assertEquals(table.getSelectionBackground(), renderer.getBackground());
assertEquals(table.getSelectionForeground(), renderer.getForeground());
assertEquals(table.getFont(), renderer.getFont());
+
+ // regression test for HARMONY-1721
+ DefaultTableCellRenderer renderer = new DefaultTableCellRenderer();
+ Color c = renderer.getBackground();
+ assertNull(c);
}
public void testIsOpaque() throws Exception {