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

Unified Diff: webkit/glue/alt_404_page_resource_fetcher.h

Issue 149172: Modify ResourceFetcher to use WebURLLoader instead of ResourceHandle.... (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src/
Patch Set: '' Created 11 years, 5 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
« no previous file with comments | « no previous file | webkit/glue/alt_404_page_resource_fetcher.cc » ('j') | webkit/glue/resource_fetcher.cc » ('J')
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: webkit/glue/alt_404_page_resource_fetcher.h
===================================================================
--- webkit/glue/alt_404_page_resource_fetcher.h (revision 19821)
+++ webkit/glue/alt_404_page_resource_fetcher.h (working copy)
@@ -2,8 +2,8 @@
// Use of this source code is governed by a BSD-style license that can be
// found in the LICENSE file.
-#ifndef WEBKIT_GLUE_ALT_404_PAGE_RESOURCE_HANDLE_CLIENT_H__
-#define WEBKIT_GLUE_ALT_404_PAGE_RESOURCE_HANDLE_CLIENT_H__
+#ifndef WEBKIT_GLUE_ALT_404_PAGE_RESOURCE_HANDLE_CLIENT_H_
+#define WEBKIT_GLUE_ALT_404_PAGE_RESOURCE_HANDLE_CLIENT_H_
#include <string>
@@ -14,19 +14,18 @@
class WebFrameLoaderClient;
+namespace webkit_glue {
+
// ResourceHandleClient implementation that is used for downloading alternate
// 404 pages. Once downloading is done (or fails), the WebFrameLoaderClient is
// notified.
-class Alt404PageResourceFetcher : public ResourceFetcher::Delegate {
+class Alt404PageResourceFetcher {
public:
Alt404PageResourceFetcher(WebFrameLoaderClient* webframeloaderclient,
WebCore::Frame* frame,
WebCore::DocumentLoader* doc_loader,
const GURL& url);
- virtual void OnURLFetchComplete(const WebCore::ResourceResponse& response,
- const std::string& data);
-
// Stop any pending loads.
void Cancel() {
if (fetcher_.get())
@@ -34,6 +33,9 @@
}
private:
+ void OnURLFetchComplete(const WebKit::WebURLResponse& response,
+ const std::string& data);
+
// Does the actual fetching.
scoped_ptr<ResourceFetcherWithTimeout> fetcher_;
@@ -45,7 +47,9 @@
// original load.
RefPtr<WebCore::DocumentLoader> doc_loader_;
- DISALLOW_EVIL_CONSTRUCTORS(Alt404PageResourceFetcher);
+ DISALLOW_COPY_AND_ASSIGN(Alt404PageResourceFetcher);
};
-#endif // WEBKIT_GLUE_ALT_404_PAGE_RESOURCE_HANDLE_CLIENT_H__
+} // namespace webkit_glue
+
+#endif // WEBKIT_GLUE_ALT_404_PAGE_RESOURCE_HANDLE_CLIENT_H_
« no previous file with comments | « no previous file | webkit/glue/alt_404_page_resource_fetcher.cc » ('j') | webkit/glue/resource_fetcher.cc » ('J')

Powered by Google App Engine
This is Rietveld 408576698