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 {
 

[darin (slow to review), 2014/08/27 06:34:19]

ContentSource perhaps?

My motivation for thinking about different names:
1- ContentHandlerResponse is kind of long
2- ContentHandlerResponse leads to awkward |response->response| code

(this choice doesn't need to block this CL... just food for thought really)

[Aaron Boodman, 2014/08/27 18:26:13]

I like ContentSource, thanks. Changed.

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