Tab Search: Implement TabsChanged API frontend

Related CL: https://chromium-review.googlesource.com/c/chromium/src/+/2290996

Bug: 1099917
Change-Id: Ic96dd27394ab8495d00a3a723c19e80239c99404
Reviewed-on: https://chrome-internal-review.googlesource.com/c/chrome/browser/resources/tab_search/+/3158203Reviewed-by: John Lee <johntlee@chromium.org>
Reviewed-by: Roman Arora <romanarora@chromium.org>
Reviewed-by: Tom Lukaszewicz <tluk@chromium.org>
Cr-Commit-Position: refs/heads/master@{#819572}

chrome/browser/resources/tab_search_merge/app.js

@@ -62,12 +62,29 @@ export class TabSearchAppElement extends PolymerElement {
 
     /** @private {!TabSearchApiProxy} */
     this.apiProxy_ = TabSearchApiProxyImpl.getInstance();
+
+    /** @private {!Array<number>} */
+    this.listenerIds_ = [];
   }
 
   /** @override */
   ready() {
     super.ready();
 
+    this.listenerIds_.push(
+        this.apiProxy_.getCallbackRouter().tabsChanged.addListener(
+            this.getTabs.bind(this)));
+    this.getTabs();
+  }
+
+  /** @override */
+  disconnectedCallback() {
+    this.listenerIds_.forEach(
+        id => this.apiProxy_.getCallbackRouter().removeListener(id));
+  }
+
+  /** @private */
+  getTabs() {
     this.apiProxy_.getProfileTabs().then(({profileTabs}) => {
       if (profileTabs) {
         this.openTabs_ = profileTabs.windows;

chrome/browser/resources/tab_search_merge/tab_search_api_proxy.js

@@ -14,6 +14,9 @@ export class TabSearchApiProxy {
 
   /** @param {!tabSearch.mojom.SwitchToTabInfo} info */
   switchToTab(info) {}
+
+  /** @return {!tabSearch.mojom.PageCallbackRouter} */
+  getCallbackRouter() {}
 }
 
 /** @implements {TabSearchApiProxy} */
@@ -40,6 +43,11 @@ export class TabSearchApiProxyImpl {
   switchToTab(info) {
     this.handler.switchToTab(info);
   }
+
+  /** override */
+  getCallbackRouter() {
+    return this.callbackRouter;
+  }
 }
 
 addSingletonGetter(TabSearchApiProxyImpl);

chrome/browser/resources/tab_search_merge/test/tab_search_app_test.js

@@ -173,4 +173,13 @@ suite('TabSearchAppTest', () => {
     keyDownOn(searchInput, 0, [], 'Home');
     assertEquals(0, tabSearchApp.getSelectedIndex());
   });
+
+  test('refresh on tabs changed', async () => {
+    await setupTest(sampleData());
+    verifyTabIds(queryRows(), [1, 5, 6, 2, 3, 4]);
+    testProxy.setProfileTabs({windows: []});
+    testProxy.getCallbackRouterRemote().tabsChanged();
+    await flushTasks();
+    verifyTabIds(queryRows(), []);
+  });
 });

chrome/browser/resources/tab_search_merge/test/test_tab_search_api_proxy.js

@@ -13,6 +13,13 @@ export class TestTabSearchApiProxy extends TestBrowserProxy {
       'switchToTab',
     ]);
 
+    /** @type {!tabSearch.mojom.PageCallbackRouter} */
+    this.callbackRouter = new tabSearch.mojom.PageCallbackRouter();
+
+    /** @type {!tabSearch.mojom.PageRemote} */
+    this.callbackRouterRemote =
+        this.callbackRouter.$.bindNewPipeAndPassRemote();
+
     /** @private {tabSearch.mojom.ProfileTabs} */
     this.profileTabs_;
   }
@@ -28,6 +35,16 @@ export class TestTabSearchApiProxy extends TestBrowserProxy {
     this.methodCalled('switchToTab');
   }
 
+  /** override */
+  getCallbackRouter() {
+    return this.callbackRouter;
+  }
+
+  /** return {!tabSearch.mojom.PageRemote} */
+  getCallbackRouterRemote() {
+    return this.callbackRouterRemote;
+  }
+
   /** @param {tabSearch.mojom.ProfileTabs} profileTabs */
   setProfileTabs(profileTabs) {
     this.profileTabs_ = profileTabs;