From d07c46dc6b99490cc7444b3871c96159d842bfdc Mon Sep 17 00:00:00 2001
From: Pablo Gamito <pablogamito@google.com>
Date: Mon, 1 Jun 2020 19:02:38 +0100
Subject: [PATCH] Implement type and id filters for transactions log

Test: N/A
Change-Id: Ie1a5f69dacbfafd6524cac6e16bf1491cfad48e4
---
 tools/winscope/src/App.vue              |  2 +-
 tools/winscope/src/TransactionsView.vue | 80 ++++++++++++++++++++++---
 2 files changed, 74 insertions(+), 8 deletions(-)

diff --git a/tools/winscope/src/App.vue b/tools/winscope/src/App.vue
index 0f6a2ac9b..3006507bd 100644
--- a/tools/winscope/src/App.vue
+++ b/tools/winscope/src/App.vue
@@ -57,7 +57,7 @@ import LocalStore from './localstore.js'
 import DataAdb from './DataAdb.vue'
 import FileType from './mixins/FileType.js'
 
-const APP_NAME = "Winscope"
+const APP_NAME = "Winscope";
 
 export default {
   name: 'app',
diff --git a/tools/winscope/src/TransactionsView.vue b/tools/winscope/src/TransactionsView.vue
index 6856da67b..24f40281d 100644
--- a/tools/winscope/src/TransactionsView.vue
+++ b/tools/winscope/src/TransactionsView.vue
@@ -14,9 +14,24 @@
 -->
 <template>
   <md-card-content class="container">
-    <md-table class="transaction-table">
+    <md-table class="transaction-table" md-card>
+      <md-table-toolbar>
+        <md-field>
+          <label>Transaction Type</label>
+          <md-select v-model="selectedTransactionTypes" multiple>
+            <md-option v-for="type in transactionTypes" :value="type">{{ type }}</md-option>
+          </md-select>
+        </md-field>
+
+        <md-field md-clearable>
+          <md-input v-model="searchInput" placeholder="Search by id or name..." />
+          <span class="md-helper-text">Comma seperated</span>
+        </md-field>
+
+      </md-table-toolbar>
+
       <div class="scrollBody" ref="tableBody">
-        <md-table-row v-for="transaction in data" :key="transaction.timestamp">
+        <md-table-row v-for="transaction in filteredData" :key="transaction.timestamp">
           <md-table-cell>{{transaction.time}}</md-table-cell>
 
           <div v-if="transaction.type == 'transaction'">
@@ -25,7 +40,10 @@
             </md-table-cell>
           </div>
           <div v-else>
-            <md-table-cell>{{transaction.type}}</md-table-cell>
+            <md-table-cell>
+              {{transaction.type}}
+              <span v-if="transaction.obj.id !== undefined">: {{transaction.obj.id}}</span>
+            </md-table-cell>
           </div>
         </md-table-row>
       </div>
@@ -38,12 +56,60 @@ export default {
   name: 'transactionsview',
   props: ['data'],
   data() {
-    return {};
+    const transactionTypes = new Set();
+    for (const entry of this.data) {
+      if (entry.type == "transaction") {
+        for (const transaction of entry.transactions) {
+          transactionTypes.add(transaction.type);
+        }
+      } else {
+        transactionTypes.add(entry.type);
+      }
+    }
+
+    return {
+      transactionTypes: Array.from(transactionTypes),
+      selectedTransactionTypes: [],
+      searchInput: "",
+    };
   },
   computed: {
-    
+    filteredData() {
+      let filteredData = this.data;
+
+      if (this.selectedTransactionTypes.length > 0) {
+        filteredData = filteredData.filter(
+          this.filterTransactions(transaction =>
+            this.selectedTransactionTypes.includes(transaction.type)));
+      }
+
+      if (this.searchInput) {
+        const terms = this.searchInput.split(/,\s*/);
+        filteredData = filteredData.filter(
+          this.filterTransactions(transaction =>
+            terms.includes("" + transaction.obj.id)));
+      }
+
+      return filteredData;
+    },
+
   },
   methods: {
+    filterTransactions(condition) {
+      return (entry) => {
+        if (entry.type == "transaction") {
+            for (const transaction of entry.transactions) {
+              if (condition(transaction)) {
+                return true;
+              }
+            }
+
+            return false;
+          } else {
+            return condition(entry);
+          }
+      };
+    },
     summarizeTranscations(transactions) {
       const surfaceChanges = {};
       const displayChanges = {};
@@ -53,12 +119,12 @@ export default {
 
         switch (transaction.type) {
           case "surfaceChange":
-            surfaceChanges[obj.id] = 
+            surfaceChanges[obj.id] =
               Object.assign(surfaceChanges[obj.id] ?? {}, obj);
             break;
 
           case "displayChange":
-            displayChanges[obj.id] = 
+            displayChanges[obj.id] =
               Object.assign(displayChanges[obj.id] ?? {}, obj);
             break;