You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@pivot.apache.org by rw...@apache.org on 2014/04/14 20:19:39 UTC

svn commit: r1587271 - /pivot/trunk/wtk-terra/src/org/apache/pivot/wtk/skin/terra/TerraVFSBrowserSkin.java

Author: rwhitcomb
Date: Mon Apr 14 18:19:39 2014
New Revision: 1587271

URL: http://svn.apache.org/r1587271
Log:
Noticed that sometimes a tooltip tries to get triggered on a non-existent row
in the VFS File browser.  So, protect against that by checking the result of
tableView.getRowAt(y) to see if there is a row there before using the value
to index into the table data.

Modified:
    pivot/trunk/wtk-terra/src/org/apache/pivot/wtk/skin/terra/TerraVFSBrowserSkin.java

Modified: pivot/trunk/wtk-terra/src/org/apache/pivot/wtk/skin/terra/TerraVFSBrowserSkin.java
URL: http://svn.apache.org/viewvc/pivot/trunk/wtk-terra/src/org/apache/pivot/wtk/skin/terra/TerraVFSBrowserSkin.java?rev=1587271&r1=1587270&r2=1587271&view=diff
==============================================================================
--- pivot/trunk/wtk-terra/src/org/apache/pivot/wtk/skin/terra/TerraVFSBrowserSkin.java (original)
+++ pivot/trunk/wtk-terra/src/org/apache/pivot/wtk/skin/terra/TerraVFSBrowserSkin.java Mon Apr 14 18:19:39 2014
@@ -1067,7 +1067,11 @@ public class TerraVFSBrowserSkin extends
                 }
 
                 // Gets the underlying file
-                FileObject file = (FileObject) fileTableView.getTableData().get(fileTableView.getRowAt(y));
+                int row = fileTableView.getRowAt(y);
+                if (row < 0) {
+                    return;
+                }
+                FileObject file = (FileObject) fileTableView.getTableData().get(row);
 
                 // Construct and show the tooltip.
                 final Tooltip tooltip = new Tooltip();