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

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

Index: chrome/browser/ui/views/omnibox/omnibox_result_view.h
diff --git a/chrome/browser/ui/views/omnibox/omnibox_result_view.h b/chrome/browser/ui/views/omnibox/omnibox_result_view.h
index 1df1b2caacc089703851e6b503cd4f3951b7e4fd..6609ed979bcca58073aaef1079114134c8425cf9 100644
--- a/chrome/browser/ui/views/omnibox/omnibox_result_view.h
+++ b/chrome/browser/ui/views/omnibox/omnibox_result_view.h
@@ -10,7 +10,6 @@
 #include "base/gtest_prod_util.h"
 #include "chrome/browser/bitmap_fetcher/bitmap_fetcher_service.h"
 #include "components/omnibox/autocomplete_match.h"
-#include "components/omnibox/suggestion_answer.h"
 #include "third_party/skia/include/core/SkColor.h"
 #include "ui/gfx/animation/animation_delegate.h"
 #include "ui/gfx/animation/slide_animation.h"
@@ -160,7 +159,8 @@ class OmniboxResultView : public views::View,
   int GetAnswerLineHeight() const;
   int GetContentLineHeight() const;
 
-  void AppendAnswerText(const SuggestionAnswer::TextField& text_field);
+  // Adds styled text to |description_rendertext_|.

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

Nit: Can you flesh this out more?  How is the text styled?  For example:

Adds |text| to |description_rendertext_|.  |text_type| is an index into the
kTextStyles constant defined in the .cc file and is used to style the text,
including setting the font size, color, and baseline style.  See the TextStyle
struct in the .cc file for more.

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

Thanks!

Done.

+  void AppendAnswerText(const base::string16& text, int text_type);

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

Nit: |text_style| might be a better name for this arg.

Now that we're passing this to a function defined in the .h I _almost_ want to
make it an enum instead of an int, but I think an int is probably still OK for
now.

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

I call the TextStyle instance text_style in the same function.  Also the
function being called to get the text_type is type().  Maybe not be best names
ever, but there somewhat consistent in this space.  (pun intended).

[Peter Kasting, 2015/03/24 01:44:56]

Fair enough, but this highlights for me that there's a type mismatch problem
here.  You take this arg as an int in this function, but GetTextStyle() in the
.cc file takes a size_t.  The conversion is done implicitly.

Probably GetTextStyle() should take an int and clamp values of 0 or less to 1. 
(Right now, calling that function with a type of 0 is actually going to crash
Chrome, which seems bad.)

[dschuyler, 2015/03/24 02:20:10]

Done.

 
   static int default_icon_size_;