Make HistoryPage an AsyncTask to be easier to use
This commit is contained in:
parent
24385c4334
commit
f6b60894f6
@ -1400,22 +1400,7 @@ public abstract class BrowserActivity extends ThemableBrowserActivity implements
|
||||
* function that opens the HTML history page in the browser
|
||||
*/
|
||||
private void openHistory() {
|
||||
// use a thread so that history retrieval doesn't block the UI
|
||||
BrowserApp.getIOThread().execute(new Runnable() {
|
||||
|
||||
@Override
|
||||
public void run() {
|
||||
final String historyPage = HistoryPage.getHistoryPage(BrowserActivity.this);
|
||||
BrowserActivity.this.runOnUiThread(new Runnable() {
|
||||
@Override
|
||||
public void run() {
|
||||
loadUrlInCurrentView(historyPage);
|
||||
mSearch.setText("");
|
||||
}
|
||||
});
|
||||
}
|
||||
|
||||
});
|
||||
new HistoryPage(mTabsManager.getCurrentTab(), getApplication()).load();
|
||||
}
|
||||
|
||||
/**
|
||||
@ -2002,6 +1987,11 @@ public abstract class BrowserActivity extends ThemableBrowserActivity implements
|
||||
}, 150);
|
||||
}
|
||||
|
||||
@Subscribe
|
||||
public void loadHistory(final BrowserEvents.OpenHistoryInCurrentTab event) {
|
||||
new HistoryPage(mTabsManager.getCurrentTab(), getApplication()).load();
|
||||
}
|
||||
|
||||
/**
|
||||
* Load the given url in a new tab, used by the the
|
||||
* {@link acr.browser.lightning.fragment.BookmarksFragment} and by the
|
||||
|
@ -16,6 +16,7 @@ import java.lang.ref.WeakReference;
|
||||
import java.util.List;
|
||||
|
||||
import acr.browser.lightning.R;
|
||||
import acr.browser.lightning.app.BrowserApp;
|
||||
import acr.browser.lightning.database.BookmarkManager;
|
||||
import acr.browser.lightning.database.HistoryItem;
|
||||
import acr.browser.lightning.utils.Utils;
|
||||
@ -65,7 +66,7 @@ public final class BookmarkPage extends AsyncTask<Void, Void, Void> {
|
||||
private final Bitmap mFolderIcon;
|
||||
private final String mTitle;
|
||||
|
||||
public BookmarkPage(BookmarkManager manager, LightningView tab, Application app, Bitmap folderIcon) {
|
||||
public BookmarkPage(LightningView tab, Application app, BookmarkManager manager, Bitmap folderIcon) {
|
||||
mFilesDir = app.getFilesDir();
|
||||
mCacheDir = app.getCacheDir();
|
||||
mTitle = app.getString(R.string.action_bookmarks);
|
||||
@ -147,4 +148,8 @@ public final class BookmarkPage extends AsyncTask<Void, Void, Void> {
|
||||
}
|
||||
}
|
||||
|
||||
public void load() {
|
||||
executeOnExecutor(BrowserApp.getIOThread());
|
||||
}
|
||||
|
||||
}
|
||||
|
@ -3,27 +3,31 @@
|
||||
*/
|
||||
package acr.browser.lightning.constant;
|
||||
|
||||
import android.app.Application;
|
||||
import android.os.AsyncTask;
|
||||
import android.support.annotation.NonNull;
|
||||
|
||||
import java.io.File;
|
||||
import java.io.FileWriter;
|
||||
import java.io.IOException;
|
||||
import java.lang.ref.WeakReference;
|
||||
import java.util.Iterator;
|
||||
import java.util.List;
|
||||
|
||||
import android.content.Context;
|
||||
|
||||
import acr.browser.lightning.app.BrowserApp;
|
||||
import acr.browser.lightning.database.HistoryItem;
|
||||
import acr.browser.lightning.R;
|
||||
import acr.browser.lightning.app.BrowserApp;
|
||||
import acr.browser.lightning.database.HistoryDatabase;
|
||||
import acr.browser.lightning.database.HistoryItem;
|
||||
import acr.browser.lightning.utils.Utils;
|
||||
import acr.browser.lightning.view.LightningView;
|
||||
|
||||
public class HistoryPage {
|
||||
public class HistoryPage extends AsyncTask<Void, Void, Void> {
|
||||
|
||||
public static final String FILENAME = "history.html";
|
||||
|
||||
private static final String HEADING = "<!DOCTYPE html><html xmlns=\"http://www.w3.org/1999/xhtml\"><head><meta content=\"en-us\" http-equiv=\"Content-Language\" /><meta content=\"text/html; charset=utf-8\" http-equiv=\"Content-Type\" /><meta name=\"viewport\" content=\"width=device-width, initial-scale=1.0, maximum-scale=1.0, user-scalable=no\"><title>"
|
||||
+ BrowserApp.getContext().getString(R.string.action_history)
|
||||
+ "</title></head><style>body { background: #e1e1e1;}.box { vertical-align:middle;position:relative; display: block; margin: 10px;padding-left:10px;padding-right:10px;padding-top:5px;padding-bottom:5px; background-color:#fff;box-shadow: 0px 2px 3px rgba( 0, 0, 0, 0.25 );font-family: Arial;color: #444;font-size: 12px;-moz-border-radius: 2px;-webkit-border-radius: 2px;border-radius: 2px;}.box a { width: 100%; height: 100%; position: absolute; left: 0; top: 0;}.black {color: black;font-size: 15px;font-family: Arial; white-space: nowrap; overflow: hidden;margin:auto; text-overflow: ellipsis; -o-text-overflow: ellipsis; -ms-text-overflow: ellipsis;}.font {color: gray;font-size: 10px;font-family: Arial; white-space: nowrap; overflow: hidden;margin:auto; text-overflow: ellipsis; -o-text-overflow: ellipsis; -ms-text-overflow: ellipsis;}</style><body><div id=\"content\">";
|
||||
private static final String HEADING_1 = "<!DOCTYPE html><html xmlns=\"http://www.w3.org/1999/xhtml\"><head><meta content=\"en-us\" http-equiv=\"Content-Language\" /><meta content=\"text/html; charset=utf-8\" http-equiv=\"Content-Type\" /><meta name=\"viewport\" content=\"width=device-width, initial-scale=1.0, maximum-scale=1.0, user-scalable=no\"><title>";
|
||||
|
||||
private static final String HEADING_2 = "</title></head><style>body { background: #e1e1e1;}.box { vertical-align:middle;position:relative; display: block; margin: 10px;padding-left:10px;padding-right:10px;padding-top:5px;padding-bottom:5px; background-color:#fff;box-shadow: 0px 2px 3px rgba( 0, 0, 0, 0.25 );font-family: Arial;color: #444;font-size: 12px;-moz-border-radius: 2px;-webkit-border-radius: 2px;border-radius: 2px;}.box a { width: 100%; height: 100%; position: absolute; left: 0; top: 0;}.black {color: black;font-size: 15px;font-family: Arial; white-space: nowrap; overflow: hidden;margin:auto; text-overflow: ellipsis; -o-text-overflow: ellipsis; -ms-text-overflow: ellipsis;}.font {color: gray;font-size: 10px;font-family: Arial; white-space: nowrap; overflow: hidden;margin:auto; text-overflow: ellipsis; -o-text-overflow: ellipsis; -ms-text-overflow: ellipsis;}</style><body><div id=\"content\">";
|
||||
|
||||
private static final String PART1 = "<div class=\"box\"><a href=\"";
|
||||
|
||||
@ -35,24 +39,52 @@ public class HistoryPage {
|
||||
|
||||
private static final String END = "</div></body></html>";
|
||||
|
||||
public static String getHistoryPage(Context context) {
|
||||
StringBuilder historyBuilder = new StringBuilder(HistoryPage.HEADING);
|
||||
private final WeakReference<LightningView> mTabReference;
|
||||
private final File mFilesDir;
|
||||
private final String mTitle;
|
||||
|
||||
private String mHistoryUrl = null;
|
||||
|
||||
public HistoryPage(LightningView tab, Application app) {
|
||||
mTabReference = new WeakReference<>(tab);
|
||||
mFilesDir = app.getFilesDir();
|
||||
mTitle = app.getString(R.string.action_history);
|
||||
}
|
||||
|
||||
@Override
|
||||
protected Void doInBackground(Void... params) {
|
||||
mHistoryUrl = getHistoryPage();
|
||||
return null;
|
||||
}
|
||||
|
||||
@Override
|
||||
protected void onPostExecute(Void aVoid) {
|
||||
super.onPostExecute(aVoid);
|
||||
LightningView tab = mTabReference.get();
|
||||
if (tab != null && mHistoryUrl != null) {
|
||||
tab.loadUrl(mHistoryUrl);
|
||||
}
|
||||
}
|
||||
|
||||
@NonNull
|
||||
private String getHistoryPage() {
|
||||
StringBuilder historyBuilder = new StringBuilder(HEADING_1 + mTitle + HEADING_2);
|
||||
List<HistoryItem> historyList = getWebHistory();
|
||||
Iterator<HistoryItem> it = historyList.iterator();
|
||||
HistoryItem helper;
|
||||
while (it.hasNext()) {
|
||||
helper = it.next();
|
||||
historyBuilder.append(HistoryPage.PART1);
|
||||
historyBuilder.append(PART1);
|
||||
historyBuilder.append(helper.getUrl());
|
||||
historyBuilder.append(HistoryPage.PART2);
|
||||
historyBuilder.append(PART2);
|
||||
historyBuilder.append(helper.getTitle());
|
||||
historyBuilder.append(HistoryPage.PART3);
|
||||
historyBuilder.append(PART3);
|
||||
historyBuilder.append(helper.getUrl());
|
||||
historyBuilder.append(HistoryPage.PART4);
|
||||
historyBuilder.append(PART4);
|
||||
}
|
||||
|
||||
historyBuilder.append(HistoryPage.END);
|
||||
File historyWebPage = new File(context.getFilesDir(), FILENAME);
|
||||
historyBuilder.append(END);
|
||||
File historyWebPage = new File(mFilesDir, FILENAME);
|
||||
FileWriter historyWriter = null;
|
||||
try {
|
||||
//noinspection IOResourceOpenedButNotSafelyClosed
|
||||
@ -70,4 +102,9 @@ public class HistoryPage {
|
||||
HistoryDatabase databaseHandler = BrowserApp.getHistoryDatabase();
|
||||
return databaseHandler.getLastHundredItems();
|
||||
}
|
||||
}
|
||||
|
||||
public void load() {
|
||||
executeOnExecutor(BrowserApp.getIOThread());
|
||||
}
|
||||
|
||||
}
|
@ -181,4 +181,8 @@ public class StartPage extends AsyncTask<Void, Void, Void> {
|
||||
return Constants.FILE + homepage;
|
||||
}
|
||||
|
||||
public void load() {
|
||||
executeOnExecutor(BrowserApp.getIOThread());
|
||||
}
|
||||
|
||||
}
|
||||
|
@ -34,9 +34,6 @@ import android.webkit.WebView;
|
||||
|
||||
import com.squareup.otto.Bus;
|
||||
|
||||
import java.io.File;
|
||||
import java.io.FileNotFoundException;
|
||||
import java.io.FileOutputStream;
|
||||
import java.lang.ref.WeakReference;
|
||||
import java.util.Map;
|
||||
|
||||
@ -177,7 +174,7 @@ public class LightningView {
|
||||
* UI thread.
|
||||
*/
|
||||
private void loadStartpage() {
|
||||
new StartPage(this, BrowserApp.get(mActivity)).executeOnExecutor(BrowserApp.getIOThread());
|
||||
new StartPage(this, BrowserApp.get(mActivity)).load();
|
||||
}
|
||||
|
||||
/**
|
||||
@ -189,7 +186,7 @@ public class LightningView {
|
||||
if (mWebView == null)
|
||||
return;
|
||||
Bitmap folderIcon = ThemeUtils.getThemedBitmap(mActivity, R.drawable.ic_folder, false);
|
||||
new BookmarkPage(mBookmarkManager, this, BrowserApp.get(mActivity), folderIcon).executeOnExecutor(BrowserApp.getIOThread());
|
||||
new BookmarkPage(this, BrowserApp.get(mActivity), mBookmarkManager, folderIcon).load();
|
||||
}
|
||||
|
||||
/**
|
||||
|
Loading…
Reference in New Issue
Block a user