Factored out ContributedLibraryTableCell.update(...) method

Author: cmaglie

app/src/cc/arduino/contributions/libraries/ui/ContributedLibraryTableCell.java

@@ -29,16 +29,17 @@
 
 public class ContributedLibraryTableCell extends JPanel {
 
-  protected final JButton installButton;
-  protected final Component installButtonPlaceholder;
-  protected final JComboBox downgradeChooser;
-  protected final JComboBox versionToInstallChooser;
-  protected final JButton downgradeButton;
-  protected final JPanel buttonsPanel;
-  protected final JPanel inactiveButtonsPanel;
-  protected final JLabel statusLabel;
-
-  public ContributedLibraryTableCell() {
+  final JButton installButton;
+  final Component installButtonPlaceholder;
+  final JComboBox downgradeChooser;
+  final JComboBox versionToInstallChooser;
+  final JButton downgradeButton;
+  final JPanel buttonsPanel;
+  final JPanel inactiveButtonsPanel;
+  final JLabel statusLabel;
+
+  public ContributedLibraryTableCell(JTable parentTable, Object value,
+                                     boolean isSelected) {
     super();
     setLayout(new BoxLayout(this, BoxLayout.Y_AXIS));
 
@@ -99,11 +100,8 @@ public ContributedLibraryTableCell() {
     add(inactiveButtonsPanel);
 
     add(Box.createVerticalStrut(15));
-  }
 
-  void update(JTable parentTable, Object value, boolean isSelected) {
     ContributedLibraryReleases releases = (ContributedLibraryReleases) value;
-
     JTextPane description = makeNewDescription();
 
     // FIXME: happens on macosx, don't know why
@@ -193,9 +191,9 @@ void update(JTable parentTable, Object value, boolean isSelected) {
 
     // See:
     // http://stackoverflow.com/questions/3081210/how-to-set-jtextarea-to-have-height-that-matches-the-size-of-a-text-it-contains
-    int width = parentTable.getBounds().width;
-    InstallerTableCell.setJTextPaneDimensionToFitContainedText(description,
-                                                               width);
+    InstallerTableCell
+        .setJTextPaneDimensionToFitContainedText(description,
+                                                 parentTable.getBounds().width);
 
     if (isSelected) {
       setBackground(parentTable.getSelectionBackground());

app/src/cc/arduino/contributions/libraries/ui/ContributedLibraryTableCellEditor.java

@@ -67,7 +67,7 @@ public Component getTableCellEditorComponent(JTable table, Object value,
                                                int column) {
     editorValue = (ContributedLibraryReleases) value;
 
-    editorCell = new ContributedLibraryTableCell();
+    editorCell = new ContributedLibraryTableCell(table, value, true);
     editorCell.installButton
         .addActionListener(e -> onInstall(editorValue.getSelected(),
                                           editorValue.getInstalled()));
@@ -134,7 +134,6 @@ public Component getTableCellEditorComponent(JTable table, Object value,
     editorCell.versionToInstallChooser
         .setVisible(installed == null && uninstalledReleases.size() > 1);
 
-    editorCell.update(table, value, true);
     editorCell.setBackground(new Color(218, 227, 227)); // #dae3e3
     return editorCell;
   }

app/src/cc/arduino/contributions/libraries/ui/ContributedLibraryTableCellRenderer.java

@@ -42,19 +42,17 @@ public Component getTableCellRendererComponent(JTable table, Object value,
                                                  boolean isSelected,
                                                  boolean hasFocus, int row,
                                                  int column) {
-
-    ContributedLibraryTableCell cell = new ContributedLibraryTableCell();
+    ContributedLibraryTableCell cell = new ContributedLibraryTableCell(table,
+        value, isSelected);
     cell.setButtonsVisible(false);
-    cell.update(table, value, isSelected);
 
     if (row % 2 == 0) {
       cell.setBackground(new Color(236, 241, 241)); // #ecf1f1
     } else {
       cell.setBackground(new Color(255, 255, 255));
     }
 
-    int height = new Double(cell.getPreferredSize().getHeight())
-        .intValue();
+    int height = new Double(cell.getPreferredSize().getHeight()).intValue();
     if (table.getRowHeight(row) < height) {
       table.setRowHeight(row, height);
     }