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

Unified Diff: webkit/child/resource_loader_bridge.h

Issue 109283006: Redirect HTML resource bytes directly to parser thread (Chrome side CL) (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: Created 7 years 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: webkit/child/resource_loader_bridge.h
diff --git a/webkit/child/resource_loader_bridge.h b/webkit/child/resource_loader_bridge.h
index e1cd310fd33d2659d94b65e4195ed4a6cd672eed..70343950683e962a0a6330dd64caa5042da0bfff 100644
--- a/webkit/child/resource_loader_bridge.h
+++ b/webkit/child/resource_loader_bridge.h
@@ -33,6 +33,10 @@
#include "webkit/common/resource_response_info.h"
#include "webkit/common/resource_type.h"
+namespace blink {
+class WebParserResourceBridge;
+}
+
namespace webkit_glue {
class ResourceRequestBody;
@@ -181,6 +185,11 @@ class ResourceLoaderBridge {
const std::string& security_info,
const base::TimeTicks& completion_time) = 0;
+ // Called by the parser-resource bridge when its resource filter has
+ // been installed on the I/O thread, and signals Blink that any further
+ // data chunks which are received by the main thread should not be passed
+ // to the parser thread as that now happens directly.
+ virtual void OnParserResourceMessageFilterAdded() { }
protected:
virtual ~Peer() {}
};
@@ -212,6 +221,10 @@ class ResourceLoaderBridge {
// call to the Start method.
virtual void DidChangePriority(net::RequestPriority new_priority) = 0;
+ // Call this method to request an implementation of WebParserResourceBridge
+ // which will be used to interact with the parser thread.
+ virtual blink::WebParserResourceBridge* ConstructParserResourceBridge() = 0;
+
// Call this method to load the resource synchronously (i.e., in one shot).
// This is an alternative to the Start method. Be warned that this method
// will block the calling thread until the resource is fully downloaded or an

Powered by Google App Engine
This is Rietveld 408576698