Open link only with . com - Android Webview

Question

In the code below, link with mcpack are open in the Webview application, however, sometimes it opens in the default browser, which I do?

@Override
public boolean shouldOverrideUrlLoading(WebView view, final String url)
{
    if (url.contains("https://www.mediafire.com/file/") || (url.endsWith("mcpack") || (url.contains("https://www.mediafire.com/download_repair.php") || (url.contains("http://engine.addroplet.com/")) ))) { // Could be cleverer and use a regex
        mWebView.loadUrl(url);

        return false;



    }

    mWebView.stopLoading();
    mWebView.goBack();
    final AlertDialog alertDialog = new AlertDialog.Builder(Textures.this).create();
    alertDialog.setTitle("Navegador");
    alertDialog.setMessage("Deseja abrir o navegador?");
    alertDialog.setButton(DialogInterface.BUTTON_POSITIVE, "Sim", new DialogInterface.OnClickListener() {
            public void onClick(DialogInterface dialog, int which) {
                Intent intent = new Intent(Intent.ACTION_VIEW, Uri.parse(url));
                startActivity(intent);

            }

        });
    alertDialog.setButton(DialogInterface.BUTTON_NEGATIVE, "Não", new DialogInterface.OnClickListener() {
            public void onClick(DialogInterface dialog, int which) {
                alertDialog.dismiss();
            }
        });


    alertDialog.show();




    return false;
}

Answer 1

In the documentation Android of the method shouldOverrideUrlLoad says:

Do not call WebView.loadUrl(String) with the request’s URL and then return true. This unnecessarily cancels the Current load and Starts a new load with the same URL. The correct way to continue loading the Given URL is to Simply return false, without Calling WebView.loadUrl(String).

That summing up means: nay call the WebView.loadUrl this will cause another request to begin, just return false.

This could be what’s making your code work in an unstable way. You can take the mWebView.loadUrl(url); and only return false or put as the code below.

Remembering that if you return true he will abort the requisition.

@Override
public boolean shouldOverrideUrlLoading(WebView view, final String url)
{
    if (url.contains("https://www.mediafire.com/file/") || (url.endsWith("mcpack") || (url.contains("https://www.mediafire.com/download_repair.php") || (url.contains("http://engine.addroplet.com/")) ))) { // Could be cleverer and use a regex
        return super.shouldOverrideUrlLoading(view, url)
    }
}

Another detail is that you talk "link with mcpack" whereas in the code you are only checking if it ends with url.endsWith("mcpack"). Then it’s only worth you putting some Log.d in your code to check if you are actually entering this if.