Hotword Audio Verification App: Fix window handling.

chrome/browser/resources/hotword_audio_verification/event_page.js

Index: chrome/browser/resources/hotword_audio_verification/event_page.js
diff --git a/chrome/browser/resources/hotword_audio_verification/event_page.js b/chrome/browser/resources/hotword_audio_verification/event_page.js
index ad421b3430bc5d725b5212775a80c776ea145844..b98ff086ba7811586c146974715bcc171df9f72c 100644
--- a/chrome/browser/resources/hotword_audio_verification/event_page.js
+++ b/chrome/browser/resources/hotword_audio_verification/event_page.js
@@ -2,15 +2,20 @@
 // Use of this source code is governed by a BSD-style license that can be
 // found in the LICENSE file.
 
+var appId = 'hotword_audio_verification_app';
+
 chrome.app.runtime.onLaunched.addListener(function() {
-  // TODO(kcarattini): Check if the app is already running. If so, bring it
-  // to focus rather than creating a new window.
+  var appWindow = chrome.app.window.get(appId);
+  if (appWindow) {
+    appWindow.focus();
+    return;
+  }

[Dan Beam, 2014/10/24 00:31:48]

seems like this should already happen[1]:

"""
If a window with a given id is created while another window with the same id
already exists, the currently opened window will be focused instead of creating
a new window.
"""

[1] https://developer.chrome.com/apps/app_window#type-CreateWindowOptions

[kcarattini, 2014/10/24 00:35:27]

When I tested, the window did not focus, that's why I added the code. Not sure
why it didn't happen ...

[kcarattini, 2014/10/24 02:28:35]

Issue https://code.google.com/p/chromium/issues/detail?id=426725 filed and added
a TODO to remove this once it's fixed.

 
-  // TODO(kcarattini): Don't show the window until the launch mode has been
-  // established.
   chrome.app.window.create('main.html', {
     'frame': 'none',
     'resizable': false,
+    'hidden': true,
+    'id': appId,
     'bounds': {
       'width': 800,
       'height': 600