When switching the visible uber frame, also switch focus.

chrome/browser/resources/uber/uber.js

Index: chrome/browser/resources/uber/uber.js
diff --git a/chrome/browser/resources/uber/uber.js b/chrome/browser/resources/uber/uber.js
index fde61ed74b77abf291558901bfc05ca4eb4d5b30..23417eaf749730ef678913687bc3b3310795a586 100644
--- a/chrome/browser/resources/uber/uber.js
+++ b/chrome/browser/resources/uber/uber.js
@@ -368,6 +368,7 @@ cr.define('uber', function() {
 
     var selectedFrame = getSelectedIframe().querySelector('iframe');
     uber.invokeMethodOnWindow(selectedFrame.contentWindow, 'frameSelected');

[Dan Beam, 2014/09/27 00:36:17]

if (!selectedFrame.contentDocument.hasFocus())

[Evan Stade, 2014/09/29 15:24:24]

what is the difference between having this check and not having this check?

[Dan Beam, 2014/09/29 19:15:47]

i believed this might break auto-focused fields (e.g. search in settings) and/or
double-focus a frame, but in trying it myself:
a) you can't access the contentDocument easily anyway (it seems)
b) re-focusing a window with focus doesn't dispatch another event

+    selectedFrame.contentWindow.focus();
 
     if (historyOption != HISTORY_STATE_OPTION.NONE)
       changePathTo({}, path, historyOption);