diff options
| author | darin@chromium.org <darin@chromium.org@0039d316-1c4b-4281-b951-d872f2087c98> | 2009-06-11 22:20:39 +0000 |
|---|---|---|
| committer | darin@chromium.org <darin@chromium.org@0039d316-1c4b-4281-b951-d872f2087c98> | 2009-06-11 22:20:39 +0000 |
| commit | b2787cf4c95b5e22162a345140a9350bcd5b675e (patch) | |
| tree | a2cb733808ccec4cab58919c120b1f0be9dde53d /webkit/api/public/WebURLLoader.h | |
| parent | 23bd81e08bf59f02ad164caa42ea1b18807c2420 (diff) | |
| download | chromium_src-b2787cf4c95b5e22162a345140a9350bcd5b675e.zip chromium_src-b2787cf4c95b5e22162a345140a9350bcd5b675e.tar.gz chromium_src-b2787cf4c95b5e22162a345140a9350bcd5b675e.tar.bz2 | |
Fix javascript-backslash.html layout test failure.
The fix is to break WebURLLoaderImpl into two classes. It now has a
reference counted inner class that is the ResourceLoaderBridge::Peer.
This change allows the inner class to stick around after the loader
has been destroyed. That is necessary since the Peer expects to still
receive OnCompletedRequest after it calls Cancel on the bridge.
R=dglazkov
TEST=covered by layout test
BUG=13786
Review URL: http://codereview.chromium.org/125002
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@18215 0039d316-1c4b-4281-b951-d872f2087c98
Diffstat (limited to 'webkit/api/public/WebURLLoader.h')
| -rw-r--r-- | webkit/api/public/WebURLLoader.h | 1 |
1 files changed, 1 insertions, 0 deletions
diff --git a/webkit/api/public/WebURLLoader.h b/webkit/api/public/WebURLLoader.h index e9027c4..b279987 100644 --- a/webkit/api/public/WebURLLoader.h +++ b/webkit/api/public/WebURLLoader.h @@ -42,6 +42,7 @@ namespace WebKit { class WebURLLoader { public: + // The WebURLLoader may be deleted in a call to its client. virtual ~WebURLLoader() {} // Load the request synchronously, returning results directly to the |