[AiS] Added option to put a space between answer values

chrome/browser/ui/views/omnibox/omnibox_result_view.cc

 // Copyright (c) 2012 The Chromium Authors. All rights reserved.
 // Use of this source code is governed by a BSD-style license that can be
 // found in the LICENSE file.
 
 // For WinDDK ATL compatibility, these ATL headers must come first.
 #include "build/build_config.h"
 #if defined(OS_WIN)
 #include <atlbase.h>  // NOLINT
 #include <atlwin.h>  // NOLINT
 #endif
@@ skipping 681 matching lines @@
     canvas->DrawImageInt(GetIcon(), GetMirroredXForRect(icon_bounds_),
                          icon_bounds_.y());
     int x = GetMirroredXForRect(text_bounds_);
     mirroring_context_->Initialize(x, text_bounds_.width());
     InitContentsRenderTextIfNecessary();
 
     if (!description_rendertext_) {
       if (match_.answer) {
         base::string16 text;
         description_rendertext_ = CreateRenderText(text);
-        for (const SuggestionAnswer::TextField& textfield :
+        for (const SuggestionAnswer::TextField& text_field :
              match_.answer->second_line().text_fields())
-          AppendAnswerText(textfield);
-        if (match_.answer->second_line().additional_text())
-          AppendAnswerText(*match_.answer->second_line().additional_text());
-        if (match_.answer->second_line().status_text())
-          AppendAnswerText(*match_.answer->second_line().status_text());
+          AppendAnswerText(text_field.text(), text_field.type());
+        if (match_.answer->second_line().additional_text()) {

[Peter Kasting, 2015/03/24 01:05:35]

Nit: Briefer, seems a bit more readable:

        base::char16 space(' ');
        const auto* text_field = match_.answer->second_line().additional_text();
        if (text_field)
          AppendAnswerText(space + text_field->text(), text_field->type());
        text_field = match_.answer->second_line().status_text();
        if (text_field)
          AppendAnswerText(space + text_field->text(), text_field->type());

(I believe that char16 trick should work, but if not one could use a string16
there with ASCIIToUTF16() instead.)

[dschuyler, 2015/03/24 01:26:47]

Done.

+          AppendAnswerText(
+              base::ASCIIToUTF16(" ") +
+                  match_.answer->second_line().additional_text()->text(),
+              match_.answer->second_line().additional_text()->type());
+        }
+        if (match_.answer->second_line().status_text()) {
+          AppendAnswerText(
+              base::ASCIIToUTF16(" ") +
+                  match_.answer->second_line().status_text()->text(),
+              match_.answer->second_line().status_text()->type());
+        }
       } else if (!match_.description.empty()) {
         description_rendertext_ = CreateClassifiedRenderText(
             match_.description, match_.description_class, true);
       }
     }
     PaintMatch(match_, contents_rendertext_.get(),
                description_rendertext_.get(), canvas, x);
   }
 
   AutocompleteMatch* keyword_match = match_.associated_keyword.get();
@@ skipping 29 matching lines @@
   return ui::ResourceBundle::GetSharedInstance()
       .GetFontList(GetTextStyle(1).font)
       .GetHeight();
 }
 
 int OmniboxResultView::GetContentLineHeight() const {
   return std::max(default_icon_size_ + (kMinimumIconVerticalPadding * 2),
                   GetTextHeight() + (kMinimumTextVerticalPadding * 2));
 }
 
-void OmniboxResultView::AppendAnswerText(
-    const SuggestionAnswer::TextField& text_field) {
+void OmniboxResultView::AppendAnswerText(const base::string16& text,
+                                         int text_type) {
   int offset = description_rendertext_->text().length();
-  gfx::Range range(offset, offset + text_field.text().length());
-  description_rendertext_->AppendText(text_field.text());
-  const TextStyle& text_style = GetTextStyle(text_field.type());
+  gfx::Range range(offset, offset + text.length());
+  description_rendertext_->AppendText(text);
+  const TextStyle& text_style = GetTextStyle(text_type);
   // TODO(dschuyler): follow up on the problem of different font sizes within
   // one RenderText.
   description_rendertext_->SetFontList(
       ui::ResourceBundle::GetSharedInstance().GetFontList(text_style.font));
   description_rendertext_->ApplyColor(
       GetNativeTheme()->GetSystemColor(text_style.colors[GetState()]), range);
   description_rendertext_->ApplyBaselineStyle(text_style.baseline, range);
 }