Android app: highlighting and inhibiting of links

5 files changed, 89 insertions, 15 deletions

diff --git a/android/src/com/artifex/mupdf/LinkInfo.java b/android/src/com/artifex/mupdf/LinkInfo.java
new file mode 100644
index 00000000..c2a395e7
--- /dev/null
+++ b/android/src/com/artifex/mupdf/LinkInfo.java
@@ -0,0 +1,12 @@
+package com.artifex.mupdf;
+
+import android.graphics.RectF;
+
+public class LinkInfo extends RectF {
+	public int pageNumber;
+
+	public LinkInfo(float l, float t, float r, float b, int p) {
+		super(l, t, r, b);
+		pageNumber = p;
+	}
+}
diff --git a/android/src/com/artifex/mupdf/MuPDFActivity.java b/android/src/com/artifex/mupdf/MuPDFActivity.java
index 8895379a..416c1834 100644
--- a/android/src/com/artifex/mupdf/MuPDFActivity.java
+++ b/android/src/com/artifex/mupdf/MuPDFActivity.java
@@ -42,6 +42,7 @@ class SearchTaskResult {
 public class MuPDFActivity extends Activity
 {
 	/* The core rendering instance */
+	private enum LinkState {DEFAULT, HIGHLIGHT, INHIBIT};
 	private final int    TAP_PAGE_MARGIN = 5;
 	private MuPDFCore    core;
 	private String       mFileName;
@@ -56,7 +57,7 @@ public class MuPDFActivity extends Activity
 	private ImageButton  mCancelButton;
 	private ImageButton  mOutlineButton;
 	private ViewSwitcher mTopBarSwitcher;
-	private View         mLowerButtons;
+	private ImageButton  mLinkButton;
 	private boolean      mTopBarIsSearch;
 	private ImageButton  mSearchBack;
 	private ImageButton  mSearchFwd;
@@ -64,6 +65,7 @@ public class MuPDFActivity extends Activity
 	private AsyncTask<Integer,Integer,SearchTaskResult> mSearchTask;
 	private SearchTaskResult mSearchTaskResult;
 	private AlertDialog.Builder mAlertBuilder;
+	private LinkState    mLinkState = LinkState.DEFAULT;
 
 	private MuPDFCore openFile(String path)
 	{
@@ -180,9 +182,11 @@ public class MuPDFActivity extends Activity
 					super.moveToNext();
 				} else if (!showButtonsDisabled) {
 					int linkPage = -1;
-					MuPDFPageView pageView = (MuPDFPageView) mDocView.getDisplayedView();
-					if (pageView != null) {
-						linkPage = pageView.hitLinkPage(e.getX(), e.getY());
+					if (mLinkState != LinkState.INHIBIT) {
+						MuPDFPageView pageView = (MuPDFPageView) mDocView.getDisplayedView();
+						if (pageView != null) {
+							linkPage = pageView.hitLinkPage(e.getX(), e.getY());
+						}
 					}
 
 					if (linkPage != -1) {
@@ -225,6 +229,8 @@ public class MuPDFActivity extends Activity
 					((PageView)v).setSearchBoxes(mSearchTaskResult.searchBoxes);
 				else
 					((PageView)v).setSearchBoxes(null);
+
+				((PageView)v).setLinkHighlighting(mLinkState == LinkState.HIGHLIGHT);
 			}
 
 			protected void onMoveToChild(int i) {
@@ -332,6 +338,29 @@ public class MuPDFActivity extends Activity
 			}
 		});
 
+		mLinkButton.setOnClickListener(new View.OnClickListener() {
+			public void onClick(View v) {
+				switch(mLinkState) {
+				case DEFAULT:
+					mLinkState = LinkState.HIGHLIGHT;
+					mLinkButton.setImageResource(R.drawable.ic_hl_link);
+					//Inform pages of the change.
+					mDocView.resetupChildren();
+					break;
+				case HIGHLIGHT:
+					mLinkState = LinkState.INHIBIT;
+					mLinkButton.setImageResource(R.drawable.ic_nolink);
+					//Inform pages of the change.
+					mDocView.resetupChildren();
+					break;
+				case INHIBIT:
+					mLinkState = LinkState.DEFAULT;
+					mLinkButton.setImageResource(R.drawable.ic_link);
+					break;
+				}
+			}
+		});
+
 		if (core.hasOutline()) {
 			mOutlineButton.setOnClickListener(new View.OnClickListener() {
 				public void onClick(View v) {
@@ -530,7 +559,7 @@ public class MuPDFActivity extends Activity
 		mSearchBack = (ImageButton)mButtonsView.findViewById(R.id.searchBack);
 		mSearchFwd = (ImageButton)mButtonsView.findViewById(R.id.searchForward);
 		mSearchText = (EditText)mButtonsView.findViewById(R.id.searchText);
-		mLowerButtons = mButtonsView.findViewById(R.id.lowerButtons);
+		mLinkButton = (ImageButton)mButtonsView.findViewById(R.id.linkButton);
 		mTopBarSwitcher.setVisibility(View.INVISIBLE);
 		mPageNumberView.setVisibility(View.INVISIBLE);
 		mPageSlider.setVisibility(View.INVISIBLE);
diff --git a/android/src/com/artifex/mupdf/MuPDFCore.java b/android/src/com/artifex/mupdf/MuPDFCore.java
index fbdd3afa..a4b80fef 100644
--- a/android/src/com/artifex/mupdf/MuPDFCore.java
+++ b/android/src/com/artifex/mupdf/MuPDFCore.java
@@ -28,6 +28,7 @@ public class MuPDFCore
 			int patchW, int patchH);
 	public static native RectF[] searchPage(String text);
 	public static native int getPageLink(int page, float x, float y);
+	public static native LinkInfo [] getPageLinksInternal(int page);
 	public static native OutlineItem [] getOutlineInternal();
 	public static native boolean hasOutlineInternal();
 	public static native boolean needsPasswordInternal();
@@ -90,6 +91,10 @@ public class MuPDFCore
 		return getPageLink(page, x, y);
 	}
 
+	public synchronized LinkInfo [] getPageLinks(int page) {
+		return getPageLinksInternal(page);
+	}
+
 	public synchronized RectF [] searchPage(int page, String text) {
 		gotoPage(page);
 		return searchPage(text);
diff --git a/android/src/com/artifex/mupdf/MuPDFPageView.java b/android/src/com/artifex/mupdf/MuPDFPageView.java
index fc5de6ba..4678fc9c 100644
--- a/android/src/com/artifex/mupdf/MuPDFPageView.java
+++ b/android/src/com/artifex/mupdf/MuPDFPageView.java
@@ -13,6 +13,10 @@ public class MuPDFPageView extends PageView {
 	}
 
 	public int hitLinkPage(float x, float y) {
+		// Since link highlighting was implemented, the super class
+		// PageView has had sufficient information to be able to
+		// perform this method directly. Making that change would
+		// make MuPDFCore.hitLinkPage superfluous.
 		float scale = mSourceScale*(float)getWidth()/(float)mSize.x;
 		float docRelX = (x - getLeft())/scale;
 		float docRelY = (y - getTop())/scale;
@@ -25,4 +29,9 @@ public class MuPDFPageView extends PageView {
 			int patchX, int patchY, int patchWidth, int patchHeight) {
 		mCore.drawPage(mPageNumber, bm, sizeX, sizeY, patchX, patchY, patchWidth, patchHeight);
 	}
+
+	@Override
+	protected LinkInfo[] getLinkInfo() {
+		return mCore.getPageLinks(mPageNumber);
+	}
 }
diff --git a/android/src/com/artifex/mupdf/PageView.java b/android/src/com/artifex/mupdf/PageView.java
index 4dce6d23..7c2e38d4 100644
--- a/android/src/com/artifex/mupdf/PageView.java
+++ b/android/src/com/artifex/mupdf/PageView.java
@@ -42,6 +42,7 @@ class OpaqueImageView extends ImageView {
 
 public abstract class PageView extends ViewGroup {
 	private static final int HIGHLIGHT_COLOR = 0x805555FF;
+	private static final int LINK_COLOR = 0x80FFCC88;
 	private static final int BACKGROUND_COLOR = 0xFFFFFFFF;
 	private final Context   mContext;
 	protected     int       mPageNumber;
@@ -51,16 +52,18 @@ public abstract class PageView extends ViewGroup {
 
 	private       ImageView mEntire; // Image rendered at minimum zoom
 	private       Bitmap    mEntireBm;
-	private       AsyncTask<Void,Void,Void> mDrawEntire;
+	private       AsyncTask<Void,Void,LinkInfo[]> mDrawEntire;
 
 	private       Point     mPatchViewSize; // View size on the basis of which the patch was created
 	private       Rect      mPatchArea;
 	private       ImageView mPatch;
 	private       AsyncTask<PatchInfo,Void,PatchInfo> mDrawPatch;
 	private       RectF     mSearchBoxes[];
+	private       LinkInfo  mLinks[];
 	private       View      mSearchView;
 	private       boolean   mIsBlank;
 	private       boolean   mUsingHardwareAcceleration;
+	private       boolean   mHighlightLinks;
 
 	private       ProgressBar mBusyIndicator;
 
@@ -73,6 +76,7 @@ public abstract class PageView extends ViewGroup {
 	}
 
 	protected abstract void drawPage(Bitmap bm, int sizeX, int sizeY, int patchX, int patchY, int patchWidth, int patchHeight);
+	protected abstract LinkInfo[] getLinkInfo();
 
 	public void blank(int page) {
 		// Cancel pending render task
@@ -137,10 +141,10 @@ public abstract class PageView extends ViewGroup {
 		}
 
 		// Render the page in the background
-		mDrawEntire = new AsyncTask<Void,Void,Void>() {
-			protected Void doInBackground(Void... v) {
+		mDrawEntire = new AsyncTask<Void,Void,LinkInfo[]>() {
+			protected LinkInfo[] doInBackground(Void... v) {
 				drawPage(mEntireBm, mSize.x, mSize.y, 0, 0, mSize.x, mSize.y);
-				return null;
+				return getLinkInfo();
 			}
 
 			protected void onPreExecute() {
@@ -154,10 +158,11 @@ public abstract class PageView extends ViewGroup {
 				}
 			}
 
-			protected void onPostExecute(Void v) {
+			protected void onPostExecute(LinkInfo[] v) {
 				removeView(mBusyIndicator);
 				mBusyIndicator = null;
 				mEntire.setImageBitmap(mEntireBm);
+				mLinks = v;
 				invalidate();
 			}
 		};
@@ -169,17 +174,27 @@ public abstract class PageView extends ViewGroup {
 				@Override
 				protected void onDraw(Canvas canvas) {
 					super.onDraw(canvas);
+					float scale = mSourceScale*(float)getWidth()/(float)mSize.x;
+					Paint paint = new Paint();
 
 					if (!mIsBlank && mSearchBoxes != null) {
 						// Work out current total scale factor
 						// from source to view
-						float scale = mSourceScale*(float)getWidth()/(float)mSize.x;
-						Paint paint = new Paint();
 						paint.setColor(HIGHLIGHT_COLOR);
 						for (RectF rect : mSearchBoxes)
-							canvas.drawRect(new RectF(rect.left * scale,
-									rect.top * scale, rect.right * scale,
-									rect.bottom * scale), paint);
+							canvas.drawRect(rect.left*scale, rect.top*scale,
+									        rect.right*scale, rect.bottom*scale,
+									        paint);
+					}
+
+					if (!mIsBlank && mLinks != null && mHighlightLinks) {
+						// Work out current total scale factor
+						// from source to view
+						paint.setColor(LINK_COLOR);
+						for (RectF rect : mLinks)
+							canvas.drawRect(rect.left*scale, rect.top*scale,
+									        rect.right*scale, rect.bottom*scale,
+									        paint);
 					}
 				}
 			};
@@ -193,6 +208,10 @@ public abstract class PageView extends ViewGroup {
 		mSearchBoxes = searchBoxes;
 	}
 
+	public void setLinkHighlighting(boolean f) {
+		mHighlightLinks = f;
+	}
+
 	@Override
 	protected void onMeasure(int widthMeasureSpec, int heightMeasureSpec) {
 		int x, y;