Give request-to-act badges a grey background, and increase spacing to make it fit.

chrome/browser/ui/omnibox/extension_action_util.h

Index: chrome/browser/ui/omnibox/extension_action_util.h
diff --git a/chrome/browser/ui/omnibox/extension_action_util.h b/chrome/browser/ui/omnibox/extension_action_util.h
new file mode 100644
index 0000000000000000000000000000000000000000..c2d478bfd7b3caa4286a4e41fc71bd0df1b39754
--- /dev/null
+++ b/chrome/browser/ui/omnibox/extension_action_util.h
@@ -0,0 +1,41 @@
+// 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 CHROME_BROWSER_UI_OMNIBOX_EXTENSION_ACTION_UTIL_H_
+#define CHROME_BROWSER_UI_OMNIBOX_EXTENSION_ACTION_UTIL_H_
+
+// Utility functions for ExtensionActions that depend on code in the browser
+// component.

[Peter Kasting, 2012/09/11 22:27:47]

Nit: Thanks for moving this to this directory.  Should this maybe go in
LocationBar (the mostly-abstract base class) itself, or if not then maybe in
location_bar_util.h?

I just don't tend to like random files with one function in them.  I'm not
convinced the function here really belongs in the extensions namespace either,
it seems more like "part of the location bar" than "part of extensions".

[Jeffrey Yasskin, 2012/09/11 23:05:28]

I don't care where it lives, but I've already moved it once to the place you
specified. If you wanted it in a different place, you should have said so in
your previous comment. I'm willing to move it one more time, so please tell me
exactly where you want it.

[Peter Kasting, 2012/09/11 23:25:41]

Sorry, I didn't mean to yank your chain around on this :(

I just wasn't thinking hard before about whether there were existing files in
this directory this should be combined with.  (I didn't even know
location_bar_util.h existed.)  I guess I'd probably put the function into the
location_bar_util file/namespace, since if you put it into LocationBar it would
have to be public and static.

[Jeffrey Yasskin, 2012/09/12 20:29:34]

Done.

+
+#include "third_party/skia/include/core/SkColor.h"
+
+class ExtensionAction;
+
+namespace gfx {
+class Canvas;
+class Rect;
+}
+namespace ui {
+class ThemeProvider;
+}
+
+namespace extensions {
+
+// Paint the background and border for |action| on |tab_id|.  |bounds| should
+// include the top and bottom of the location bar, and the middle column
+// exactly between two ExtensionActions, so both ExtensionActions can draw on
+// it.  This background is only drawn for script badges in the WANTS_ATTENTION
+// state, and (when text is black and the background is white) consists of a
+// dark grey, 1px border and a lighter grey background with a slight vertical
+// gradient.
+void PaintExtensionActionBackground(const ExtensionAction& action,
+                                    int tab_id,
+                                    gfx::Canvas* canvas,
+                                    const gfx::Rect& bounds,
+                                    SkColor text_color,
+                                    SkColor background_color);
+
+}  // namespace extensions
+
+#endif  // CHROME_BROWSER_UI_OMNIBOX_EXTENSION_ACTION_UTIL_H_