Adding option for Stop/Reload in location bar for content shell.

content/shell/android/java/src/org/chromium/content_shell/Shell.java

 // Copyright 2012 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.content_shell;
 
 import android.content.Context;
 import android.graphics.drawable.ClipDrawable;
 import android.text.TextUtils;
 import android.util.AttributeSet;
@@ skipping 31 matching lines @@
         public void run() {
             mProgressDrawable.setLevel(0);
         }
     };
 
     private ContentViewCore mContentViewCore;
     private ContentViewClient mContentViewClient;
     private EditText mUrlTextView;
     private ImageButton mPrevButton;
     private ImageButton mNextButton;
+    private ImageButton mStopButton;
+    private ImageButton mReloadButton;
 
     private ClipDrawable mProgressDrawable;
 
     private long mNativeShell;
     private ContentViewRenderView mContentViewRenderView;
     private WindowAndroid mWindow;
 
     private boolean mLoading = false;
 
     /**
@@ skipping 144 matching lines @@
             }
         });
 
         mNextButton = (ImageButton) findViewById(R.id.next);
         mNextButton.setOnClickListener(new OnClickListener() {
             @Override
             public void onClick(View v) {
                 if (mContentViewCore.canGoForward()) mContentViewCore.goForward();
             }
         });
+        mStopButton = (ImageButton)findViewById(R.id.stop);
+        mStopButton.setOnClickListener(new OnClickListener() {
+            @Override
+            public void onClick(View v) {
+                if (mLoading)
+                    mContentViewCore.stopLoading();

[Ted C, 2014/08/28 16:21:13]

in java, braces are required unless the statement and conditional can all fit on
the same line.  I think in this case, you should be able to just move this line
up.

+            }
+        });
+        mReloadButton = (ImageButton)findViewById(R.id.reload);
+        mReloadButton.setOnClickListener(new OnClickListener() {
+            @Override
+            public void onClick(View v) {
+                mContentViewCore.reload(true);
+            }
+        });
     }
 
     @SuppressWarnings("unused")
     @CalledByNative
     private void onUpdateUrl(String url) {
         mUrlTextView.setText(url);
     }
 
     @SuppressWarnings("unused")
     @CalledByNative
@@ skipping 44 matching lines @@
     /**
      * Enable/Disable navigation(Prev/Next) button if navigation is allowed/disallowed
      * in respective direction.
      * @param controlId Id of button to update
      * @param enabled enable/disable value
      */
     @CalledByNative
     private void enableUiControl(int controlId, boolean enabled) {
         if (controlId == 0) mPrevButton.setEnabled(enabled);
         else if (controlId == 1) mNextButton.setEnabled(enabled);
+        else if (controlId == 2) {
+            mStopButton.setVisibility(enabled ? VISIBLE : GONE);
+            mReloadButton.setVisibility(enabled ? GONE : VISIBLE);
+        }
     }
 
     /**
      * @return The {@link ViewGroup} currently shown by this Shell.
      */
     public ViewGroup getContentView() {
         return mContentViewCore.getContainerView();
     }
 
     /**
      * @return The {@link ContentViewCore} currently managing the view shown by this Shell.
      */
     public ContentViewCore getContentViewCore() {
         return mContentViewCore;
     }
 
     private void setKeyboardVisibilityForUrl(boolean visible) {
         InputMethodManager imm = (InputMethodManager) getContext().getSystemService(
                 Context.INPUT_METHOD_SERVICE);
         if (visible) {
             imm.showSoftInput(mUrlTextView, InputMethodManager.SHOW_IMPLICIT);
         } else {
             imm.hideSoftInputFromWindow(mUrlTextView.getWindowToken(), 0);
         }
     }
 
     private static native void nativeCloseShell(long shellPtr);
 }