Passing BackgroundColorSpan and UnderlineSpan from Clank to Blink.

ui/base/ime/composition_underline.h

 // 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.
 
 #ifndef UI_BASE_IME_COMPOSITION_UNDERLINE_H_
 #define UI_BASE_IME_COMPOSITION_UNDERLINE_H_
 
 #include <vector>
 
 #include "third_party/skia/include/core/SkColor.h"
 
 namespace ui {
 
 // Intentionally keep sync with blink::WebCompositionUnderline defined in:
 // third_party/WebKit/public/web/WebCompositionUnderline.h
 struct CompositionUnderline {
   CompositionUnderline()
     : start_offset(0),
       end_offset(0),
       color(0),
-      thick(false) {}
+      thick(false),
+      background_color(0) {}
 
+  // TODO(huangs): remove this constructor.
   CompositionUnderline(unsigned s, unsigned e, SkColor c, bool t)
     : start_offset(s),
       end_offset(e),
       color(c),
-      thick(t) {}
+      thick(t),
+      background_color(0) {}
+
+  CompositionUnderline(unsigned s, unsigned e, SkColor c, bool t, SkColor bc)

[palmer, 2014/06/10 20:32:23]

Ahh, I see. Since these values cross the native/Java barrier, it might be better
to be explicit about their widths. uint32_t and juint, are 32-bit unsigned, for
example.

[huangs, 2014/06/11 02:41:01]

Please note the comment below in line 51 stating why "unsigned" was used.

Previously std::vector<ui::CompositionUnderlines> was reinterpret_casted to
std::vector<blink::WebCompositionUnderline>.  *However*, in

https://codereview.chromium.org/319553002

I got rid of the code.  So I'll go with using uint32 (since this is used often
in the module), and do explicit conversion in render_widget_host_view_aura.cc.

Also updated imm32_manager.cc to reflect this.

+    : start_offset(s),
+      end_offset(e),
+      color(c),
+      thick(t),
+      background_color(bc) {}
 
   bool operator==(const CompositionUnderline& rhs) const {
     return (this->start_offset == rhs.start_offset) &&
         (this->end_offset == rhs.end_offset) &&
         (this->color == rhs.color) &&
-        (this->thick == rhs.thick);
+        (this->thick == rhs.thick) &&
+        (this->background_color == rhs.background_color);
   }
 
   bool operator!=(const CompositionUnderline& rhs) const {
     return !(*this == rhs);
   }
 
   // Though use of unsigned is discouraged, we use it here to make sure it's
   // identical to blink::WebCompositionUnderline.
   unsigned start_offset;
   unsigned end_offset;
   SkColor color;
   bool thick;
+  SkColor background_color;
 };
 
 typedef std::vector<CompositionUnderline> CompositionUnderlines;
 
 }  // namespace ui
 
 #endif  // UI_BASE_IME_COMPOSITION_UNDERLINE_H_