Merge pull request #116 from kuc/code-fixes-part2

Code fixes part 2

src/acr/browser/lightning/BrowserActivity.java

@@ -1014,11 +1014,11 @@ public class BrowserActivity extends Activity implements BrowserController {
 			return;
 		}
 		if (mCurrentView != null) {
-			mCurrentView.setIsForgroundTab(false);
+			mCurrentView.setForegroundTab(false);
 			mCurrentView.onPause();
 		}
 		mCurrentView = view;
-		mCurrentView.setIsForgroundTab(true);
+		mCurrentView.setForegroundTab(true);
 		if (view.getWebView() != null) {
 			updateUrl(view.getUrl());
 			updateProgress(view.getProgress());
@@ -1587,7 +1587,7 @@ public class BrowserActivity extends Activity implements BrowserController {
 
 			LightningView web = data.get(position);
 			holder.txtTitle.setText(web.getTitle());
-			if (web.getIsForgroundTab()) {
+			if (web.isForegroundTab()) {
 				holder.txtTitle.setTextAppearance(context, R.style.boldText);
 			} else {
 				holder.txtTitle.setTextAppearance(context, R.style.normalText);

src/acr/browser/lightning/BrowserApp.java

@@ -7,6 +7,7 @@ public class BrowserApp extends Application {
 
 	private static Context context;
 
+	@Override
 	public void onCreate() {
 		super.onCreate();
 		context = getApplicationContext();

src/acr/browser/lightning/ClickHandler.java

@@ -15,16 +15,14 @@ public class ClickHandler extends Handler {
 		try {
 			mBrowserController = (BrowserController) context;
 		} catch (ClassCastException e) {
-			throw new ClassCastException(context.toString()
+			throw new ClassCastException(context + " must implement BrowserController");
-					+ " must implement BrowserController");
 		}
 	}
 
 	@Override
 	public void handleMessage(Message msg) {
 		super.handleMessage(msg);
-		String url = null;
+		String url = msg.getData().getString("url");
-		url = msg.getData().getString("url");
 		mBrowserController.longClickPage(url);
 	}
 }

src/acr/browser/lightning/Constants.java

@@ -5,10 +5,9 @@ package acr.browser.lightning;
 
 import android.os.Environment;
 
-public class Constants {
+public final class Constants {
 
-	public Constants() {
+	private Constants() {
-		// TODO Auto-generated constructor stub
 	}
 
 	public static final String DESKTOP_USER_AGENT = "Mozilla/5.0 (Windows NT 6.2; WOW64) AppleWebKit/537.17 (KHTML, like Gecko) Chrome/20 Safari/537.17";

src/acr/browser/lightning/DownloadHandler.java

@@ -93,7 +93,7 @@ public class DownloadHandler {
 				break;
 			}
 		}
-		if (needed == false) {
+		if (!needed) {
 			return path;
 		}
 
@@ -160,9 +160,9 @@ public class DownloadHandler {
 			webAddress = new WebAddress(url);
 			webAddress.setPath(encodePath(webAddress.getPath()));
 		} catch (Exception e) {
-			// This only happens for very bad urls, we want to chatch the
+			// This only happens for very bad urls, we want to catch the
 			// exception here
-			Log.e(LOGTAG, "Exception trying to parse url:" + url);
+			Log.e(LOGTAG, "Exception while trying to parse url '" + url + '\'', e);
 			return;
 		}
 
@@ -204,6 +204,7 @@ public class DownloadHandler {
 			final DownloadManager manager
 					= (DownloadManager) activity.getSystemService(Context.DOWNLOAD_SERVICE);
 			new Thread("Browser download") {
+				@Override
 				public void run() {
 					manager.enqueue(request);
 				}

src/acr/browser/lightning/HistoryItem.java

@@ -101,4 +101,47 @@ public class HistoryItem implements Comparable<HistoryItem> {
 	public int compareTo(HistoryItem another) {
 		return mTitle.compareTo(another.mTitle);
 	}
+
+	@Override
+	public boolean equals(Object o) {
+
+		if (this == o) {
+			return true;
+		}
+		if (o == null || ((Object) this).getClass() != o.getClass()) {
+			return false;
+		}
+
+		HistoryItem that = (HistoryItem) o;
+
+		if (mId != that.mId) {
+			return false;
+		}
+		if (mImageId != that.mImageId) {
+			return false;
+		}
+		if (mBitmap != null ? !mBitmap.equals(that.mBitmap) : that.mBitmap != null) {
+			return false;
+		}
+		if (!mTitle.equals(that.mTitle)) {
+			return false;
+		}
+		if (!mUrl.equals(that.mUrl)) {
+			return false;
+		}
+
+		return true;
+	}
+
+	@Override
+	public int hashCode() {
+
+		int result = mId;
+		result = 31 * result + mUrl.hashCode();
+		result = 31 * result + mTitle.hashCode();
+		result = 31 * result + (mBitmap != null ? mBitmap.hashCode() : 0);
+		result = 31 * result + mImageId;
+
+		return result;
+	}
 }

src/acr/browser/lightning/IntentUtils.java

@@ -80,7 +80,7 @@ public class IntentUtils {
 		PackageManager pm = mActivity.getPackageManager();
 		List<ResolveInfo> handlers = pm.queryIntentActivities(intent,
 				PackageManager.GET_RESOLVED_FILTER);
-		if (handlers == null || handlers.size() == 0) {
+		if (handlers == null || handlers.isEmpty()) {
 			return false;
 		}
 		for (ResolveInfo resolveInfo : handlers) {

src/acr/browser/lightning/LightningView.java

@@ -61,9 +61,9 @@ public class LightningView {
 
 	private AdBlock mAdBlock;
 
-	private boolean isForgroundTab = false;
+	private boolean isForegroundTab;
 
-	private IntentUtils mIntentUtils = null;
+	private IntentUtils mIntentUtils;
 
 	@SuppressWarnings("deprecation")
 	@SuppressLint("NewApi")
@@ -81,7 +81,7 @@ public class LightningView {
 		try {
 			mBrowserController = (BrowserController) activity;
 		} catch (ClassCastException e) {
-			throw new ClassCastException(activity.toString()
+			throw new ClassCastException(activity
 					+ " must implement BrowserController");
 		}
 		mIntentUtils = new IntentUtils(mBrowserController);
@@ -109,11 +109,11 @@ public class LightningView {
 				new CustomGestureListener());
 		mWebView.setOnTouchListener(new OnTouchListener() {
 
-			float mLocation = 0;
+			float mLocation;
 
-			float mY = 0;
+			float mY;
 
-			int mAction = 0;
+			int mAction;
 
 			@Override
 			public boolean onTouch(View view, MotionEvent arg1) {
@@ -142,10 +142,8 @@ public class LightningView {
 		initializeSettings(mWebView.getSettings(), activity);
 		initializePreferences(activity);
 
-		if (url != null) {
+		if (url != null && !url.trim().isEmpty()) {
-			if (!url.equals("")) {
+			mWebView.loadUrl(url);
-				mWebView.loadUrl(url);
-			}
 		} else {
 			if (mHomepage.startsWith("about:home")) {
 				mSettings.setUseWideViewPort(false);
@@ -159,7 +157,7 @@ public class LightningView {
 	}
 
 	public String getHomepage() {
-		String home = "";
+		String home;
 		home = HomepageVariables.HEAD;
 		switch (mPreferences.getInt(PreferenceConstants.SEARCH, 1)) {
 			case 0:
@@ -394,11 +392,7 @@ public class LightningView {
 	}
 
 	public boolean isShown() {
-		if (mWebView != null) {
+		return mWebView != null && mWebView.isShown();
-			return mWebView.isShown();
-		} else {
-			return false;
-		}
 	}
 
 	public synchronized void onPause() {
@@ -413,13 +407,13 @@ public class LightningView {
 		}
 	}
 
-	public void setIsForgroundTab(boolean isForground) {
+	public void setForegroundTab(boolean isForeground) {
-		isForgroundTab = isForground;
+		isForegroundTab = isForeground;
 		mBrowserController.update();
 	}
 
-	public boolean getIsForgroundTab() {
+	public boolean isForegroundTab() {
-		return isForgroundTab;
+		return isForegroundTab;
 	}
 
 	public int getProgress() {
@@ -449,10 +443,8 @@ public class LightningView {
 	}
 
 	public void requestFocus() {
-		if (mWebView != null) {
+		if (mWebView != null && !mWebView.hasFocus()) {
-			if (!mWebView.hasFocus()) {
+			mWebView.requestFocus();
-				mWebView.requestFocus();
-			}
 		}
 	}
 
@@ -524,19 +516,11 @@ public class LightningView {
 	}
 
 	public boolean canGoBack() {
-		if (mWebView != null) {
+		return mWebView != null && mWebView.canGoBack();
-			return mWebView.canGoBack();
-		} else {
-			return false;
-		}
 	}
 
 	public boolean canGoForward() {
-		if (mWebView != null) {
+		return mWebView != null && mWebView.canGoForward();
-			return mWebView.canGoForward();
-		} else {
-			return false;
-		}
 	}
 
 	public WebView getWebView() {
@@ -583,11 +567,8 @@ public class LightningView {
 		public WebResourceResponse shouldInterceptRequest(WebView view,
 				String url) {
 			if (mAdBlock.isAd(url)) {
-				ByteArrayInputStream EMPTY = new ByteArrayInputStream(
+				ByteArrayInputStream EMPTY = new ByteArrayInputStream("".getBytes());
-						"".getBytes());
+				return new WebResourceResponse("text/plain", "utf-8", EMPTY);
-				WebResourceResponse response = new WebResourceResponse(
-						"text/plain", "utf-8", EMPTY);
-				return response;
 			}
 
 			boolean useProxy = mPreferences.getBoolean(
@@ -650,12 +631,12 @@ public class LightningView {
 				}
 
 				if (cType != null && cType.startsWith("text")) {
-					InputStream fStream = null;
+					InputStream fStream;
 
 					BufferedInputStream bis = new BufferedInputStream(
 							conn.getInputStream());
 					ByteArrayBuffer baf = new ByteArrayBuffer(connLen);
-					int read = 0;
+					int read;
 					int bufSize = 2048;
 					byte[] buffer = new byte[bufSize];
 					while (true) {
@@ -681,10 +662,7 @@ public class LightningView {
 					fStream = new ReplacingInputStream(fStream,
 							"\"poster\"".getBytes(), "\"foo\"".getBytes());
 
-					WebResourceResponse response = new WebResourceResponse(
+					return new WebResourceResponse(cType, cEnc, fStream);
-							cType, cEnc, fStream);
-
-					return response;
 				}/**
 				 * else if (mDoLeakHardening) { WebResourceResponse response =
 				 * new WebResourceResponse( cType, cEnc, conn.getInputStream());
@@ -700,11 +678,8 @@ public class LightningView {
 				Log.e(Constants.TAG, "Error filtering stream", e);
 				ByteArrayInputStream EMPTY = new ByteArrayInputStream(
 						"".getBytes());
-				WebResourceResponse response = new WebResourceResponse(
+				return new WebResourceResponse("text/plain", "utf-8", EMPTY);
-						"text/plain", "utf-8", EMPTY);
-				return response;
 			}
-
 		}
 
 		@Override
@@ -714,7 +689,7 @@ public class LightningView {
 			}
 			if (view.getTitle() == null) {
 				mTitle.setTitle(mActivity.getString(R.string.untitled));
-			} else if (view.getTitle().length() > 0) {
+			} else if (!view.getTitle().isEmpty()) {
 				mTitle.setTitle(view.getTitle());
 			} else {
 				mTitle.setTitle(mActivity.getString(R.string.untitled));
@@ -912,7 +887,7 @@ public class LightningView {
 
 		@Override
 		public void onReceivedTitle(WebView view, String title) {
-			if (title.length() > 0) {
+			if (!title.isEmpty()) {
 				mTitle.setTitle(title);
 			} else {
 				mTitle.setTitle(mActivity.getString(R.string.untitled));
@@ -929,7 +904,7 @@ public class LightningView {
 			builder.setTitle(mActivity.getString(R.string.location));
 			String org = null;
 			if (origin.length() > 50) {
-				org = (String) origin.subSequence(0, 50) + "...";
+				org = origin.subSequence(0, 50) + "...";
 			} else {
 				org = origin;
 			}
@@ -1004,7 +979,7 @@ public class LightningView {
 
 		@Override
 		public void onShowCustomView(View view, CustomViewCallback callback) {
-		// While these lines might look like they work, in practive,
+		// While these lines might look like they work, in practice,
 		// Full-screen videos won't work correctly. I may test this out some more
 		//	if (view instanceof FrameLayout) {
 		//		FrameLayout frame = (FrameLayout) view;
@@ -1028,7 +1003,7 @@ public class LightningView {
 		@Deprecated
 		public void onShowCustomView(View view, int requestedOrientation,
 				CustomViewCallback callback) {
-		// While these lines might look like they work, in practive,
+		// While these lines might look like they work, in practice,
 		// Full-screen videos won't work correctly. I may test this out some more
 		//	if (view instanceof FrameLayout) {
 		//		FrameLayout frame = (FrameLayout) view;
@@ -1046,7 +1021,6 @@ public class LightningView {
 
 			super.onShowCustomView(view, requestedOrientation, callback);
 		}
-
 	}
 
 	public class Title {
@@ -1073,9 +1047,10 @@ public class LightningView {
 
 		public void setTitle(String title) {
 			if (title == null) {
-				title = "";
+				mTitle = "";
+			} else {
+				mTitle = title;
 			}
-			mTitle = title;
 		}
 
 		public void setTitleAndFavicon(String title, Bitmap favicon) {

src/acr/browser/lightning/PreferenceConstants.java

@@ -3,7 +3,10 @@
  */
 package acr.browser.lightning;
 
-public class PreferenceConstants {
+public final class PreferenceConstants {
+
+	private PreferenceConstants() {
+	}
 
 	public static final String ADOBE_FLASH_SUPPORT = "enableflash";
 

src/acr/browser/lightning/ReplacingInputStream.java

@@ -3,14 +3,15 @@ package acr.browser.lightning;
 import java.io.FilterInputStream;
 import java.io.IOException;
 import java.io.InputStream;
+import java.util.Deque;
 import java.util.Iterator;
 import java.util.LinkedList;
 
 public class ReplacingInputStream extends FilterInputStream {
 
-	LinkedList<Integer> inQueue = new LinkedList<Integer>();
+	Deque<Integer> inQueue = new LinkedList<Integer>();
 
-	LinkedList<Integer> outQueue = new LinkedList<Integer>();
+	Deque<Integer> outQueue = new LinkedList<Integer>();
 
 	final byte[] search, replacement;
 
@@ -70,6 +71,7 @@ public class ReplacingInputStream extends FilterInputStream {
 	/**
 	 * Returns false.  REFilterInputStream does not support mark() and reset() methods.
 	 */
+	@Override
 	public boolean markSupported() {
 		return false;
 	}
@@ -77,6 +79,7 @@ public class ReplacingInputStream extends FilterInputStream {
 	/**
 	 * Reads from the stream into the provided array.
 	 */
+	@Override
 	public int read(byte[] b, int off, int len) throws IOException {
 		int i;
 		int ok = 0;

src/acr/browser/lightning/SearchAdapter.java

@@ -45,7 +45,7 @@ public class SearchAdapter extends BaseAdapter implements Filterable {
 
 	private Context mContext;
 
-	private boolean mIncognito = false;
+	private boolean mIncognito;
 
 	public SearchAdapter(Context context, boolean incognito) {
 		mDatabaseHandler = new DatabaseHandler(context);
@@ -278,13 +278,13 @@ public class SearchAdapter extends BaseAdapter implements Filterable {
 					int counter = 0;
 					while (eventType != XmlPullParser.END_DOCUMENT) {
 						if (eventType == XmlPullParser.START_TAG) {
-							if (xpp.getName().equals("suggestion")) {
+							if ("suggestion".equals(xpp.getName())) {
 								String suggestion = xpp.getAttributeValue(null,
 										"data");
 								filter.add(new HistoryItem(mContext
 										.getString(R.string.suggestion)
 										+ " \""
-										+ suggestion + "\"", suggestion,
+										+ suggestion + '"', suggestion,
 										R.drawable.ic_search));
 								counter++;
 								if (counter >= 5) {

src/acr/browser/lightning/SettingsController.java

@@ -5,24 +5,24 @@ package acr.browser.lightning;
 
 public class SettingsController {
 
-	static boolean clearHistory = false;
+	private static boolean clearHistory;
 
 	/**
 	 * The purpose of this class is so that I can clear the dropdown history in the main activities if the user selects
 	 * to clear the history from the disk in advanced settings
 	 */
-	static void setClearHistory(boolean choice) {
+	public static void setClearHistory(boolean choice) {
 		clearHistory = choice;
 	}
 
 	/**
 	 * return the choice
 	 */
-	static boolean getClearHistory() {
+	public static boolean getClearHistory() {
 		if (clearHistory) {
 			clearHistory = false;
 			return true;
 		}
-		return clearHistory;
+		return false;
 	}
 }

src/acr/browser/lightning/Utils.java

@@ -18,7 +18,10 @@ import java.net.URISyntaxException;
 import java.util.ArrayList;
 import java.util.List;
 
-public class Utils {
+public final class Utils {
+
+	private Utils() {
+	}
 
 	public static void downloadFile(final Activity activity, final String url, final String userAgent,
 			final String contentDisposition, final boolean privateBrowsing) {
@@ -31,7 +34,7 @@ public class Utils {
 	public static synchronized void addBookmark(Context context, String title, String url) {
 		File book = new File(context.getFilesDir(), "bookmarks");
 		File bookUrl = new File(context.getFilesDir(), "bookurl");
-		if ((title.equals("Bookmarks") || title.equals("History")) && url.startsWith("file://")) {
+		if (("Bookmarks".equals(title) || "History".equals(title)) && url.startsWith("file://")) {
 			return;
 		}
 		try {
@@ -102,8 +105,7 @@ public class Utils {
 	 */
 	public static int convertToDensityPixels(Context context, int densityPixels) {
 		float scale = context.getResources().getDisplayMetrics().density;
-		int pixels = (int) (densityPixels * scale + 0.5f);
+		return (int) (densityPixels * scale + 0.5f);
-		return pixels;
 	}
 
 	public static String getDomainName(String url) {
@@ -169,6 +171,6 @@ public class Utils {
 			}
 		}
 		// The directory is now empty so delete it
-		return dir.delete();
+		return dir != null && dir.delete();
 	}
 }

src/acr/browser/lightning/WebAddress.java

@@ -10,8 +10,6 @@ import java.util.regex.Pattern;
 import static android.util.Patterns.GOOD_IRI_CHAR;
 
 /**
- * {@hide}
- * 
  * Web Address Parser
  * 
  * This is called WebAddress, rather than URL or URI, because it attempts to
@@ -54,15 +52,14 @@ public class WebAddress {
 					/* anchor */".*", Pattern.CASE_INSENSITIVE);
 
 	/**
-	 * parses given uriString.
+	 * Parses given URI-like string.
 	 */
-	public WebAddress(String address) throws Exception {
+	public WebAddress(String address) {
+
 		if (address == null) {
-			throw new NullPointerException();
+			throw new IllegalArgumentException("address can't be null");
 		}
 
-		// android.util.Log.d(LOGTAG, "WebAddress: " + address);
-
 		mScheme = "";
 		mHost = "";
 		mPort = -1;
@@ -71,73 +68,73 @@ public class WebAddress {
 
 		Matcher m = sAddressPattern.matcher(address);
 		String t;
-		if (m.matches()) {
+		if (!m.matches()) {
-			t = m.group(MATCH_GROUP_SCHEME);
+			throw new IllegalArgumentException("Parsing of address '" +
-			if (t != null) {
+					address + "' failed");
-				mScheme = t.toLowerCase(Locale.ROOT);
+		}
-			}
+
-			t = m.group(MATCH_GROUP_AUTHORITY);
+		t = m.group(MATCH_GROUP_SCHEME);
-			if (t != null) {
+		if (t != null) {
-				mAuthInfo = t;
+			mScheme = t.toLowerCase(Locale.ROOT);
-			}
+		}
-			t = m.group(MATCH_GROUP_HOST);
+		t = m.group(MATCH_GROUP_AUTHORITY);
-			if (t != null) {
+		if (t != null) {
-				mHost = t;
+			mAuthInfo = t;
-			}
+		}
-			t = m.group(MATCH_GROUP_PORT);
+		t = m.group(MATCH_GROUP_HOST);
-			if (t != null && t.length() > 0) {
+		if (t != null) {
-				// The ':' character is not returned by the regex.
+			mHost = t;
-				try {
+		}
-					mPort = Integer.parseInt(t);
+		t = m.group(MATCH_GROUP_PORT);
-				} catch (NumberFormatException ex) {
+		if (t != null && !t.isEmpty()) {
-					throw new Exception();
+			// The ':' character is not returned by the regex.
-				}
+			try {
+				mPort = Integer.parseInt(t);
+			} catch (NumberFormatException ex) {
+				throw new RuntimeException("Parsing of port number failed", ex);
 			}
-			t = m.group(MATCH_GROUP_PATH);
+		}
-			if (t != null && t.length() > 0) {
+		t = m.group(MATCH_GROUP_PATH);
-				/*
+		if (t != null && !t.isEmpty()) {
-				 * handle busted myspace frontpage redirect with missing initial
+			/*
-				 * "/"
+			 * handle busted myspace frontpage redirect with missing initial
-				 */
+			 * "/"
-				if (t.charAt(0) == '/') {
+			 */
-					mPath = t;
+			if (t.charAt(0) == '/') {
-				} else {
+				mPath = t;
-					mPath = "/" + t;
+			} else {
-				}
+				mPath = '/' + t;
 			}
-
-		} else {
-			// nothing found... outa here
-			throw new Exception();
 		}
 
 		/*
 		 * Get port from scheme or scheme from port, if necessary and possible
 		 */
-		if (mPort == 443 && mScheme.equals("")) {
+		if (mPort == 443 && "".equals(mScheme)) {
 			mScheme = "https";
 		} else if (mPort == -1) {
-			if (mScheme.equals("https")) {
+			if ("https".equals(mScheme)) {
 				mPort = 443;
 			} else {
 				mPort = 80; // default
 			}
 		}
-		if (mScheme.equals("")) {
+		if ("".equals(mScheme)) {
 			mScheme = "http";
 		}
 	}
 
 	@Override
 	public String toString() {
+
 		String port = "";
-		if ((mPort != 443 && mScheme.equals("https"))
+		if ((mPort != 443 && "https".equals(mScheme))
-				|| (mPort != 80 && mScheme.equals("http"))) {
+				|| (mPort != 80 && "http".equals(mScheme))) {
-			port = ":" + Integer.toString(mPort);
+			port = ':' + Integer.toString(mPort);
 		}
 		String authInfo = "";
-		if (mAuthInfo.length() > 0) {
+		if (!mAuthInfo.isEmpty()) {
-			authInfo = mAuthInfo + "@";
+			authInfo = mAuthInfo + '@';
 		}
 
 		return mScheme + "://" + authInfo + mHost + port + mPath;