commit 07b72b3a4c872d11c6945112c37d57ac71d8bc48
author Michael Zhou <zhoumotongxue008@gmail.com> Mon Apr 25 03:33:37 2016 -0400
committer Michael Zhou <zhoumotongxue008@gmail.com> Mon Apr 25 07:16:25 2016 -0400
tree 806f31e69c8cbdb9524a63e647baebeed51f28b6
parent 825ba7e64554c86613f371bbad9ff7d7efe2c2f6

Unified: Remove unnecessary onGutterClick()

This is already handled in the DiffScreen superclass.

Change-Id: Icd564fd633174061a54815f128697b85c769a604

gerrit-gwtui/src/main/java/com/google/gerrit/client/diff/Unified.java

diff --git a/gerrit-gwtui/src/main/java/com/google/gerrit/client/diff/Unified.java b/gerrit-gwtui/src/main/java/com/google/gerrit/client/diff/Unified.java
index 9b04d6f..5e26663 100644
--- a/gerrit-gwtui/src/main/java/com/google/gerrit/client/diff/Unified.java
+++ b/gerrit-gwtui/src/main/java/com/google/gerrit/client/diff/Unified.java
@@ -32,9 +32,6 @@
 import com.google.gwt.core.client.Scheduler;
 import com.google.gwt.core.client.Scheduler.ScheduledCommand;
 import com.google.gwt.dom.client.Element;
-import com.google.gwt.dom.client.NativeEvent;
-import com.google.gwt.event.dom.client.ClickEvent;
-import com.google.gwt.event.dom.client.ClickHandler;
 import com.google.gwt.event.dom.client.FocusEvent;
 import com.google.gwt.event.dom.client.FocusHandler;
 import com.google.gwt.event.dom.client.KeyPressEvent;
@@ -48,7 +45,6 @@
 import com.google.gwtexpui.globalkey.client.GlobalKey;
 
 import net.codemirror.lib.CodeMirror;
-import net.codemirror.lib.CodeMirror.GutterClickHandler;
 import net.codemirror.lib.CodeMirror.LineHandle;
 import net.codemirror.lib.Configuration;
 import net.codemirror.lib.Pos;
@@ -255,36 +251,8 @@
     // TODO: Implement this
   }
 
-  private GutterClickHandler onGutterClick(final int cmLine) {
-    return new GutterClickHandler() {
-      @Override
-      public void handle(CodeMirror instance, int line, String gutter,
-          NativeEvent clickEvent) {
-        if (clickEvent.getButton() == NativeEvent.BUTTON_LEFT
-            && !clickEvent.getMetaKey()
-            && !clickEvent.getAltKey()
-            && !clickEvent.getCtrlKey()
-            && !clickEvent.getShiftKey()) {
-          cm.setCursor(Pos.create(cmLine));
-          Scheduler.get().scheduleDeferred(new ScheduledCommand() {
-            @Override
-            public void execute() {
-              commentManager.newDraftCallback(cm).run();
-            }
-          });
-        }
-      }
-    };
-  }
-
   LineHandle setLineNumber(DisplaySide side, final int cmLine, int line) {
     Label gutter = new Label(String.valueOf(line));
-    gutter.addClickHandler(new ClickHandler() {
-      @Override
-      public void onClick(ClickEvent event) {
-        onGutterClick(cmLine);
-      }
-    });
     diffTable.add(gutter);
     gutter.setStyleName(UnifiedTable.style.unifiedLineNumber());
     return cm.setGutterMarker(cmLine,