Split off logging/ package from giant server package

We want to keep the dependencies of the logging/ package as small as
possible. To avoid a dependency on
"//java/com/google/gerrit/reviewdb:server" remove the
RequestId.forChange(Change) and RequestId.forProject(Project.NameKey)
methods which each only had a single caller in production code. Instead
let these 2 callers call the RequestId constructor directly.

Change-Id: Ied6989eadabc1884212a0c0c8aafc7532e215324
Signed-off-by: Edwin Kempin <ekempin@google.com>
diff --git a/plugins/BUILD b/plugins/BUILD
index 8b9f4de..a7622b2 100644
--- a/plugins/BUILD
+++ b/plugins/BUILD
@@ -38,6 +38,7 @@
     "//java/com/google/gerrit/metrics/dropwizard",
     "//java/com/google/gerrit/reviewdb:server",
     "//java/com/google/gerrit/server/audit",
+    "//java/com/google/gerrit/server/logging",
     "//java/com/google/gerrit/server/schema",
     "//java/com/google/gerrit/util/http",
     "//lib/commons:compress",