fix(gax): unconditionally wrap logging tracer

Author: westarle

gax-java/gax/src/main/java/com/google/api/gax/rpc/ClientContext.java

@@ -281,13 +281,11 @@ public static ClientContext create(StubSettings settings) throws IOException {
     if (apiTracerFactory instanceof SpanTracerFactory) {
       apiTracerFactory = apiTracerFactory.withContext(apiTracerContext);
     }
-    if (com.google.api.gax.logging.LoggingUtils.isLoggingEnabled()) {
-      com.google.api.gax.tracing.ApiTracerFactory loggingTracerFactory =
-          new com.google.api.gax.tracing.LoggingTracerFactory().withContext(apiTracerContext);
-      apiTracerFactory =
-          new com.google.api.gax.tracing.CompositeTracerFactory(
-              apiTracerFactory, loggingTracerFactory);
-    }
+    com.google.api.gax.tracing.ApiTracerFactory loggingTracerFactory =
+        new com.google.api.gax.tracing.LoggingTracerFactory().withContext(apiTracerContext);
+    apiTracerFactory =
+        new com.google.api.gax.tracing.CompositeTracerFactory(
+            apiTracerFactory, loggingTracerFactory);
 
     return newBuilder()
         .setBackgroundResources(backgroundResources.build())

gax-java/gax/src/main/java/com/google/api/gax/tracing/CompositeTracer.java

@@ -52,7 +52,8 @@ public Scope inScope() {
       scopes[i] = tracers.get(i).inScope();
     }
     return () -> {
-      for (Scope scope : scopes) {
+      for (int i = scopes.length - 1; i >= 0; i--) {
+        Scope scope = scopes[i];
         if (scope != null) {
           scope.close();
         }