Minor cosmetic changes to OmniboxEditModel and OmniboxController.

Minor cosmetic changes to OmniboxEditModel and OmniboxController.

This follow-up on some late comments by pkasting on https://codereview.chromium.org/14698028

BUG=

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

[beaudoin, 2013-06-18 22:38:44]

Fixing the nits on the previous CL.

[Peter Kasting, 2013-06-18 23:31:19]

LGTM

https://codereview.chromium.org/17426004/diff/1/chrome/browser/autocomplete/a...
File chrome/browser/autocomplete/autocomplete_result.cc (right):

https://codereview.chromium.org/17426004/diff/1/chrome/browser/autocomplete/a...
chrome/browser/autocomplete/autocomplete_result.cc:137: alternate_nav_url_ =
default_match_ == end() ?
Nit: Parens around comparison

https://codereview.chromium.org/17426004/diff/1/chrome/browser/ui/omnibox/omn...
File chrome/browser/ui/omnibox/omnibox_edit_model.cc (right):

https://codereview.chromium.org/17426004/diff/1/chrome/browser/ui/omnibox/omn...
chrome/browser/ui/omnibox/omnibox_edit_model.cc:192: if
(match.destination_url.is_valid()) {
Nit: Less indenting:

  if (!match.destination_url.is_valid()) {
    GetInfoForCurrentText(&match, alternate_nav_url);
  } else if (alternate_nav_url) {
    ...

[beaudoin, 2013-06-19 18:21:10]

https://codereview.chromium.org/17426004/diff/1/chrome/browser/autocomplete/a...
File chrome/browser/autocomplete/autocomplete_result.cc (right):

https://codereview.chromium.org/17426004/diff/1/chrome/browser/autocomplete/a...
chrome/browser/autocomplete/autocomplete_result.cc:137: alternate_nav_url_ =
default_match_ == end() ?
On 2013/06/18 23:31:19, Peter Kasting wrote:
> Nit: Parens around comparison

Done.

https://codereview.chromium.org/17426004/diff/1/chrome/browser/ui/omnibox/omn...
File chrome/browser/ui/omnibox/omnibox_edit_model.cc (right):

https://codereview.chromium.org/17426004/diff/1/chrome/browser/ui/omnibox/omn...
chrome/browser/ui/omnibox/omnibox_edit_model.cc:192: if
(match.destination_url.is_valid()) {
On 2013/06/18 23:31:19, Peter Kasting wrote:
> Nit: Less indenting:
> 
>   if (!match.destination_url.is_valid()) {
>     GetInfoForCurrentText(&match, alternate_nav_url);
>   } else if (alternate_nav_url) {
>     ...

Done.

[commit-bot: I haz the power, 2013-06-19 18:22:10]

CQ is trying da patch. Follow status at
https://chromium-status.appspot.com/cq/beaudoin@chromium.org/17426004/6001

[commit-bot: I haz the power, 2013-06-19 18:59:47]

Step "update" is always a major failure.
Look at the try server FAQ for more details.
http://build.chromium.org/p/tryserver.chromium/buildstatus?builder=mac&number...

[commit-bot: I haz the power, 2013-06-19 19:01:21]

CQ is trying da patch. Follow status at
https://chromium-status.appspot.com/cq/beaudoin@chromium.org/17426004/6001

[commit-bot: I haz the power, 2013-06-20 14:09:14]

Change committed as 207429