Add dropped code review changes from google#2696

- drop the magic window size constant and simply use Perfetto's default.
- Remove a TODO comment

gapis/perfetto/cc/processor.cpp

@@ -28,16 +28,13 @@ namespace pp = perfetto::protos;
 // Callback back into go.
 extern "C" void on_query_complete(int id, void* data, long unsigned int size);
 
-processor new_processor(uint64_t window_size_ns) {
+processor new_processor() {
   ptp::Config config;
-  config.window_size_ns = window_size_ns;
-
   return ptp::TraceProcessor::CreateInstance(config).release();
 }
 
 bool parse_data(processor processor, const void* data, size_t size) {
   ptp::TraceProcessor* p = static_cast<ptp::TraceProcessor*>(processor);
-  // TODO: do we need this copy?
   std::unique_ptr<uint8_t[]> buf(new uint8_t[size]);
   memcpy(buf.get(), data, size);
   if (!p->Parse(std::move(buf), size)) {

gapis/perfetto/cc/processor.h

@@ -27,7 +27,7 @@ extern "C" {
 
 typedef void* processor;
 
-processor new_processor(uint64_t window_size_ns);
+processor new_processor();
 bool parse_data(processor processor, const void* data, size_t size);
 void execute_query(processor processor, int id, const char* query);
 void delete_processor(processor processor);

gapis/perfetto/processor.go

@@ -47,7 +47,7 @@ type Result struct {
 }
 
 func NewProcessor(ctx context.Context, data []byte) (*Processor, error) {
-	p := C.new_processor(60 * 1000 * 1000 * 1000)
+	p := C.new_processor()
 	log.D(ctx, "[perfetto] Parsing %d bytes", len(data))
 	if !C.parse_data(p, unsafe.Pointer(&data[0]), C.size_t(len(data))) {
 		log.W(ctx, "[perfetto] Parsing failed")