Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(783)

Issues Search
    My Issues | Starred     Open | Closed | All

Unified Diff: javatests/org/chromium/distiller/DomUtilTest.java

Issue 1411603004: Discard hidden articles when using fast path (Closed) Base URL: https://github.com/chromium/dom-distiller.git@master
Patch Set: nitpick 3 fixed Created 4 years, 6 months ago
Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.
Jump to:
View side-by-side diff with in-line comments
Download patch
« no previous file with comments | « javatests/org/chromium/distiller/ContentExtractorTest.java ('k') | javatests/org/chromium/distiller/TestUtil.java » ('j') | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: javatests/org/chromium/distiller/DomUtilTest.java
diff --git a/javatests/org/chromium/distiller/DomUtilTest.java b/javatests/org/chromium/distiller/DomUtilTest.java
index 2336b36e53689841d37b0fb6b6d956a2be6f20e8..b7b444a789562a9227ca466d42bb33ddeca32189 100644
--- a/javatests/org/chromium/distiller/DomUtilTest.java
+++ b/javatests/org/chromium/distiller/DomUtilTest.java
@@ -10,11 +10,13 @@ import com.google.gwt.core.client.JsArray;
import com.google.gwt.dom.client.Document;
import com.google.gwt.dom.client.Element;
import com.google.gwt.dom.client.Node;
+import com.google.gwt.dom.client.NodeList;
import java.util.Map;
import java.util.List;
public class DomUtilTest extends DomDistillerJsTestCase {
+
wychen 2016/06/07 16:20:08 nit: extra line
marcelorcorrea 2016/06/07 17:06:02 Done.
public void testGetAttributes() {
Element e = Document.get().createDivElement();
e.setInnerHTML("<div style=\"width:50px; height:100px\" id=\"f\" class=\"sdf\"></div>");
@@ -110,8 +112,9 @@ public class DomUtilTest extends DomDistillerJsTestCase {
currDiv.appendChild(TestUtil.createDiv(5));
assertEquals(div2, DomUtil.getNearestCommonAncestor(finalDiv1, currDiv.getChild(0)));
- assertEquals(div2, DomUtil.getNearestCommonAncestor(
- DomUtil.querySelectorAll(mRoot, "[id=\"3\"],[id=\"5\"]")));
+ NodeList<Element> nodeList = DomUtil.querySelectorAll(mRoot, "[id=\"3\"],[id=\"5\"]");
+ assertEquals(div2, DomUtil.getNearestCommonAncestor(TestUtil.nodeListToList(nodeList)));
+
wychen 2016/06/07 16:20:08 nit: extra line
marcelorcorrea 2016/06/07 17:06:02 Done.
}
/**
@@ -129,8 +132,8 @@ public class DomUtilTest extends DomDistillerJsTestCase {
div2.appendChild(div3);
assertEquals(div, DomUtil.getNearestCommonAncestor(div, div3));
- assertEquals(div, DomUtil.getNearestCommonAncestor(
- DomUtil.querySelectorAll(mRoot, "[id=\"1\"],[id=\"3\"]")));
+ NodeList<Element> nodeList = DomUtil.querySelectorAll(mRoot, "[id=\"1\"],[id=\"3\"]");
+ assertEquals(div, DomUtil.getNearestCommonAncestor(TestUtil.nodeListToList(nodeList)));
}
public void testNodeDepth() {
@@ -378,6 +381,212 @@ public class DomUtilTest extends DomDistillerJsTestCase {
assertEquals(expected, mBody.getInnerHTML());
}
+ public void testIsVisibleByOffsetParentDisplayNone() {
+ String html =
+ "<div style=\"display: none;\">" +
+ "<div></div>" +
+ "</div>";
+ mBody.setInnerHTML(html);
+ Element child = mBody.getFirstChildElement().getFirstChildElement();
+ assertFalse(DomUtil.isVisibleByOffset(child));
+ }
+
+ public void testIsVisibleByOffsetChildDisplayNone() {
+ String html =
+ "<div>" +
+ "<div style=\"display: none;\"></div>" +
+ "</div>";
+ mBody.setInnerHTML(html);
+ Element child = mBody.getFirstChildElement().getFirstChildElement();
+ assertFalse(DomUtil.isVisibleByOffset(child));
+ }
+
+ public void testIsVisibleByOffsetDisplayBlock() {
+ String html =
+ "<div>" +
+ "<div></div>" +
+ "</div>";
+ mBody.setInnerHTML(html);
+ Element child = mBody.getFirstChildElement().getFirstChildElement();
+ assertTrue(DomUtil.isVisibleByOffset(child));
+ }
+
+ public void testOnlyProcessArticleElement() {
+ final String htmlArticle =
+ "<h1></h1>" +
+ "<article></article>";
+
+ String expected = "<article></article>";
+
+ Element result = getArticleElement(htmlArticle);
+ assertEquals(expected, result.getString());
+ }
+
+ public void testOnlyProcessArticleElementWithHiddenArticleElement() {
+ final String htmlArticle =
+ "<h1></h1>" +
+ "<article></article>" +
+ "<article style=\"display:none\"></article>";
+
+ String expected = "<article></article>";
+
+ Element result = getArticleElement(htmlArticle);
+ assertEquals(expected, result.getString());
+ }
+
+ public void testOnlyProcessArticleElementMultiple() {
+ final String htmlArticle =
+ "<h1></h1>" +
+ "<article></article>" +
+ "<article></article>";
+
+ // The existence of multiple articles disables the fast path.
+ assertNull(getArticleElement(htmlArticle));
+ }
+
+ public void testOnlyProcessSchemaOrgArticle() {
+ final String htmlArticle =
+ "<h1></h1>" +
+ "<div itemscope itemtype=\"http://schema.org/Article\">" +
+ "</div>";
+
+ final String expected =
+ "<div itemscope=\"\" " +
+ "itemtype=\"http://schema.org/Article\">" +
+ "</div>";
+
+ Element result = getArticleElement(htmlArticle);
+ assertEquals(expected, result.getString());
+ }
+
+ public void testOnlyProcessSchemaOrgArticleWithHiddenArticleElement() {
+ final String htmlArticle =
+ "<h1></h1>" +
+ "<div itemscope itemtype=\"http://schema.org/Article\">" +
+ "</div>" +
+ "<div itemscope itemtype=\"http://schema.org/Article\" " +
+ "style=\"display:none\">" +
+ "</div>";
+
+ String expected =
+ "<div itemscope=\"\" itemtype=\"http://schema.org/Article\">" +
+ "</div>";
+
+ Element result = getArticleElement(htmlArticle);
+ assertEquals(expected, result.getString());
+ }
+
+ public void testOnlyProcessSchemaOrgArticleNews() {
+ final String htmlArticle =
+ "<h1></h1>" +
+ "<div itemscope itemtype=\"http://schema.org/NewsArticle\">" +
+ "</div>";
+
+ final String expected =
+ "<div itemscope=\"\" " +
+ "itemtype=\"http://schema.org/NewsArticle\">" +
+ "</div>";
+
+ Element result = getArticleElement(htmlArticle);
+ assertEquals(expected, result.getString());
+ }
+
+ public void testOnlyProcessSchemaOrgArticleBlog() {
+ final String htmlArticle =
+ "<h1></h1>" +
+ "<div itemscope itemtype=\"http://schema.org/BlogPosting\">" +
+ "</div>";
+
+ final String expected =
+ "<div itemscope=\"\" " +
+ "itemtype=\"http://schema.org/BlogPosting\">" +
+ "</div>";
+
+ Element result = getArticleElement(htmlArticle);
+ assertEquals(expected, result.getString());
+ }
+
+ public void testOnlyProcessSchemaOrgArticleNested() {
+ final String htmlArticle =
+ "<h1></h1>" +
+ "<div itemscope itemtype=\"http://schema.org/Article\">" +
+ "<div itemscope itemtype=\"http://schema.org/Article\">" +
+ "</div>" +
+ "</div>";
+
+ final String expected =
+ "<div itemscope=\"\" itemtype=\"http://schema.org/Article\">" +
+ "<div itemscope=\"\" itemtype=\"http://schema.org/Article\">" +
+ "</div>" +
+ "</div>";
+
+ Element result = getArticleElement(htmlArticle);
+ assertEquals(expected, result.getString());
+ }
+
+ public void testOnlyProcessSchemaOrgArticleNestedWithNestedHiddenArticleElement() {
+ final String htmlArticle =
+ "<h1></h1>" +
+ "<div itemscope itemtype=\"http://schema.org/Article\">" +
+ "<div itemscope itemtype=\"http://schema.org/Article\">" +
+ "</div>" +
+ "<div itemscope itemtype=\"http://schema.org/Article\" " +
+ "style=\"display:none\">" +
+ "</div>" +
+ "</div>";
+
+ final String expected =
+ "<div itemscope=\"\" itemtype=\"http://schema.org/Article\">" +
+ "<div itemscope=\"\" itemtype=\"http://schema.org/Article\">" +
+ "</div>" +
+ "<div itemscope=\"\" itemtype=\"http://schema.org/Article\" " +
+ "style=\"display:none\">" +
+ "</div>" +
+ "</div>";
+
+ Element result = getArticleElement(htmlArticle);
+ assertEquals(expected, result.getString());
+ }
+
+ public void testOnlyProcessSchemaOrgArticleNestedWithHiddenArticleElement() {
+ final String paragraph = "<p></p>";
+
+ final String htmlArticle =
+ "<h1></h1>" +
+ "<div itemscope itemtype=\"http://schema.org/Article\">" +
+ "<div itemscope itemtype=\"http://schema.org/Article\">" +
+ "</div>" +
+ "</div>" +
+ "<div itemscope itemtype=\"http://schema.org/Article\" " +
+ "style=\"display:none\">" +
+ "</div>";
+
+ final String expected =
+ "<div itemscope=\"\" itemtype=\"http://schema.org/Article\">" +
+ "<div itemscope=\"\" itemtype=\"http://schema.org/Article\">" +
+ "</div>" +
+ "</div>";
+
+ Element result = getArticleElement(htmlArticle);
+ assertEquals(expected, result.getString());
+ }
+
+ public void testOnlyProcessSchemaOrgNonArticleMovie() {
+ final String htmlArticle =
+ "<h1></h1>" +
+ "<div itemscope itemtype=\"http://schema.org/Movie\">" +
+ "</div>";
+
+ // Non-article schema.org types should not use the fast path.
+ Element result = getArticleElement(htmlArticle);
+ assertNull(result);
+ }
+
+ private Element getArticleElement(String html) {
+ mBody.setInnerHTML(html);
+ return DomUtil.getArticleElement(mRoot);
+ }
+
public void testGetArea() {
String elements =
"<div style=\"width: 200px; height: 100px\">w</div>" +
« no previous file with comments | « javatests/org/chromium/distiller/ContentExtractorTest.java ('k') | javatests/org/chromium/distiller/TestUtil.java » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698