Eliminate the OmniboxEditModel::GetViewText() method.

Eliminate the OmniboxEditModel::GetViewText() method.

The method is only used one place, which is a somewhat unnecessary call in the same class.

BUG=

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

[Justin Donnelly, 2013-12-02 21:23:56]

Hi Peter - the goal in this change is consistency.  I have no opinion about
whether the use of the method or direct access of the view_ variable is better. 
Alternatively, I could just get rid of the one existing call to GetViewText,
which is not used anywhere outside OmniboxEditModel.

[Peter Kasting, 2013-12-02 21:41:48]

Since GetViewText() is only used internally and only used once, let's just nuke
it and use view_->GetText() everywhere.

[Justin Donnelly, 2013-12-03 01:52:13]

On 2013/12/02 21:41:48, Peter Kasting wrote:
> Since GetViewText() is only used internally and only used once, let's just
nuke
> it and use view_->GetText() everywhere.

Done

[Peter Kasting, 2013-12-03 01:54:30]

LGTM

[commit-bot: I haz the power, 2013-12-03 17:44:17]

CQ is trying da patch. Follow status at
https://chromium-status.appspot.com/cq/jdonnelly@chromium.org/89273002/20001

[commit-bot: I haz the power, 2013-12-03 19:13:29]

CQ is trying da patch. Follow status at
https://chromium-status.appspot.com/cq/jdonnelly@chromium.org/89273002/20001

[commit-bot: I haz the power, 2013-12-03 23:41:49]

CQ is trying da patch. Follow status at
https://chromium-status.appspot.com/cq/jdonnelly@chromium.org/89273002/20001

[commit-bot: I haz the power, 2013-12-04 04:24:05]

Change committed as 238563