/*
- * Copyright © 2016-2018 Soren Stoutner <soren@stoutner.com>.
+ * Copyright © 2016-2019 Soren Stoutner <soren@stoutner.com>.
*
* This file is part of Privacy Browser <https://www.stoutner.com/privacy-browser>.
*
import android.os.Bundle;
import android.support.annotation.NonNull;
import android.support.v4.content.ContextCompat;
-// We have to use `AppCompatDialogFragment` instead of `DialogFragment` or an error is produced on API <= 22. `android.support.v7.app.AlertDialog` also uses more of the horizontal screen real estate versus `android.app.AlertDialog's` smaller width.
+// `AppCompatDialogFragment` must be used instead of `DialogFragment` or an error is produced on API <= 22.
+// `android.support.v7.app.AlertDialog` also uses more of the horizontal screen real estate versus `android.app.AlertDialog's` smaller width.
import android.support.v7.app.AppCompatDialogFragment;
import android.util.Base64;
import android.view.LayoutInflater;
private int currentPageId;
public static UrlHistoryDialog loadBackForwardList(Context context, WebBackForwardList webBackForwardList) {
- // Create `argumentsBundle`.
+ // Create an arguments bundle.
Bundle argumentsBundle = new Bundle();
- // Store `currentPageIndex`.
+ // Store the current page index.
int currentPageIndex = webBackForwardList.getCurrentIndex();
- // Setup `urlArrayList` and `iconArrayList`.
+ // Setup the URL array list and the icon array list.
ArrayList<String> urlArrayList = new ArrayList<>();
ArrayList<String> iconBase64StringArrayList = new ArrayList<>();
- // Get the default favorite icon `Drawable`.
+ // Get the default favorite icon drawable. `ContextCompat` must be used until the minimum API >= 21.
Drawable defaultFavoriteIconDrawable = ContextCompat.getDrawable(context, R.drawable.world);
- // Convert `defaultFavoriteIconDrawable` to a `BitmapDrawable`.
+ // Convert the default favorite icon drawable to a `BitmapDrawable`.
BitmapDrawable defaultFavoriteIconBitmapDrawable = (BitmapDrawable) defaultFavoriteIconDrawable;
// Remove the incorrect lint error that `getBitmap()` might be null.
assert defaultFavoriteIconBitmapDrawable != null;
- // Extract a `Bitmap` from `defaultFavoriteIconBitmapDrawable`.
+ // Extract a `Bitmap` from the default favorite icon `BitmapDrawable`.
Bitmap defaultFavoriteIcon = defaultFavoriteIconBitmapDrawable.getBitmap();
- // Populate `urlArrayList` and `iconArrayList` from `webBackForwardList`.
+ // Populate the URL array list and the icon array list from `webBackForwardList`.
for (int i=0; i < webBackForwardList.getSize(); i++) {
// Store the URL.
urlArrayList.add(webBackForwardList.getItemAtIndex(i).getUrl());
- // Create a variable to store the icon `Bitmap`.
+ // Create a variable to store the icon bitmap.
Bitmap iconBitmap;
- // Store the icon `Bitmap`.
+ // Store the icon bitmap.
if (webBackForwardList.getItemAtIndex(i).getFavicon() == null) {
// If `webBackForwardList` does not have a favorite icon, use Privacy Browser's default world icon.
iconBitmap = defaultFavoriteIcon;
argumentsBundle.putStringArrayList("URL_History", urlArrayList);
argumentsBundle.putStringArrayList("Favorite_Icons", iconBase64StringArrayList);
- // Add `argumentsBundle` to this instance of `UrlHistoryDialog`.
+ // Add the arguments bundle to this instance of `UrlHistoryDialog`.
UrlHistoryDialog thisUrlHistoryDialog = new UrlHistoryDialog();
thisUrlHistoryDialog.setArguments(argumentsBundle);
return thisUrlHistoryDialog;
alertDialog.show();
// Instantiate a `HistoryArrayAdapter`.
- final HistoryArrayAdapter historyArrayAdapter = new HistoryArrayAdapter(getContext(), historyArrayList, currentPageId);
+ HistoryArrayAdapter historyArrayAdapter = new HistoryArrayAdapter(getContext(), historyArrayList, currentPageId);
// Get a handle for the list view.
ListView listView = alertDialog.findViewById(R.id.history_listview);
// Listen for clicks on entries in the list view.
listView.setOnItemClickListener((AdapterView<?> parent, View view, int position, long id) -> {
- // Convert the `long` `id` to an `int`.
+ // Convert the long ID to an int.
int itemId = (int) id;
// Only consume the click if it is not on the `currentPageId`.