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

Unified Diff: content/public/browser/resource_dispatcher_host_delegate.h

Issue 12645004: Add Resource Handler for creating Streams to forward to extensions (Closed) Base URL: http://git.chromium.org/chromium/src.git@master
Patch Set: Make sure things are called on the right thread. Created 7 years, 9 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: content/public/browser/resource_dispatcher_host_delegate.h
diff --git a/content/public/browser/resource_dispatcher_host_delegate.h b/content/public/browser/resource_dispatcher_host_delegate.h
index 890f1c7b728f77867de115c5d85df3a2ba7a7937..bb8e1e6a5274a980e03218c3512a0bb6927a925c 100644
--- a/content/public/browser/resource_dispatcher_host_delegate.h
+++ b/content/public/browser/resource_dispatcher_host_delegate.h
@@ -8,6 +8,7 @@
#include <string>
#include "base/basictypes.h"
+#include "base/memory/scoped_ptr.h"
#include "content/common/content_export.h"
#include "webkit/glue/resource_type.h"
@@ -21,6 +22,7 @@ class AppCacheService;
namespace content {
class ResourceContext;
class ResourceThrottle;
+class StreamHandle;
struct Referrer;
struct ResourceResponse;
}
@@ -107,6 +109,32 @@ class CONTENT_EXPORT ResourceDispatcherHostDelegate {
virtual bool ShouldForceDownloadResource(
const GURL& url, const std::string& mime_type);
+ // Returns true and sets |security_origin| and |target_id| if a Stream should
+ // be created for the resource.
+ // If true is returned, a new Stream will be created and OnStreamCreated will
+ // be called with the |target_id| returned by this function and a URL to read
+ // the Stream which is accessible from the |security_origin| returned by this
+ // function.
+ virtual bool ShouldInterceptResourceAsStream(
+ content::ResourceContext* resource_context,
+ const GURL& url,
+ const std::string& mime_type,
+ GURL* security_origin,
+ std::string* target_id);
+
+ // Informs the delegate that a stream was created. |target_id| will be filled
+ // with the parameter returned by ShouldInterceptResourceAsStream(). The
+ // Stream can be read from the blob URL |stream_url|, but can only be read
darin (slow to review) 2013/03/19 06:12:51 nit: stream_url no longer exists nit: you should d
Zachary Kuznia 2013/03/19 06:59:35 Done.
+ // once.
+ virtual void OnStreamCreated(
+ content::ResourceContext* resource_context,
+ int render_process_id,
+ int render_view_id,
+ const std::string& target_id,
+ scoped_ptr<StreamHandle> stream,
+ const std::string& mime_type,
+ const GURL& original_url);
+
// Informs the delegate that a response has started.
virtual void OnResponseStarted(
net::URLRequest* request,

Powered by Google App Engine
This is Rietveld 408576698