harmony-commits mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From "Sergey Krivenko (JIRA)" <j...@apache.org>
Subject [jira] Commented: (HARMONY-1721) [classlib][swing] javax.swing.table.DefaultTableCellRenderer.getBackground() returns some default value while RI returns null
Date Mon, 19 Feb 2007 14:21:06 GMT

    [ https://issues.apache.org/jira/browse/HARMONY-1721?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel#action_12474178
] 

Sergey Krivenko commented on HARMONY-1721:
------------------------------------------

The proposed patch is a workaround. I didn't quite understand all the manipulations Harmony
does while initializing DefaultTableCellRenderer. Particularly, when it calls updateUI() method
three times. This is where background and foreground are changed. But any other fixes just
ruin the whole thing. 

> [classlib][swing] javax.swing.table.DefaultTableCellRenderer.getBackground() returns
some default value while RI returns null
> -----------------------------------------------------------------------------------------------------------------------------
>
>                 Key: HARMONY-1721
>                 URL: https://issues.apache.org/jira/browse/HARMONY-1721
>             Project: Harmony
>          Issue Type: Bug
>          Components: Classlib
>            Reporter: Vera Aristova
>            Priority: Minor
>         Attachments: Harmony-1721-DefaultTableCellRenderer.patch, Harmony-1721-DefaultTableCellRendererTest.patch,
patch1721, patch1721
>
>
> javax.swing.table.DefaultTableCellRenderer.getBackground() returns some default value
while RI returns null.
> Test for reproducing:
> import junit.framework.TestCase;
> import javax.swing.table.DefaultTableCellRenderer;
> import javax.swing.text.LabelView;
> import java.awt.Color;
> public class test extends TestCase 
> {
> 	public void testGetBackground() 
> 	{
> 		DefaultTableCellRenderer returnValue = new DefaultTableCellRenderer();
> 		Color c = returnValue.getBackground();
> 		assertNull(c);
> 	}
> }
> OUTPUT:
> HARMONY
> Apache Harmony Launcher : (c) Copyright 1991, 2006 The Apache Software Foundatio
> n or its licensors, as applicable.
> java version "1.5.0"
> pre-alpha : not complete or compatible
> svn = r450941, (Sep 28 2006), Windows/ia32/msvc 1310, release build
> http://incubator.apache.org/harmony
> .F
> Time: 0.344
> There was 1 failure:
> 1) testGetBackground(test)junit.framework.AssertionFailedError
>         at test.testGetBackground(test.java:10)
>         at java.lang.reflect.VMReflection.invokeMethod(Native Method)
> FAILURES!!!
> Tests run: 1,  Failures: 1,  Errors: 0
> ------------------------------------------------------------------------
> SUN
> java version "1.5.0_06"
> Java(TM) 2 Runtime Environment, Standard Edition (build 1.5.0_06-b05)
> Java HotSpot(TM) Client VM (build 1.5.0_06-b05, mixed mode)
> .
> Time: 1.766
> OK (1 test)

-- 
This message is automatically generated by JIRA.
-
You can reply to this email to add a comment to the issue online.


Mime
View raw message