Visually disable VR Shell back/forward buttons when no history is available

chrome/android/java/src/org/chromium/chrome/browser/vr_shell/VrShellImpl.java

Index: chrome/android/java/src/org/chromium/chrome/browser/vr_shell/VrShellImpl.java
diff --git a/chrome/android/java/src/org/chromium/chrome/browser/vr_shell/VrShellImpl.java b/chrome/android/java/src/org/chromium/chrome/browser/vr_shell/VrShellImpl.java
index 0a9b713d14b3e3fd4047691638b5360a4a1906d2..c74fa935163805dc8974247270e3ef2b7fc9a6a9 100644
--- a/chrome/android/java/src/org/chromium/chrome/browser/vr_shell/VrShellImpl.java
+++ b/chrome/android/java/src/org/chromium/chrome/browser/vr_shell/VrShellImpl.java
@@ -204,6 +204,7 @@ public class VrShellImpl
                 } else {
                     nativeSwapContents(mNativeVrShell, null, mMotionEventSynthesizer);
                 }
+                updateHistoryButtonsVisibility();
             }
 
             @Override
@@ -558,11 +559,19 @@ public class VrShellImpl
     @CalledByNative
     public void navigateForward() {
         mActivity.getToolbarManager().forward();
+        updateHistoryButtonsVisibility();
     }
 
     @CalledByNative
     public void navigateBack() {
         mActivity.getToolbarManager().back();
+        updateHistoryButtonsVisibility();
+    }
+
+    private void updateHistoryButtonsVisibility() {
+        boolean canGoBack = mTab != null && mTab.canGoBack(),
+                canGoForward = mTab != null && mTab.canGoForward();

[mthiesse, 2017/03/08 16:15:17]

Chrome style prefers 'boolean canGoForward = mTab != null &&
mTab.canGoForward();'
We don't usually create multiple variables on one line.

[acondor_, 2017/03/08 16:42:26]

Done.

+        nativeSetHistoryButtonsEnabled(mNativeVrShell, canGoBack, canGoForward);
     }
 
     @CalledByNative
@@ -619,4 +628,6 @@ public class VrShellImpl
     private native void nativeOnTabRemoved(long nativeVrShell, boolean incognito, int id);
     private native Surface nativeTakeContentSurface(long nativeVrShell);
     private native void nativeRestoreContentSurface(long nativeVrShell);
+    private native void nativeSetHistoryButtonsEnabled(
+            long nativeVrShell, boolean canGoBack, boolean canGoForward);
 }