filter out invisible elements

src/com/dom_distiller/client/DomUtil.java

 // Copyright 2014 The Chromium Authors. All rights reserved.
 // Use of this source code is governed by a BSD-style license that can be
 // found in the LICENSE file.
 
 package com.dom_distiller.client;
 
 import com.google.gwt.core.client.JsArray;
 
 import com.google.gwt.dom.client.Element;
 import com.google.gwt.dom.client.Node;
 import com.google.gwt.dom.client.NodeList;
+import com.google.gwt.dom.client.Style;
 
 public class DomUtil {
     /**
      * GWT does not provide a way to get a list of all attributes that have been explicitly set on a
      * DOM element (only a way to query the value of a particular attribute). In javascript, this
      * list is accessible as elem.attributes.
      *
      * @Return The element's attribute list from javascript.
      */
     public static native JsArray<Node> getAttributes(Element elem) /*-{
@@ skipping 17 matching lines @@
 
         return @com.dom_distiller.client.DomUtil::javaGetFirstElementWithClassName(Lcom/google/gwt/dom/client/Element;Ljava/lang/String;)(root, className);
     }-*/;
 
     // Returns the first element with |className| in the tree rooted at |root|, null if none is
     // found.  |className| is expected to be in lower case.
     public static Element javaGetFirstElementWithClassName(Element root, String className) {
         NodeList<Element> allElems = root.getElementsByTagName("*");
         for (int i = 0; i < allElems.getLength(); i++) {
             Element elem = allElems.getItem(i);
-            String classAttr = elem.getClassName().toLowerCase();
-            // Make sure |className| is not the substring of another name in |classAttr|, so check
-            // for whitespaces before and after.
-            if ((" " + classAttr + " ").contains(" " + className + " ")) return elem;
+            if (hasClassName(elem, className)) return elem;
         }
         return null;
     }
 
+    public static boolean hasClassName(Element elem, String className) {
+        String classAttr = elem.getClassName().toLowerCase();
+        // Make sure |className| is not the substring of another name in |classAttr|, so check
+        // for whitespaces before and after.
+        return (" " + classAttr + " ").contains(" " + className + " ");
+    }
+
+    /**
+      * @Return The CSS style of an element after applying the active stylesheets and resolving any
+      * basic computation the style's value(s) may contain.
+      * @param el - DOM element
+    */
+    public static native Style getComputedStyle(Element el) /*-{
+      // Standard (includes IE9)

[cjhopman, 2014/05/13 16:20:53]

We don't care about IE.

In everything that we care about (except maybe gwt's test modes),
getComputedStyle is available on the window object, you don't need the
document.defaultView.

So, just:
return getComputedStyle(el, null);

If required for gwt tests, just fall back to el.style (i.e. get rid of the case
for older ie).

[kuan, 2014/05/13 16:55:43]

Done.

+      if (document.defaultView && document.defaultView.getComputedStyle) {
+          return document.defaultView.getComputedStyle(el, null);
+      }
+
+      // Older IE
+      if (el.currentStyle) return el.currentStyle;
+
+      // Inline style
+      return el.style;
+    }-*/;
 }