Makes SetFocus() notify callback correctly on failure

Makes SetFocus() notify callback correctly on failure

SetFocus() may fail in the FocusController and this wasn't being
propagated correctly to the client. This meant the client might think
a window had focus when it didn't.

BUG=600869
TEST=covered by tests
R=ben@chromium.org

Committed: https://crrev.com/f9b88e631421343f0fd3d695b9cee0b090eb6ac1
Cr-Commit-Position: refs/heads/master@{#385866}

[Ben Goodger (Google), 2016-04-07 20:19:27]

lgtm

[sky, 2016-04-07 20:20:14]

The CQ bit was checked by sky@chromium.org

[commit-bot: I haz the power, 2016-04-07 20:20:55]

CQ is trying da patch. Follow status at
 https://chromium-cq-status.appspot.com/patch-status/1868783003/1
View timeline at
 https://chromium-cq-status.appspot.com/patch-timeline/1868783003/1

[commit-bot: I haz the power, 2016-04-07 21:24:37]

Committed patchset #1 (id:1)

[commit-bot: I haz the power, 2016-04-07 21:30:00]

Description was changed from

==========
Makes SetFocus() notify callback correctly on failure

SetFocus() may fail in the FocusController and this wasn't being
propagated correctly to the client. This meant the client might think
a window had focus when it didn't.

BUG=600869
TEST=covered by tests
R=ben@chromium.org
==========

to

==========
Makes SetFocus() notify callback correctly on failure

SetFocus() may fail in the FocusController and this wasn't being
propagated correctly to the client. This meant the client might think
a window had focus when it didn't.

BUG=600869
TEST=covered by tests
R=ben@chromium.org

Committed: https://crrev.com/f9b88e631421343f0fd3d695b9cee0b090eb6ac1
Cr-Commit-Position: refs/heads/master@{#385866}
==========

[commit-bot: I haz the power, 2016-04-07 21:30:01]

Patchset 1 (id:??) landed as
https://crrev.com/f9b88e631421343f0fd3d695b9cee0b090eb6ac1
Cr-Commit-Position: refs/heads/master@{#385866}