Move spellcheck underline colors into LayoutTheme

third_party/WebKit/Source/core/layout/LayoutTheme.h

Index: third_party/WebKit/Source/core/layout/LayoutTheme.h
diff --git a/third_party/WebKit/Source/core/layout/LayoutTheme.h b/third_party/WebKit/Source/core/layout/LayoutTheme.h
index e87c075406884822ff905cf1e4a1c5a284397020..4650147fef6f23eaa64f6bc5bd32cbd4dceb6f2d 100644
--- a/third_party/WebKit/Source/core/layout/LayoutTheme.h
+++ b/third_party/WebKit/Source/core/layout/LayoutTheme.h
@@ -130,6 +130,10 @@ class CORE_EXPORT LayoutTheme : public RefCounted<LayoutTheme> {
   Color InactiveListBoxSelectionBackgroundColor() const;
   Color InactiveListBoxSelectionForegroundColor() const;
 
+  // Underline colors for spelling and grammar markers.

[chrishtr, 2017/06/10 00:45:56]

I think this comment is redundant, you can just remove it.

+  Color SpellingMarkerUnderlineColor() const;
+  Color GrammarMarkerUnderlineColor() const;

[chrishtr, 2017/06/10 00:45:56]

What's the point of having these extra methods?

[rlanday, 2017/06/10 16:34:03]

I'm not sure, I was just copying how the existing methods are set up...

+
   // Highlight and text colors for TextMatches.
   Color PlatformTextSearchHighlightColor(bool active_match) const;
   Color PlatformTextSearchColor(bool active_match) const;
@@ -238,6 +242,9 @@ class CORE_EXPORT LayoutTheme : public RefCounted<LayoutTheme> {
   virtual Color PlatformActiveListBoxSelectionForegroundColor() const;
   virtual Color PlatformInactiveListBoxSelectionForegroundColor() const;
 
+  virtual Color PlatformSpellingMarkerUnderlineColor() const;
+  virtual Color PlatformGrammarMarkerUnderlineColor() const;
+
   virtual bool ThemeDrawsFocusRing(const ComputedStyle&) const = 0;
 
   // Methods for each appearance value.