commit 77dd565df1238fc5528d0b18f8558d487824c16d
author Shawn O. Pearce <sop@google.com> Sat May 16 09:22:31 2009 -0700
committer Shawn O. Pearce <sop@google.com> Sat May 16 10:51:57 2009 -0700
tree 94ed050f0425595e6905522f28e3d0505e0d7fa0
parent 3511f86ae418729b0c3800a38d251bc0ee82db6a

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>

src/main/java/com/google/gerrit/client/Gerrit.java

diff --git a/src/main/java/com/google/gerrit/client/Gerrit.java b/src/main/java/com/google/gerrit/client/Gerrit.java
index a03da2b..850c8b8 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 @@
           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 @@
           });
           break;
       }
-      if (GWT.isClient() && !GWT.isScript()) {
+      if (!GWT.isScript()) {
         menuRight.addItem("Become", new Command() {
           public void execute() {
             Window.Location.assign(GWT.getHostPageBaseURL() + "become");