Mojo: Fix two bugs in content handling

mojo/services/public/interfaces/content_handler/content_handler.mojom

Index: mojo/services/public/interfaces/content_handler/content_handler.mojom
diff --git a/mojo/services/public/interfaces/content_handler/content_handler.mojom b/mojo/services/public/interfaces/content_handler/content_handler.mojom
index 1a0ce072f074f5658a95b14e1b69d02127226816..134827909a11b796fe8fd04f2b289827c64c199e 100644
--- a/mojo/services/public/interfaces/content_handler/content_handler.mojom
+++ b/mojo/services/public/interfaces/content_handler/content_handler.mojom
@@ -7,10 +7,15 @@ import "mojo/services/public/interfaces/network/url_loader.mojom"
 
 module mojo {
 
+struct ContentHandlerResponse {

[darin (slow to review), 2014/08/27 03:22:04]

hmm..... this reminds me of the DocumentLoader object in blink. that object is
also called a WebDataSource at the blink API layer. just food for thought re:
naming.

[Aaron Boodman, 2014/08/27 05:43:51]

Doesn't really do anything for me, but if you want me to change it, I will.

+  URLLoader loader;
+  URLResponse response;
+};
+
 interface ContentHandler {
-  OnConnect(string? url,
-            URLResponse? url_response,
-            ServiceProvider? service_provider);
+  OnConnect(string url,
+            ContentHandlerResponse url_response,
+            ServiceProvider&? service_provider);
 };
 
 }