| Index: content/public/browser/color_chooser.h
|
| diff --git a/content/public/browser/color_chooser.h b/content/public/browser/color_chooser.h
|
| index 9b0bf40f22bb22fdd248180c1c0c1027dd59026f..9cde94f883b88cb46a9662fb2f86d41d12c265b0 100644
|
| --- a/content/public/browser/color_chooser.h
|
| +++ b/content/public/browser/color_chooser.h
|
| @@ -11,24 +11,10 @@ namespace content {
|
|
|
| class WebContents;
|
|
|
| -// Abstraction object for color choosers for each platform.
|
| +// Interface for a color chooser.
|
| class ColorChooser {
|
| public:
|
| - static ColorChooser* Create(int identifier,
|
| - WebContents* web_contents,
|
| - SkColor initial_color);
|
| -
|
| - explicit ColorChooser(int identifier) : identifier_(identifier) {}
|
| - virtual ~ColorChooser() {}
|
| -
|
| - // Returns a unique identifier for this chooser. Identifiers are unique
|
| - // across a renderer process. This avoids race conditions in synchronizing
|
| - // the browser and renderer processes. For example, if a renderer closes one
|
| - // chooser and opens another, and simultaneously the user picks a color in the
|
| - // first chooser, the IDs can be used to drop the "chose a color" message
|
| - // rather than erroneously tell the renderer that the user picked a color in
|
| - // the second chooser.
|
| - int identifier() const { return identifier_; }
|
| + virtual ~ColorChooser() {};
|
|
|
| // Ends connection with color chooser. Closes color chooser depending on the
|
| // platform.
|
| @@ -36,9 +22,6 @@ class ColorChooser {
|
|
|
| // Sets the selected color.
|
| virtual void SetSelectedColor(SkColor color) = 0;
|
| -
|
| - private:
|
| - int identifier_;
|
| };
|
|
|
| } // namespace content
|
|
|