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

Unified Diff: java/org/chromium/distiller/PagingLinksFinder.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/PagingLinksFinder.java
diff --git a/java/org/chromium/distiller/PagingLinksFinder.java b/java/org/chromium/distiller/PagingLinksFinder.java
index 9597d018d562648a717c5042a93040d23c0c0a02..3e78bd7a413832acddf3ceb1e9f5e3a755080de6 100644
--- a/java/org/chromium/distiller/PagingLinksFinder.java
+++ b/java/org/chromium/distiller/PagingLinksFinder.java
@@ -19,14 +19,9 @@ import com.google.gwt.dom.client.Document;
import com.google.gwt.dom.client.Element;
import com.google.gwt.dom.client.NodeList;
import com.google.gwt.regexp.shared.RegExp;
-import com.google.gwt.user.client.Window;
-import java.util.ArrayList;
-import java.util.Arrays;
-import java.util.Collections;
import java.util.HashMap;
import java.util.HashSet;
-import java.util.List;
import java.util.Map;
import java.util.Set;
@@ -61,11 +56,6 @@ public class PagingLinksFinder {
private static final RegExp REG_LINK_PAGINATION =
RegExp.compile("p(a|g|ag)?(e|ing|ination)?(=|\\/)[0-9]{1,2}$", "i");
private static final RegExp REG_FIRST_LAST = RegExp.compile("(first|last)", "i");
- // Examples that match PAGE_NUMBER_REGEX are: "_p3", "-pg3", "p3", "_1", "-12-2".
- // Examples that don't match PAGE_NUMBER_REGEX are: "_p3 ", "p", "p123".
- private static final RegExp REG_PAGE_NUMBER =
- RegExp.compile("((_|-)?p[a-z]*|(_|-))[0-9]{1,2}$", "gi");
-
private static final RegExp REG_HREF_CLEANER = RegExp.compile("/?(#.*)?$");
private static final RegExp REG_NUMBER = RegExp.compile("\\d");
@@ -381,12 +371,6 @@ public class PagingLinksFinder {
return a;
}
- private static String fixMissingScheme(String url) {
- if (url.isEmpty()) return "";
- if (!url.contains("://")) return "http://" + url;
- return url;
- }
-
// The link is resolved using an anchor within a new HTML document with a base tag.
public static String resolveLinkHref(AnchorElement link, AnchorElement baseAnchor) {
String linkHref = link.getAttribute("href");
@@ -409,12 +393,6 @@ public class PagingLinksFinder {
return StringUtil.split(url, "\\/")[0];
}
- private static String getPath(String url) {
- url = StringUtil.split(url, ":\\/\\/")[1];
- if (!url.contains("/")) return "";
- return StringUtil.findAndReplace(url, "^([^/]*)/", "");
- }
-
public static Integer pageDiff(String url, String linkHref, AnchorElement link, int skip) {
int commonLen = skip;
int i;

Powered by Google App Engine
This is Rietveld 408576698