Remove pointless GWT.isClient calls in Gerrit module

The module entry point only runs on the client, and this code is
about building the top menu bar for the client UI.  The call is
always true, and is just confusing to read in context.

Signed-off-by: Shawn O. Pearce <sop@google.com>

1 files changed, 2 insertions, 2 deletions

diff --git a/src/main/java/com/google/gerrit/client/Gerrit.java b/src/main/java/com/google/gerrit/client/Gerrit.java
index a03da2bff0..850c8b846e 100644
--- a/src/main/java/com/google/gerrit/client/Gerrit.java
+++ b/src/main/java/com/google/gerrit/client/Gerrit.java
@@ -372,7 +372,7 @@ public class Gerrit implements EntryPoint {
           signout = true;
           break;
       }
-      if (signout || (GWT.isClient() && !GWT.isScript())) {
+      if (signout || !GWT.isScript()) {
         menuRight.addItem(C.menuSignOut(), new Command() {
           public void execute() {
             doSignOut();
@@ -393,7 +393,7 @@ public class Gerrit implements EntryPoint {
           });
           break;
       }
-      if (GWT.isClient() && !GWT.isScript()) {
+      if (!GWT.isScript()) {
         menuRight.addItem("Become", new Command() {
           public void execute() {
             Window.Location.assign(GWT.getHostPageBaseURL() + "become");