Allow Perfetto traces without specifying an application.

gapic/src/main/com/google/gapid/views/TracerDialog.java

@@ -582,9 +582,10 @@ protected String formatTraceName(String name) {
       }
 
       public boolean isReady() {
-        return getSelectedDevice() != null && getSelectedApi() != null &&
-            !traceTarget.getText().isEmpty() && !directory.getText().isEmpty() &&
-            !file.getText().isEmpty();
+        TraceTypeCapabilities config = getSelectedApi();
+        return getSelectedDevice() != null && config != null &&
+            (!config.getRequiresApplication() || !traceTarget.getText().isEmpty()) &&
+            !directory.getText().isEmpty() && !file.getText().isEmpty();
       }
 
       public void addModifyListener(Listener listener) {

gapis/service/service.proto

@@ -1235,6 +1235,8 @@ message TraceTypeCapabilities {
   FeatureStatus mid_execution_capture_support = 3;
   // Whether unsupported extensions can be enabled.
   bool can_enable_unsupported_extensions = 4;
+  // Does this trace require starting an application.
+  bool requires_application = 6;
 }
 
 // GetTimestampsRequest is the request send to server to get the timestamps for

gapis/trace/android/trace.go

@@ -449,26 +449,28 @@ func (t *androidTracer) SetupTrace(ctx context.Context, o *service.TraceOptions)
 				match[1], match[3],
 				strings.Join(lines, "\n  "))
 		}
-	} else {
+	} else if o.Type != service.TraceType_Perfetto || len(o.GetUri()) != 0 {
 		return ret, nil, fmt.Errorf("Could not find package matching %s", o.GetUri())
 	}
 
-	if !pkg.Debuggable {
-		err = t.b.Root(ctx)
-		switch err {
-		case nil:
-		case adb.ErrDeviceNotRooted:
-			return ret, cleanup.Invoke(ctx), err
-		default:
-			return ret, cleanup.Invoke(ctx), fmt.Errorf("Failed to restart ADB as root: %v", err)
+	if pkg != nil {
+		if !pkg.Debuggable {
+			err = t.b.Root(ctx)
+			switch err {
+			case nil:
+			case adb.ErrDeviceNotRooted:
+				return ret, cleanup.Invoke(ctx), err
+			default:
+				return ret, cleanup.Invoke(ctx), fmt.Errorf("Failed to restart ADB as root: %v", err)
+			}
+			log.I(ctx, "Device is rooted")
 		}
-		log.I(ctx, "Device is rooted")
-	}
 
-	if o.ClearCache {
-		log.I(ctx, "Clearing package cache")
-		if err := pkg.ClearCache(ctx); err != nil {
-			return ret, nil, err
+		if o.ClearCache {
+			log.I(ctx, "Clearing package cache")
+			if err := pkg.ClearCache(ctx); err != nil {
+				return ret, nil, err
+			}
 		}
 	}
 

gapis/trace/tracer/default_api_trace_options.go

@@ -26,6 +26,7 @@ func VulkanTraceOptions() *service.TraceTypeCapabilities {
 		CanDisablePcs:                  false,
 		MidExecutionCaptureSupport:     service.FeatureStatus_Supported,
 		CanEnableUnsupportedExtensions: true,
+		RequiresApplication:            true,
 	}
 }
 
@@ -37,6 +38,7 @@ func GLESTraceOptions() *service.TraceTypeCapabilities {
 		CanDisablePcs:                  true,
 		MidExecutionCaptureSupport:     service.FeatureStatus_Experimental,
 		CanEnableUnsupportedExtensions: false,
+		RequiresApplication:            true,
 	}
 }
 
@@ -47,5 +49,6 @@ func PerfettoTraceOptions() *service.TraceTypeCapabilities {
 		CanDisablePcs:                  false,
 		MidExecutionCaptureSupport:     service.FeatureStatus_Supported,
 		CanEnableUnsupportedExtensions: false,
+		RequiresApplication:            false,
 	}
 }