Mojo: Fix two bugs in content handling

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

 // Copyright 2014 The Chromium Authors. All rights reserved.
 // Use of this source code is governed by a BSD-style license that can be
 // found in the LICENSE file.
 
 import "mojo/public/interfaces/application/service_provider.mojom"
 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);
 };
 
 }