Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(1964)

Unified Diff: Source/web/ColorChooserPopupUIController.h

Issue 631833002: Move color chooser objects to the Oilpan heap. (Closed) Base URL: https://chromium.googlesource.com/chromium/blink.git@master
Patch Set: Make ColorChooserUIController::m_client protected, not private Created 6 years, 2 months ago
Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.
Jump to:
View side-by-side diff with in-line comments
Download patch
Index: Source/web/ColorChooserPopupUIController.h
diff --git a/Source/web/ColorChooserPopupUIController.h b/Source/web/ColorChooserPopupUIController.h
index ee9d531c8c32595e407da629e44a765a692b3e6f..b0694b3178782a2b200c672d6dc2eb1dfc26fdc5 100644
--- a/Source/web/ColorChooserPopupUIController.h
+++ b/Source/web/ColorChooserPopupUIController.h
@@ -36,38 +36,43 @@ class ChromeClientImpl;
class ColorChooserClient;
class PagePopup;
-class ColorChooserPopupUIController FINAL : public ColorChooserUIController, public PagePopupClient {
+class ColorChooserPopupUIController final : public ColorChooserUIController, public PagePopupClient {
public:
- ColorChooserPopupUIController(LocalFrame*, ChromeClientImpl*, ColorChooserClient*);
+ static PassOwnPtrWillBeRawPtr<ColorChooserPopupUIController> create(LocalFrame* frame, ChromeClientImpl* chromeClient, ColorChooserClient* client)
+ {
+ return adoptPtrWillBeNoop(new ColorChooserPopupUIController(frame, chromeClient, client));
+ }
+
virtual ~ColorChooserPopupUIController();
// ColorChooserUIController functions:
- virtual void openUI() OVERRIDE;
+ virtual void openUI() override;
// ColorChooser functions
- virtual void endChooser() OVERRIDE;
- virtual AXObject* rootAXObject() OVERRIDE;
+ virtual void endChooser() override;
+ virtual AXObject* rootAXObject() override;
// PagePopupClient functions:
- virtual IntSize contentSize() OVERRIDE;
- virtual void writeDocument(SharedBuffer*) OVERRIDE;
- virtual Locale& locale() OVERRIDE;
- virtual void setValueAndClosePopup(int, const String&) OVERRIDE;
- virtual void setValue(const String&) OVERRIDE;
- virtual void closePopup() OVERRIDE;
- virtual Element& ownerElement() OVERRIDE;
- virtual void didClosePopup() OVERRIDE;
+ virtual IntSize contentSize() override;
+ virtual void writeDocument(SharedBuffer*) override;
+ virtual Locale& locale() override;
+ virtual void setValueAndClosePopup(int, const String&) override;
+ virtual void setValue(const String&) override;
+ virtual void closePopup() override;
+ virtual Element& ownerElement() override;
+ virtual void didClosePopup() override;
private:
+ ColorChooserPopupUIController(LocalFrame*, ChromeClientImpl*, ColorChooserClient*);
+
void openPopup();
ChromeClientImpl* m_chromeClient;
- ColorChooserClient* m_client;
PagePopup* m_popup;
Locale& m_locale;
};
-}
+} // namespace blink
#endif // ColorChooserPopupUIController_h

Powered by Google App Engine
This is Rietveld 408576698