Set the focus ring color to match the Gtk theme focus color.

chrome/common/render_messages.h

 // Copyright (c) 2006-2009 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 CHROME_COMMON_RENDER_MESSAGES_H_
 #define CHROME_COMMON_RENDER_MESSAGES_H_
 
 #include <string>
 #include <vector>
 #include <map>
@@ skipping 1556 matching lines @@
 
 // Traits for RendererPreferences structure to pack/unpack.
 template <>
 struct ParamTraits<RendererPreferences> {
   typedef RendererPreferences param_type;
   static void Write(Message* m, const param_type& p) {
     WriteParam(m, p.can_accept_load_drops);
     WriteParam(m, p.should_antialias_text);
     WriteParam(m, static_cast<int>(p.hinting));
     WriteParam(m, static_cast<int>(p.subpixel_rendering));
+    WriteParam(m, p.focus_ring_color_r);

[darin (slow to review), 2009/09/10 22:50:15]

why not just send the WebKit::WebColor over IPC?  it is just an unsigned int, so
you won't have to write ParamTraits for it.  you could also use a SkColor which
is equivalent to WebColor.

[Evan Stade, 2009/09/10 22:53:28]

well, I can change the format of how the color is sent, but I believe it does
belong in RenderPreferences, so I would still need to edit the ParamTraits.

+    WriteParam(m, p.focus_ring_color_g);
+    WriteParam(m, p.focus_ring_color_b);
     WriteParam(m, p.browser_handles_top_level_requests);
   }
   static bool Read(const Message* m, void** iter, param_type* p) {
     if (!ReadParam(m, iter, &p->can_accept_load_drops))
       return false;
     if (!ReadParam(m, iter, &p->should_antialias_text))
       return false;
 
     int hinting = 0;
     if (!ReadParam(m, iter, &hinting))
       return false;
     p->hinting = static_cast<RendererPreferencesHintingEnum>(hinting);
 
     int subpixel_rendering = 0;
     if (!ReadParam(m, iter, &subpixel_rendering))
       return false;
     p->subpixel_rendering =
         static_cast<RendererPreferencesSubpixelRenderingEnum>(
             subpixel_rendering);
 
+    int focus_ring_color_r;
+    if (!ReadParam(m, iter, &focus_ring_color_r))
+      return false;
+    p->focus_ring_color_r = focus_ring_color_r;
+
+    int focus_ring_color_g;
+    if (!ReadParam(m, iter, &focus_ring_color_g))
+      return false;
+    p->focus_ring_color_g = focus_ring_color_g;
+
+    int focus_ring_color_b;
+    if (!ReadParam(m, iter, &focus_ring_color_b))
+      return false;
+    p->focus_ring_color_b = focus_ring_color_b;
+
     if (!ReadParam(m, iter, &p->browser_handles_top_level_requests))
       return false;
 
     return true;
   }
   static void Log(const param_type& p, std::wstring* l) {
     l->append(L"<RendererPreferences>");
   }
 };
 
@@ skipping 561 matching lines @@
   }
 };
 
 }  // namespace IPC
 
 
 #define MESSAGES_INTERNAL_FILE "chrome/common/render_messages_internal.h"
 #include "ipc/ipc_message_macros.h"
 
 #endif  // CHROME_COMMON_RENDER_MESSAGES_H_