Make infobar background gray

chrome/browser/ui/infobar.h

+// Copyright (c) 2011 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_INFOBAR_H_
+#define CHROME_BROWSER_UI_INFOBAR_H_
+#pragma once
+
+#include "chrome/browser/tab_contents/infobar_delegate.h"
+#include "third_party/skia/include/core/SkColor.h"
+

[Robert Sesek, 2011/06/07 14:48:53]

Namespace level comment that explains what these functions are for.

[sail, 2011/06/07 19:21:02]

Moved to tab_contents/infobar.h instead.

+namespace infobar {

[tfarina, 2011/06/07 15:06:27]

please, could you rename this file to infobar_util/infobar_utils? And change the
namespace according? There are too many infobar.* files already.

[Peter Kasting, 2011/06/07 18:13:13]

I would prefer this to be named infobar_background.*, have no namespace inside,
and live in chrome/browser/tab_contents or chrome/browser/ui/infobars.

[sail, 2011/06/07 19:21:02]

Moved to tab_contents/infobar.h instead.

[sail, 2011/06/07 19:21:02]

Done.

[Robert Sesek, 2011/06/07 20:03:50]

I actually prefer the namespace rather than global functions. Why would you
prefer no namespace? This seems to be the opposite direction Chromium is going
with chrome/content refactoring.

+
+SkColor GetTopColor(InfoBarDelegate::Type infobar_type);
+SkColor GetBottomColor(InfoBarDelegate::Type infobar_type);
+
+} // namespace infobar
+
+#endif  // CHROME_BROWSER_UI_INFOBAR_H_