[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 683 matching lines @@
     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 :
              match_.answer->second_line().text_fields())
-          AppendAnswerText(textfield);
+          AppendAnswerText(textfield, false);
         if (match_.answer->second_line().additional_text())
-          AppendAnswerText(*match_.answer->second_line().additional_text());
+          AppendAnswerText(*match_.answer->second_line().additional_text(),
+                           true);

[Peter Kasting, 2015/03/23 20:19:21]

I think it would be easier to just manually prepend the space in this call and
the next one than adding functionality to the other function.

[dschuyler, 2015/03/24 00:52:41]

Done.

         if (match_.answer->second_line().status_text())
-          AppendAnswerText(*match_.answer->second_line().status_text());
+          AppendAnswerText(*match_.answer->second_line().status_text(), true);
       } 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 30 matching lines @@
       .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) {
+    const SuggestionAnswer::TextField& text_field,
+    bool prefix_space) {
+  base::string16 text = text_field.text();
+  if (prefix_space)
+    text.insert(0, base::UTF8ToUTF16(" "));

[Peter Kasting, 2015/03/23 20:19:21]

Use ASCIIToUTF16(), it's cheaper.

[dschuyler, 2015/03/24 00:52:41]

Done.

   int offset = description_rendertext_->text().length();
-  gfx::Range range(offset, offset + text_field.text().length());
-  description_rendertext_->AppendText(text_field.text());
+  gfx::Range range(offset, offset + text.length());
+  description_rendertext_->AppendText(text);
   const TextStyle& text_style = GetTextStyle(text_field.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);
 }