InstantExtended: add back focus() API call.

InstantExtended: add back focus() API call.

BUG=none
TESTED=Open NTP, run chrome.embeddedSearch.searchBox.focus() in the DOM inspector.

Committed: https://src.chromium.org/viewvc/chrome?view=rev&revision=220531

[samarth, 2013-08-29 00:21:17]

Please review.

Thanks,
Samarth

[jfweitz, 2013-08-29 00:47:52]

LGTM with one nit.

You will need a OWNERS stamp.

https://codereview.chromium.org/23462014/diff/1/chrome/renderer/resources/ext...
File chrome/renderer/resources/extensions/searchbox_api.js (right):

https://codereview.chromium.org/23462014/diff/1/chrome/renderer/resources/ext...
chrome/renderer/resources/extensions/searchbox_api.js:47: }
nit: missing ';'

[samarth, 2013-08-29 00:51:06]

Thanks!

I'm in the OWNERS file :)

https://codereview.chromium.org/23462014/diff/1/chrome/renderer/resources/ext...
File chrome/renderer/resources/extensions/searchbox_api.js (right):

https://codereview.chromium.org/23462014/diff/1/chrome/renderer/resources/ext...
chrome/renderer/resources/extensions/searchbox_api.js:47: }
On 2013/08/29 00:47:52, jfweitz wrote:
> nit: missing ';'

Done.

[commit-bot: I haz the power, 2013-08-29 00:52:22]

No LGTM from a valid reviewer yet. Only full committers are accepted.
Even if an LGTM may have been provided, it was from a non-committer or
a lowly provisional committer, _not_ a full super star committer.
See http://www.chromium.org/getting-involved/become-a-committer
Note that this has nothing to do with OWNERS files.

[samarth, 2013-08-29 00:54:36]

+kmadhusu

[jfweitz, 2013-08-29 00:58:17]

On 2013/08/29 00:51:06, samarth wrote:
> Thanks!
> 
> I'm in the OWNERS file :)
> 
>
https://codereview.chromium.org/23462014/diff/1/chrome/renderer/resources/ext...
> File chrome/renderer/resources/extensions/searchbox_api.js (right):
> 
>
https://codereview.chromium.org/23462014/diff/1/chrome/renderer/resources/ext...
> chrome/renderer/resources/extensions/searchbox_api.js:47: }
> On 2013/08/29 00:47:52, jfweitz wrote:
> > nit: missing ';'
> 
> Done.

Sorry I meant a committer stamp :(

[kmadhusu, 2013-08-29 16:56:51]

lgtm

[commit-bot: I haz the power, 2013-08-29 17:04:18]

CQ is trying da patch. Follow status at
https://chromium-status.appspot.com/cq/samarth@chromium.org/23462014/9001

[commit-bot: I haz the power, 2013-08-29 21:09:01]

Retried try job too often on win_rel for step(s) browser_tests,
chrome_frame_net_tests, chrome_frame_tests, chrome_frame_unittests,
content_browsertests, mini_installer_test, nacl_integration,
sync_integration_tests
http://build.chromium.org/p/tryserver.chromium/buildstatus?builder=win_rel&nu...

[commit-bot: I haz the power, 2013-08-29 21:13:09]

CQ is trying da patch. Follow status at
https://chromium-status.appspot.com/cq/samarth@chromium.org/23462014/9001

[commit-bot: I haz the power, 2013-08-30 07:24:53]

Change committed as 220531