Bring windows with new tabs to foreground

chrome/browser/resources/google_now/background.js

 // Copyright (c) 2013 The Chromium Authors. All rights reserved.
 // Use of this source code is governed by a BSD-style license that can be
 // found in the LICENSE file.
 
 'use strict';
 
 /**
  * @fileoverview The event page for Google Now for Chrome implementation.
  * The Google Now event page gets Google Now cards from the server and shows
  * them as Chrome notifications.
@@ skipping 573 matching lines @@
     if (typeof actionUrls != 'object') {
       return;
     }
 
     var url = selector(actionUrls);
 
     if (typeof url != 'string')
       return;
 
     instrumented.tabs.create({url: url}, function(tab) {
-      if (!tab)
+      if (tab)
+        chrome.windows.update(tab.windowId, {focused: true});
+      else
         chrome.windows.create({url: url});

[rgustafson, 2013/08/14 01:02:03]

My concern is whether the same will happen with this guy as with the tabs. Set
focused to true here also for completeness? (don't need the extra update call,
just do it in create)

[vadimt, 2013/08/14 01:35:03]

Done.

     });
   });
 }
 
 /**
  * Responds to a click of one of the buttons on the welcome toast.
  * @param {number} buttonIndex The index of the button which was clicked.
  */
 function onToastNotificationClicked(buttonIndex) {
   chrome.storage.local.set({userRespondedToToast: true});
@@ skipping 319 matching lines @@
 
 instrumented.location.onLocationUpdate.addListener(function(position) {
   recordEvent(GoogleNowEvent.LOCATION_UPDATE);
   updateNotificationsCards(position);
 });
 
 instrumented.omnibox.onInputEntered.addListener(function(text) {
   localStorage['server_url'] = NOTIFICATION_CARDS_URL = text;
   initialize();
 });