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

Side by Side Diff: mojo/services/public/interfaces/content_handler/content_handler.mojom

Issue 423963004: First cut at "content handling" support in Mojo. (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src
Patch Set: blech Created 6 years, 4 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 unified diff | Download patch | Annotate | Revision Log
OLDNEW
1 // Copyright 2014 The Chromium Authors. All rights reserved. 1 // Copyright 2014 The Chromium Authors. All rights reserved.
2 // Use of this source code is governed by a BSD-style license that can be 2 // Use of this source code is governed by a BSD-style license that can be
3 // found in the LICENSE file. 3 // found in the LICENSE file.
4 4
5 import "mojo/public/interfaces/application/service_provider.mojom" 5 import "mojo/public/interfaces/application/service_provider.mojom"
6 import "mojo/services/public/interfaces/network/url_loader.mojom"
6 7
7 module mojo { 8 module mojo {
8 9
9 // Applications vend Services through the ServiceProvider interface. Services 10 interface ContentHandler {
darin (slow to review) 2014/08/06 23:10:02 by the way, I'm not 100% sold on the name "OnConne
Aaron Boodman 2014/08/07 00:03:12 Yeah. I am not 100% with it either. There is also
10 // implement Interfaces. 11 OnConnect(string url,
darin (slow to review) 2014/08/06 23:10:02 Isn't this |url| parameter redundant? The URLRespo
Aaron Boodman 2014/08/07 00:03:12 No, because redirects. Also, as I said above in th
11 [Client=Shell] 12 URLResponse url_response,
12 interface Application { 13 ServiceProvider service_provider);
13 AcceptConnection(string requestor_url, ServiceProvider provider);
14 }; 14 };
15 15
16 } 16 }
OLDNEW

Powered by Google App Engine
This is Rietveld 408576698