diff options
| author | jam@chromium.org <jam@chromium.org@0039d316-1c4b-4281-b951-d872f2087c98> | 2011-12-30 00:53:09 +0000 |
|---|---|---|
| committer | jam@chromium.org <jam@chromium.org@0039d316-1c4b-4281-b951-d872f2087c98> | 2011-12-30 00:53:09 +0000 |
| commit | 46adf7ff93ebbbfdaf6f8ad7b7253018f5cff3dc (patch) | |
| tree | 1c9dc2964223d04d4146d57bf4878642589bcb3c /chrome/browser/ui/browser_list.h | |
| parent | 34f15c3622301880f8cfd3e34b6cf4e1c6452ee3 (diff) | |
| download | chromium_src-46adf7ff93ebbbfdaf6f8ad7b7253018f5cff3dc.zip chromium_src-46adf7ff93ebbbfdaf6f8ad7b7253018f5cff3dc.tar.gz chromium_src-46adf7ff93ebbbfdaf6f8ad7b7253018f5cff3dc.tar.bz2 | |
Simplify WebUIMessageHandler a bit in preparation for moving it into the Content API:
-got rid of the Attach function which could be overridden by subclasses. Apart from not fitting in with the pure virtual function convention of the Content API, it was redundant to have it and RegisterMessages, since any initialization could be done in the latter.
-made WebUI::AddMessageHandler automatically set the WebUI object on WebUIMessageHandler and call RegisterMessages. It seemed busy work to require every user of WebUIMessageHandler to both add it to the WebUI object and also tell the WebUIMessageHandler about the WebUI.
-removed the usage of the web_ui_ member variable and switched all callers to use web_ui() instead
-made the helper functions static to make it clear that they don't depend on the object
In followup changes, I'll move WebUIMessageHandler to a separate file in content/public/browser and put it in the content namespace.
BUG=98716
Review URL: http://codereview.chromium.org/8968031
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@116011 0039d316-1c4b-4281-b951-d872f2087c98
Diffstat (limited to 'chrome/browser/ui/browser_list.h')
0 files changed, 0 insertions, 0 deletions