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

Unified Diff: java/org/chromium/distiller/TableClassifier.java

Issue 1725243002: Fix some warnings in Eclipse (Closed) Base URL: git@github.com:chromium/dom-distiller.git@master
Patch Set: rebase Created 4 years, 10 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
Index: java/org/chromium/distiller/TableClassifier.java
diff --git a/java/org/chromium/distiller/TableClassifier.java b/java/org/chromium/distiller/TableClassifier.java
index 5ae9d3f40d8817ec314606873199a644000b8ace..bc8bda762d4e10f72ba1efccfb5e9f67b52caac1 100644
--- a/java/org/chromium/distiller/TableClassifier.java
+++ b/java/org/chromium/distiller/TableClassifier.java
@@ -197,7 +197,7 @@ public class TableClassifier {
// many (old) pages have layout tables with the "summary" attribute (#10).
Element docElement = t.getOwnerDocument().getDocumentElement();
int docWidth = docElement.getOffsetWidth();
- if (docWidth > 0 && (double) t.getOffsetWidth() > 0.95 * (double) docWidth) {
+ if (docWidth > 0 && t.getOffsetWidth() > 0.95 * docWidth) {
boolean viewportFound = false;
NodeList<Element> allMeta = docElement.getElementsByTagName("META");
for (int i = 0; i < allMeta.getLength() && !viewportFound; i++) {
@@ -257,7 +257,7 @@ public class TableClassifier {
// into any of the above heuristics but are for layout, and hence shouldn't default to data
// by #18.
int docHeight = docElement.getOffsetHeight();
- if (docHeight > 0 && (double) t.getOffsetHeight() > 0.9 * (double) docHeight) {
+ if (docHeight > 0 && t.getOffsetHeight() > 0.9 * docHeight) {
return logAndReturn(Reason.MORE_90_PERCENT_DOC_HEIGHT, "", Type.LAYOUT);
}

Powered by Google App Engine
This is Rietveld 408576698