Prevent Blimp from reloading default URL on resume from minimized state

blimp/client/app/android/java/src/org/chromium/blimp/BlimpRendererActivity.java

 // Copyright 2015 The Chromium Authors. All rights reserved.
 // Use of this source code is governed by a BSD-style license that can be
 // found in the LICENSE file.
 
 package org.chromium.blimp;
 
 import android.app.Activity;
 import android.content.Intent;
 import android.os.Bundle;
 import android.text.TextUtils;
@@ skipping 147 matching lines @@
         mWebInputBox.initialize(mBlimpClientSession);
 
         mTabControlFeature = new TabControlFeature(mBlimpClientSession, mBlimpView);
 
         handleUrl(getUrlFromIntent(getIntent()));
     }
 
     @Override
     protected void onNewIntent(Intent intent) {
         super.onNewIntent(intent);
-        handleUrl(getUrlFromIntent(intent));
+        if (getUrlFromIntent(intent) != null) {

[Khushal, 2016/04/29 04:37:30]

This is making the code really hard to follow. Let's pass the intent directly to
handleUrl, both in the startup complete case and new intent case, you can rename
it handleUrlFromIntent.

Add a bool to keep track of when we load the first Url and only load the default
url if the intent did not have a url and this is the first time we got a call to
handle the url request. You can explain this in the comments in the code and
that will make it easy to understand the flow.

[shaktisahu, 2016/05/03 19:47:28]

Done.

+            handleUrl(getUrlFromIntent(intent));
+        }
     }
 
     /**
      * Retrieve the URL from the Intent.
      * @param intent Intent to examine.
      * @return URL from the Intent, or null if a valid URL couldn't be found.
      */
     private String getUrlFromIntent(Intent intent) {
         if (intent == null) return null;
 
@@ skipping 51 matching lines @@
     }
 
     @Override
     public void onDisconnected(String reason) {
         Toast.makeText(this,
                      String.format(getResources().getString(R.string.network_disconnected), reason),
                      Toast.LENGTH_LONG)
                 .show();
     }
 }