Add custom default theme support and create a managed user default theme.

chrome/browser/themes/custom_theme_provider.h

+// Copyright 2013 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_THEMES_CUSTOM_THEME_PROVIDER_H_
+#define CHROME_BROWSER_THEMES_CUSTOM_THEME_PROVIDER_H_
+
+#include "base/memory/ref_counted.h"
+#include "content/public/browser/browser_thread.h"
+#include "third_party/skia/include/core/SkColor.h"
+#include "ui/base/layout.h"
+#include "ui/gfx/image/image.h"

[pkotwicz, 2013/07/19 05:54:13]

gfx::Image could be forward declared as well?

[Adrian Kuegel, 2013/07/19 13:54:39]

Done. I also changed this in browser_theme_pack.h

+
+namespace base {
+class RefCountedMemory;
+}
+
+namespace color_utils {
+struct HSL;
+}
+
+class CustomThemeProvider : public base::RefCountedThreadSafe<

[pkotwicz, 2013/07/19 05:54:13]

I do not like the name CustomThemeProvider because it leads me to believe that
CustomThemeProvider implements ui::ThemeProvider which it does not.
How about
CustomThemeProperyProvider /  CustomTheme / CustomThemeSupplier

[Adrian Kuegel, 2013/07/19 13:54:39]

I have chosen the name CustomThemeSupplier now. CustomThemePropertyProvider
would probably better cover it, but it seems awfully long.

+    CustomThemeProvider, content::BrowserThread::DeleteOnFileThread> {
+ public:
+  enum ThemeType {
+    EXTENSION,
+    NATIVE_X11,
+    MANAGED_USER_THEME,
+  };
+
+  explicit CustomThemeProvider(ThemeType type);
+
+  ThemeType GetThemeType() const {
+    return theme_type_;
+  }
+
+  // Called when the theme is started being used.

[pkotwicz, 2013/07/19 05:54:13]

Nit: starts being used

[Adrian Kuegel, 2013/07/19 13:54:39]

Done.

+  virtual void StartUsingTheme();
+
+  // Called when the theme is not used anymore.
+  virtual void StopUsingTheme();
+
+  // If the theme specifies data for the corresponding |id|, return true and
+  // write the corresponding value to the output parameter. These functions
+  // should not return the default data. These methods should only be called
+  // from the UI thread.
+  virtual bool GetTint(int id, color_utils::HSL* hsl) const;
+  virtual bool GetColor(int id, SkColor* color) const;
+  virtual bool GetDisplayProperty(int id, int* result) const;
+
+  // Returns the theme image for |id|. Returns an empty image if no image is
+  // found for |id|.
+  virtual gfx::Image GetImageNamed(int id);
+
+  // Whether this theme provides an image for |id|.
+  virtual bool HasCustomImage(int id) const;
+
+  // Returns the raw PNG encoded data for IDR_THEME_NTP_*. This method is only
+  // supposed to work for the NTP attribution and background resources.
+  virtual base::RefCountedMemory* GetRawData(
+      int id, ui::ScaleFactor scale_factor) const;
+
+ protected:
+  virtual ~CustomThemeProvider();
+
+ private:
+  friend struct content::BrowserThread::DeleteOnThread<
+      content::BrowserThread::FILE>;
+  friend class base::DeleteHelper<CustomThemeProvider>;
+
+  ThemeType theme_type_;
+
+  DISALLOW_COPY_AND_ASSIGN(CustomThemeProvider);
+};
+
+#endif  // CHROME_BROWSER_THEMES_CUSTOM_THEME_PROVIDER_H_