diff options
| author | darin@chromium.org <darin@chromium.org@0039d316-1c4b-4281-b951-d872f2087c98> | 2009-08-11 19:36:56 +0000 |
|---|---|---|
| committer | darin@chromium.org <darin@chromium.org@0039d316-1c4b-4281-b951-d872f2087c98> | 2009-08-11 19:36:56 +0000 |
| commit | 99519dc7184abc349683c092a292923604ba9f5b (patch) | |
| tree | 4c103ff4ee68bd0d4e1d39d3b88c6d7e582a949c /webkit/glue/webview.h | |
| parent | 0b2765e449455ccd543d829df055a61fb3001108 (diff) | |
| download | chromium_src-99519dc7184abc349683c092a292923604ba9f5b.zip chromium_src-99519dc7184abc349683c092a292923604ba9f5b.tar.gz chromium_src-99519dc7184abc349683c092a292923604ba9f5b.tar.bz2 | |
Remove WebView::SetDelegate because I'd like to avoid having a method like this
in the WebKit API.
The only consumer was TestShell. It was using this method to replace its
TestWebViewDelegate instance. Instead, with this change, it has a manual Reset
method. To avoid duplication with the constructor, Reset uses operator=().
This required a couple changes:
1- Remove DISALLOW_COPY_AND_ASSIGN from WebViewDelegate. Anyways, that didn't
make sense since you cannot 'copy' a class with pure virtual methods.
2- Change scoped_ptr members of TestWebViewDelegate to linked_ptr. The extra
overhead of the linked_ptr seems warranted in this case.
I also changed TestWebViewDelegate to not be reference counted since it wasn't
necessary.
R=tony
BUG=none
TEST=none
Review URL: http://codereview.chromium.org/164308
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@23068 0039d316-1c4b-4281-b951-d872f2087c98
Diffstat (limited to 'webkit/glue/webview.h')
| -rw-r--r-- | webkit/glue/webview.h | 3 |
1 files changed, 0 insertions, 3 deletions
diff --git a/webkit/glue/webview.h b/webkit/glue/webview.h index 6e2c794..c329452 100644 --- a/webkit/glue/webview.h +++ b/webkit/glue/webview.h @@ -66,9 +66,6 @@ class WebView : public WebKit::WebWidget { // it, it will be NULL during closing of the view. virtual WebViewDelegate* GetDelegate() = 0; - // Changes the delegate for this WebView. It is valid to set this to NULL. - virtual void SetDelegate(WebViewDelegate* delegate) = 0; - // Instructs the EditorClient whether to pass editing notifications on to a // delegate, if one is present. This allows embedders that haven't // overridden any editor delegate methods to avoid the performance impact of |