From ae42464225be2dd3919b641b417da1b195932a0a Mon Sep 17 00:00:00 2001
From: Edwin Kempin <edwin.kempin@sap.com>
Date: Tue, 21 Jul 2015 16:02:44 +0200
Subject: [PATCH] Remove deprecated gerrit_addHistoryHook(f) JavaScript
 function

Instead of this function Gerrit.on('history', f) should be used.

gerrit_addHistoryHook(f) is deprecated since Gerrit 2.9. It's time to
remove it.

Change-Id: I87b23b2d7e404c4a43974d9fff202d11965aad6c
Signed-off-by: Edwin Kempin <edwin.kempin@sap.com>
---
 Documentation/config-themes.txt                          | 3 ---
 .../main/java/com/google/gerrit/client/api/ApiGlue.java  | 9 ---------
 2 files changed, 12 deletions(-)

diff --git a/Documentation/config-themes.txt b/Documentation/config-themes.txt
index 5c3a448ef5..b165e37d38 100644
--- a/Documentation/config-themes.txt
+++ b/Documentation/config-themes.txt
@@ -133,9 +133,6 @@ page.  Because GWT loads the module asynchronously any `<script>`
 block in the header or footer will execute before Gerrit has defined
 the function and is ready to register the hook callback.
 
-The function `gerrit_addHistoryHook` is deprecated and may be
-removed in a future release.
-
 GERRIT
 ------
 Part of link:index.html[Gerrit Code Review]
diff --git a/gerrit-gwtui/src/main/java/com/google/gerrit/client/api/ApiGlue.java b/gerrit-gwtui/src/main/java/com/google/gerrit/client/api/ApiGlue.java
index e4a5446f74..05cc17369e 100644
--- a/gerrit-gwtui/src/main/java/com/google/gerrit/client/api/ApiGlue.java
+++ b/gerrit-gwtui/src/main/java/com/google/gerrit/client/api/ApiGlue.java
@@ -30,7 +30,6 @@ public class ApiGlue {
     ActionContext.init();
     HtmlTemplate.init();
     Plugin.init();
-    addHistoryHook();
   }
 
   private static native void init0() /*-{
@@ -203,14 +202,6 @@ public class ApiGlue {
     };
   }-*/;
 
-  /** Install deprecated {@code gerrit_addHistoryHook()} function. */
-  private static native void addHistoryHook() /*-{
-    $wnd.gerrit_addHistoryHook = function(h) {
-      var p = @com.google.gwt.user.client.Window.Location::getPath()();
-      $wnd.Gerrit.on('history', function(t) { h(p + "#" + t) })
-     };
-  }-*/;
-
   private static void install(JavaScriptObject cb, Plugin p) throws Exception {
     try {
       pluginName = p.name();