X-Git-Url: https://gitweb.stoutner.com/?a=blobdiff_plain;f=app%2Fsrc%2Fmain%2Fjava%2Fcom%2Fstoutner%2Fprivacybrowser%2FWebview.java;h=a627a780f4af35d5661ee2f7dd2bdd182e4bba4a;hb=405b0fa00eef8a6239f1f7ae6b0db06edb23ccb9;hp=6e8f237e0f5beb7937776bbb7bb653a317ec5709;hpb=9523d99614150970890e0d01cf0bf5e38c6ccf14;p=PrivacyBrowserAndroid.git

diff --git a/app/src/main/java/com/stoutner/privacybrowser/Webview.java b/app/src/main/java/com/stoutner/privacybrowser/Webview.java
index 6e8f237e..a627a780 100644
--- a/app/src/main/java/com/stoutner/privacybrowser/Webview.java
+++ b/app/src/main/java/com/stoutner/privacybrowser/Webview.java
@@ -3,24 +3,24 @@ package com.stoutner.privacybrowser;
 import android.app.Activity;
 import android.content.Intent;
 import android.net.Uri;
-import android.support.v7.app.ActionBarActivity;
 import android.os.Bundle;
+import android.support.v7.app.AppCompatActivity;
 import android.view.KeyEvent;
 import android.view.Menu;
 import android.view.MenuItem;
 import android.view.View;
 import android.view.inputmethod.InputMethodManager;
+import android.webkit.WebChromeClient;
 import android.webkit.WebView;
 import android.webkit.WebViewClient;
 import android.widget.EditText;
+import android.widget.ProgressBar;
 
-import java.net.URL;
-
-
-public class Webview extends ActionBarActivity {
+public class Webview extends AppCompatActivity {
 
     static String formattedUrlString;
     static WebView mainWebView;
+    static ProgressBar progressBar;
     static final String homepage = "https://www.duckduckgo.com";
 
     @Override
@@ -30,9 +30,30 @@ public class Webview extends ActionBarActivity {
 
         final EditText urlTextBox = (EditText) findViewById(R.id.urlTextBox);
         mainWebView = (WebView) findViewById(R.id.mainWebView);
+        progressBar = (ProgressBar) findViewById(R.id.progressBar);
 
         // setWebViewClient makes this WebView the default handler for URLs inside the app, so that links are not kicked out to other apps.
-        mainWebView.setWebViewClient(new WebViewClient());
+        // Save the URL to formattedUrlString and update urlTextBox before loading mainWebView.
+        mainWebView.setWebViewClient(new WebViewClient() {
+            public boolean shouldOverrideUrlLoading(WebView view, String url) {
+                formattedUrlString=url;
+                urlTextBox.setText(formattedUrlString);
+                mainWebView.loadUrl(url);
+                return true;
+            }
+        });
+
+        // Update the progress bar when a page is loading.
+        mainWebView.setWebChromeClient(new WebChromeClient() {
+            public void onProgressChanged(WebView view, int progress) {
+                progressBar.setProgress(progress);
+                if (progress < 100) {
+                    progressBar.setVisibility(View.VISIBLE);
+                } else {
+                    progressBar.setVisibility(View.GONE);
+                }
+            }
+        });
 
         // Allow pinch to zoom.
         mainWebView.getSettings().setBuiltInZoomControls(true);
@@ -50,18 +71,9 @@ public class Webview extends ActionBarActivity {
         final Intent intent = getIntent();
 
         if (intent.getData() != null) {
-            // Get the intent data.
+            // Get the intent data and convert it to a string.
             final Uri intentUriData = intent.getData();
-
-            // Try to parse the intent data and store it in urlData.
-            URL urlData = null;
-            try {
-                urlData = new URL(intentUriData.getScheme(), intentUriData.getHost(), intentUriData.getPath());
-            } catch (Exception e) {
-                e.printStackTrace();
-            }
-
-            Webview.formattedUrlString = urlData.toString();
+            Webview.formattedUrlString = intentUriData.toString();
         }
 
         // If formattedUrlString is null assign the homepage to it.
@@ -80,7 +92,7 @@ public class Webview extends ActionBarActivity {
                 if ((event.getAction() == KeyEvent.ACTION_DOWN) &&
                         (keyCode == KeyEvent.KEYCODE_ENTER)) {
                     // Load the URL into the mainWebView and consume the event.
-                    loadUrl(mainWebView);
+                    loadUrlFromTextBox(mainWebView);
                     return true;
                 }
                 // Do not consume the event.
@@ -102,8 +114,11 @@ public class Webview extends ActionBarActivity {
         int menuItemId = menuItem.getItemId();
         final WebView mainWebView = (WebView) findViewById(R.id.mainWebView);
 
-        // Use the menu items to go forward or back.
+        // Sets the commands that relate to the menu entries.
         switch (menuItemId) {
+            case R.id.home:
+                mainWebView.loadUrl(homepage);
+                break;
             case R.id.back:
                 mainWebView.goBack();
                 break;
@@ -125,7 +140,7 @@ public class Webview extends ActionBarActivity {
         }
     }
 
-    public void loadUrl(View view) {
+    public void loadUrlFromTextBox(View view) {
         // Get the text from urlTextInput and convert it to a string.
         final EditText urlTextBox = (EditText) findViewById(R.id.urlTextBox);
         final String unformattedUrlString = urlTextBox.getText().toString();
@@ -150,24 +165,6 @@ public class Webview extends ActionBarActivity {
                 formattedUrlString = "http://" + unformattedUrlString;
         }
 
-        /*
-        // Parse the unformattedURLString into a Uri.
-
-        final Uri uriData = Uri.parse(unformattedUrlString);
-
-        // Convert the Uri to a URL, chicking for any problems with the formatting.
-        URL urlData = null;
-
-        try {
-            urlData = new URL(uriData.getScheme(), uriData.getHost(), uriData.getPath());
-        } catch (Exception e) {
-            e.printStackTrace();
-        }
-
-        //  Convert urlData to a string, which is reqauired by loadUrl method.
-        formattedUrlString = urlData.toString();
-        */
-
         final WebView mainWebView = (WebView) findViewById(R.id.mainWebView);
 
         // Place the URL text back in the address bar and load the website.