Mercurial > hg > release > thermostat-1.6
changeset 1907:fe6cbf89e84e
Rename TreeMapPanel to SwingHeapTreeMapView
This class is not a general TreeMapPanel. It only uses TreeMapComponent,
but otherwise has nothing to do with a TreeMap. It is actually a swing
implementation of the HeapTreeMapView. Clarify the name to avoid
confusion in the future.
PR2998
Reviewed-by: jkang
Review-thread: http://icedtea.classpath.org/pipermail/thermostat/2016-June/019248.html
Original-thread: http://icedtea.classpath.org/pipermail/thermostat/2015-August/015541.html
author | James Aziz <jaziz@redhat.com> |
---|---|
date | Tue, 07 Jun 2016 11:55:47 -0400 |
parents | 686b04914043 |
children | cb6390cf824b |
files | vm-heap-analysis/client-swing/src/main/java/com/redhat/thermostat/vm/heap/analysis/client/swing/internal/SwingHeapTreeMapView.java vm-heap-analysis/client-swing/src/main/java/com/redhat/thermostat/vm/heap/analysis/client/swing/internal/SwingHeapTreeMapViewProvider.java vm-heap-analysis/client-swing/src/main/java/com/redhat/thermostat/vm/heap/analysis/client/swing/internal/TreeMapPanel.java |
diffstat | 3 files changed, 74 insertions(+), 74 deletions(-) [+] |
line wrap: on
line diff
--- /dev/null Thu Jan 01 00:00:00 1970 +0000 +++ b/vm-heap-analysis/client-swing/src/main/java/com/redhat/thermostat/vm/heap/analysis/client/swing/internal/SwingHeapTreeMapView.java Tue Jun 07 11:55:47 2016 -0400 @@ -0,0 +1,73 @@ +/* + * Copyright 2012-2016 Red Hat, Inc. + * + * This file is part of Thermostat. + * + * Thermostat is free software; you can redistribute it and/or modify + * it under the terms of the GNU General Public License as published + * by the Free Software Foundation; either version 2, or (at your + * option) any later version. + * + * Thermostat is distributed in the hope that it will be useful, but + * WITHOUT ANY WARRANTY; without even the implied warranty of + * MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE. See the GNU + * General Public License for more details. + * + * You should have received a copy of the GNU General Public License + * along with Thermostat; see the file COPYING. If not see + * <http://www.gnu.org/licenses/>. + * + * Linking this code with other modules is making a combined work + * based on this code. Thus, the terms and conditions of the GNU + * General Public License cover the whole combination. + * + * As a special exception, the copyright holders of this code give + * you permission to link this code with independent modules to + * produce an executable, regardless of the license terms of these + * independent modules, and to copy and distribute the resulting + * executable under terms of your choice, provided that you also + * meet, for each linked independent module, the terms and conditions + * of the license of that module. An independent module is a module + * which is not derived from or based on this code. If you modify + * this code, you may extend this exception to your version of the + * library, but you are not obligated to do so. If you do not wish + * to do so, delete this exception statement from your version. + */ + +package com.redhat.thermostat.vm.heap.analysis.client.swing.internal; + +import java.awt.BorderLayout; +import java.awt.Component; +import java.awt.Dimension; + +import javax.swing.JPanel; + +import com.redhat.thermostat.client.swing.SwingComponent; +import com.redhat.thermostat.client.swing.components.experimental.TreeMapComponent; +import com.redhat.thermostat.client.swing.components.experimental.TreeMapToolbar; +import com.redhat.thermostat.vm.heap.analysis.client.core.HeapTreeMapView; +import com.redhat.thermostat.vm.heap.analysis.common.ObjectHistogram; + +public class SwingHeapTreeMapView extends HeapTreeMapView implements SwingComponent { + + private final JPanel panel; + + private TreeMapComponent treeMap; + + public SwingHeapTreeMapView() { + panel = new JPanel(); + panel.setLayout(new BorderLayout()); + } + + @Override + public void display(ObjectHistogram histogram) { + treeMap = new TreeMapComponent(HistogramConverter.convertToTreeMap(histogram), new Dimension()); + panel.add(treeMap, BorderLayout.CENTER); + panel.add(new TreeMapToolbar(treeMap), BorderLayout.NORTH); + } + + @Override + public Component getUiComponent() { + return panel; + } +}
--- a/vm-heap-analysis/client-swing/src/main/java/com/redhat/thermostat/vm/heap/analysis/client/swing/internal/SwingHeapTreeMapViewProvider.java Tue Jun 07 11:14:49 2016 -0400 +++ b/vm-heap-analysis/client-swing/src/main/java/com/redhat/thermostat/vm/heap/analysis/client/swing/internal/SwingHeapTreeMapViewProvider.java Tue Jun 07 11:55:47 2016 -0400 @@ -43,7 +43,7 @@ @Override public HeapTreeMapView createView() { - return new TreeMapPanel(); + return new SwingHeapTreeMapView(); } }
--- a/vm-heap-analysis/client-swing/src/main/java/com/redhat/thermostat/vm/heap/analysis/client/swing/internal/TreeMapPanel.java Tue Jun 07 11:14:49 2016 -0400 +++ /dev/null Thu Jan 01 00:00:00 1970 +0000 @@ -1,73 +0,0 @@ -/* - * Copyright 2012-2016 Red Hat, Inc. - * - * This file is part of Thermostat. - * - * Thermostat is free software; you can redistribute it and/or modify - * it under the terms of the GNU General Public License as published - * by the Free Software Foundation; either version 2, or (at your - * option) any later version. - * - * Thermostat is distributed in the hope that it will be useful, but - * WITHOUT ANY WARRANTY; without even the implied warranty of - * MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE. See the GNU - * General Public License for more details. - * - * You should have received a copy of the GNU General Public License - * along with Thermostat; see the file COPYING. If not see - * <http://www.gnu.org/licenses/>. - * - * Linking this code with other modules is making a combined work - * based on this code. Thus, the terms and conditions of the GNU - * General Public License cover the whole combination. - * - * As a special exception, the copyright holders of this code give - * you permission to link this code with independent modules to - * produce an executable, regardless of the license terms of these - * independent modules, and to copy and distribute the resulting - * executable under terms of your choice, provided that you also - * meet, for each linked independent module, the terms and conditions - * of the license of that module. An independent module is a module - * which is not derived from or based on this code. If you modify - * this code, you may extend this exception to your version of the - * library, but you are not obligated to do so. If you do not wish - * to do so, delete this exception statement from your version. - */ - -package com.redhat.thermostat.vm.heap.analysis.client.swing.internal; - -import java.awt.BorderLayout; -import java.awt.Component; -import java.awt.Dimension; - -import javax.swing.JPanel; - -import com.redhat.thermostat.client.swing.SwingComponent; -import com.redhat.thermostat.client.swing.components.experimental.TreeMapComponent; -import com.redhat.thermostat.client.swing.components.experimental.TreeMapToolbar; -import com.redhat.thermostat.vm.heap.analysis.client.core.HeapTreeMapView; -import com.redhat.thermostat.vm.heap.analysis.common.ObjectHistogram; - -public class TreeMapPanel extends HeapTreeMapView implements SwingComponent { - - private final JPanel panel; - - private TreeMapComponent treeMap; - - public TreeMapPanel() { - panel = new JPanel(); - panel.setLayout(new BorderLayout()); - } - - @Override - public void display(ObjectHistogram histogram) { - treeMap = new TreeMapComponent(HistogramConverter.convertToTreeMap(histogram), new Dimension()); - panel.add(treeMap, BorderLayout.CENTER); - panel.add(new TreeMapToolbar(treeMap), BorderLayout.NORTH); - } - - @Override - public Component getUiComponent() { - return panel; - } -}