Passing BackgroundColorSpan and UnderlineSpan from Clank to Blink.

ui/aura/remote_window_tree_host_win.cc

Index: ui/aura/remote_window_tree_host_win.cc
diff --git a/ui/aura/remote_window_tree_host_win.cc b/ui/aura/remote_window_tree_host_win.cc
index 018406f372a2ad622522119ca9d87380b3be6653..0cd5ee878e9eef89d3ed56b17c374d24b275fe45 100644
--- a/ui/aura/remote_window_tree_host_win.cc
+++ b/ui/aura/remote_window_tree_host_win.cc
@@ -77,6 +77,7 @@ void FillCompositionText(
     composition_text->underlines[i].end_offset = underlines[i].end_offset;
     composition_text->underlines[i].color = SK_ColorBLACK;
     composition_text->underlines[i].thick = underlines[i].thick;
+    composition_text->underlines[i].background_color = SK_ColorTRANSPARENT;

[sadrul, 2014/06/11 21:49:39]

I assume you are not adding a background-color to metro_viewer::UnderlineInfo
because it also lacks a foreground-color?

[huangs, 2014/06/12 20:25:34]

Originally yes, so I dismissed it as irrelevant.  I just saw the comment in

ui/metro_viewer/ime_types.h

states that metro_viewer::UnderlineInfo is a copy of ui::CompositionUnderline. 
But doing so would require dependency on

#include "third_party/skia/include/core/SkColor.h"

I think adding support will open a can or worms, and I prefer to save the
cleanup to a future CL if you're okay with that.

   }
 }