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

Side by Side Diff: javatests/org/chromium/distiller/webdocument/DomConverterTest.java

Issue 1411603004: Discard hidden articles when using fast path (Closed) Base URL: https://github.com/chromium/dom-distiller.git@master
Patch Set: nit 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 unified diff | Download patch
OLDNEW
1 // Copyright 2014 The Chromium Authors. All rights reserved. 1 // Copyright 2014 The Chromium Authors. All rights reserved.
2 // Use of this source code is governed by a BSD-style license that can be 2 // Use of this source code is governed by a BSD-style license that can be
3 // found in the LICENSE file. 3 // found in the LICENSE file.
4 4
5 package org.chromium.distiller.webdocument; 5 package org.chromium.distiller.webdocument;
6 6
7 import org.chromium.distiller.DomDistillerJsTestCase; 7 import org.chromium.distiller.DomDistillerJsTestCase;
8 import org.chromium.distiller.DomWalker; 8 import org.chromium.distiller.DomWalker;
9 9
10 import com.google.gwt.dom.client.Document; 10 import com.google.gwt.dom.client.Document;
(...skipping 116 matching lines...) Expand 10 before | Expand all | Expand 10 after
127 runTest("<noscript></noscript>", ""); 127 runTest("<noscript></noscript>", "");
128 runTest("<applet></applet>", ""); 128 runTest("<applet></applet>", "");
129 runTest("<object></object>", ""); 129 runTest("<object></object>", "");
130 runTest("<option></option>", ""); 130 runTest("<option></option>", "");
131 runTest("<embed></embed>", ""); 131 runTest("<embed></embed>", "");
132 } 132 }
133 133
134 public void testElementOrder() { 134 public void testElementOrder() {
135 Element container = Document.get().createDivElement(); 135 Element container = Document.get().createDivElement();
136 container.setInnerHTML("Text content <img src=\"http://example.com/1.jpg \"> more content"); 136 container.setInnerHTML("Text content <img src=\"http://example.com/1.jpg \"> more content");
137 mBody.appendChild(container);
137 138
138 WebDocumentBuilder builder = new WebDocumentBuilder(); 139 WebDocumentBuilder builder = new WebDocumentBuilder();
139 DomConverter converter = new DomConverter(builder); 140 DomConverter converter = new DomConverter(builder);
140 new DomWalker(converter).walk(container); 141 new DomWalker(converter).walk(container);
141 142
142 WebDocument doc = builder.toWebDocument(); 143 WebDocument doc = builder.toWebDocument();
143 List<WebElement> elements = doc.getElements(); 144 List<WebElement> elements = doc.getElements();
144 145
145 assertEquals(3, elements.size()); 146 assertEquals(3, elements.size());
146 assertTrue(elements.get(0) instanceof WebText); 147 assertTrue(elements.get(0) instanceof WebText);
147 assertTrue(elements.get(1) instanceof WebImage); 148 assertTrue(elements.get(1) instanceof WebImage);
148 assertTrue(elements.get(2) instanceof WebText); 149 assertTrue(elements.get(2) instanceof WebText);
149 } 150 }
150 151
151 public void testLineBreak() throws Throwable { 152 public void testLineBreak() throws Throwable {
152 String html = "text<br>split<br/>with<br/>lines"; 153 String html = "text<br>split<br/>with<br/>lines";
153 runTest(html, "text\nsplit\nwith\nlines"); 154 runTest(html, "text\nsplit\nwith\nlines");
154 } 155 }
155 156
156 public void testList() throws Throwable { 157 public void testList() throws Throwable {
157 Element container = Document.get().createDivElement(); 158 Element container = Document.get().createDivElement();
158 container.setInnerHTML("<ol><li>some text1</li><li>some text2</li></ol>" ); 159 container.setInnerHTML("<ol><li>some text1</li><li>some text2</li></ol>" );
160 mBody.appendChild(container);
159 161
160 WebDocumentBuilder builder = new WebDocumentBuilder(); 162 WebDocumentBuilder builder = new WebDocumentBuilder();
161 DomConverter converter = new DomConverter(builder); 163 DomConverter converter = new DomConverter(builder);
162 new DomWalker(converter).walk(container); 164 new DomWalker(converter).walk(container);
163 165
164 WebDocument doc = builder.toWebDocument(); 166 WebDocument doc = builder.toWebDocument();
165 List<WebElement> elements = doc.getElements(); 167 List<WebElement> elements = doc.getElements();
166 168
167 assertEquals(8, elements.size()); 169 assertEquals(8, elements.size());
168 assertTrue(elements.get(0) instanceof WebTag); 170 assertTrue(elements.get(0) instanceof WebTag);
(...skipping 12 matching lines...) Expand all
181 183
182 assertTrue(elements.get(5) instanceof WebText); 184 assertTrue(elements.get(5) instanceof WebText);
183 185
184 assertTrue(elements.get(6) instanceof WebTag); 186 assertTrue(elements.get(6) instanceof WebTag);
185 assertFalse(((WebTag) elements.get(6)).isStartTag()); 187 assertFalse(((WebTag) elements.get(6)).isStartTag());
186 188
187 assertTrue(elements.get(7) instanceof WebTag); 189 assertTrue(elements.get(7) instanceof WebTag);
188 assertFalse(((WebTag) elements.get(7)).isStartTag()); 190 assertFalse(((WebTag) elements.get(7)).isStartTag());
189 } 191 }
190 } 192 }
OLDNEW
« java/org/chromium/distiller/DomUtil.java ('K') | « javatests/org/chromium/distiller/TestUtil.java ('k') | no next file » | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698