Minor refactoring to avoid redundant calls to VisibleSelection::visibleStart()

Minor refactoring to avoid redundant calls to VisibleSelection::visibleStart()

We can avoid redundant calls to visibleStart() which indirectly calls
several functions like canonicalPosition, isCandidate, upstream,
downstream, etc. by using a local variable.

Current code already makes use of a local variable to store the VisiblePosition
returned by visibleStart(). Just preponing the creation of this
local variable would avoid few more redundant calls.

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

[vanihegde, 2014-01-22 09:52:16]

Minor change. Please take a look. Thanks!

[leviw_travelin_and_unemployed, 2014-01-22 21:28:06]

lgtm

Okay.

[commit-bot: I haz the power, 2014-01-22 21:29:21]

CQ is trying da patch. Follow status at
https://chromium-status.appspot.com/cq/vani.hegde@samsung.com/132903007/1

[commit-bot: I haz the power, 2014-01-22 22:52:11]

Change committed as 165561