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

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

Issue 2287113005: Skip unrecognized iframes (Closed) Base URL: git@github.com:chromium/dom-distiller.git@innertext
Patch Set: address comment Created 4 years, 4 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 | « java/org/chromium/distiller/webdocument/DomConverter.java ('k') | no next file » | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: javatests/org/chromium/distiller/ContentExtractorTest.java
diff --git a/javatests/org/chromium/distiller/ContentExtractorTest.java b/javatests/org/chromium/distiller/ContentExtractorTest.java
index 00995a33a83ea90e1ac6b281e9f7d68bd82566f7..c26b7db480bf44ac3b016ac6e31a00a4dfdb19cc 100644
--- a/javatests/org/chromium/distiller/ContentExtractorTest.java
+++ b/javatests/org/chromium/distiller/ContentExtractorTest.java
@@ -632,4 +632,17 @@ public class ContentExtractorTest extends DomDistillerJsTestCase {
assertExtractor(expected, htmlArticle);
}
+
+ public void testUnwantedIFrame() {
+ final String html =
+ "<p>" + CONTENT_TEXT + "</p>" +
+ "<iframe>dummy</iframe>" +
+ "<p>" + CONTENT_TEXT + "</p>";
+
+ final String expected =
+ "<p>" + CONTENT_TEXT + "</p>" +
+ "<p>" + CONTENT_TEXT + "</p>";
+
+ assertExtractor(expected, html);
+ }
}
« no previous file with comments | « java/org/chromium/distiller/webdocument/DomConverter.java ('k') | no next file » | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698