Clicking a non-focusable element in an inactive frame should clear focus in the frame.

Clicking a non-focusable element in an inactive frame should clear focus in the frame.

Clicking a non-focusable element in an inactive frame did not remove focus
from the focused element in the frame because
 - FocusController::setFocusedElement incorrectly returned earlier if there
    was no focused element in the currently focused frame, and
 - FocusController::setFocusedElement cleared focus in the currently
    focused frame if new focused element is 0, but did not clear the focus in
    the new focused frame.

We shouldn't do early-return for a element==0 case. Also, we need to to
call clearSelectionIfNeeded() later than same-element check to avoid test
breakage.


BUG=304145
TEST=automated.

Committed: https://src.chromium.org/viewvc/blink?view=rev&revision=161312

[tkent, 2013-11-05 03:48:15]

Hayato-san, would you review this please?

[hayato, 2013-11-05 06:50:44]

lgtm

[commit-bot: I haz the power, 2013-11-05 06:51:32]

CQ is trying da patch. Follow status at
https://chromium-status.appspot.com/cq/tkent@chromium.org/57813008/60001

[commit-bot: I haz the power, 2013-11-05 07:00:34]

Change committed as 161312