Make CapabilityFilter be part of Identity

mojo/shell/connect_to_application_params.h

Index: mojo/shell/connect_to_application_params.h
diff --git a/mojo/shell/connect_to_application_params.h b/mojo/shell/connect_to_application_params.h
index 13268f710a6b705a5cd307a96033efd71aa356fc..330d1d2d7a811ad0095cf5276f7b5a67066f93d8 100644
--- a/mojo/shell/connect_to_application_params.h
+++ b/mojo/shell/connect_to_application_params.h
@@ -12,7 +12,6 @@
 #include "mojo/application/public/interfaces/shell.mojom.h"
 #include "mojo/public/cpp/bindings/interface_request.h"
 #include "mojo/services/network/public/interfaces/url_loader.mojom.h"
-#include "mojo/shell/capability_filter.h"
 #include "mojo/shell/identity.h"
 #include "url/gurl.h"
 
@@ -28,35 +27,26 @@ class ConnectToApplicationParams {
   ConnectToApplicationParams();
   ~ConnectToApplicationParams();
 
-  // Sets both |originator_identity_| and |originator_filter_|. If |originator|
-  // is null, both fields are reset.
-  void SetOriginatorInfo(ApplicationInstance* originator);
+  // Sets |source_|. If |source| is null, |source_| is reset.
+  void SetSource(ApplicationInstance* source);
 
-  // Sets both |app_url_| and |app_url_request_|.
-  void SetURLInfo(const GURL& app_url);
-  // Sets both |app_url_| and |app_url_request_|.
-  void SetURLInfo(URLRequestPtr app_url_request);
+  // Sets both |target_| and |target_url_request_|.
+  void SetTarget(const Identity& target);
+  void SetTargetURL(const GURL& target_url);
+  // Sets both |target_| and |target_url_request_|.

[yzshen1, 2015/09/18 06:07:54]

nit: please consider removing this comment and changing line 33 to "The
following four methods set both ...".

+  void SetTargetURLRequest(URLRequestPtr request);
+  void SetTargetURLRequest(URLRequestPtr request, const Identity& target);
 
-  void set_originator_identity(const Identity& value) {
-    originator_identity_ = value;
-  }
-  const Identity& originator_identity() const { return originator_identity_; }
+  void set_source(const Identity& source) { source_ = source;  }
+  const Identity& source() const { return source_; }
+  const GURL& target_url() const { return target_.url; }
+  const Identity& target() const { return target_; }
 
-  void set_originator_filter(const CapabilityFilter& value) {
-    originator_filter_ = value;
-  }
-  const CapabilityFilter& originator_filter() const {
-    return originator_filter_;
+  const URLRequest* target_url_request() const {
+    return target_url_request_.get();
   }
-
-  const GURL& app_url() const { return app_url_; }
-
-  const URLRequest* app_url_request() const { return app_url_request_.get(); }
-  // NOTE: This doesn't reset |app_url_|.
-  URLRequestPtr TakeAppURLRequest() { return app_url_request_.Pass(); }
-
-  void set_qualifier(const std::string& value) { qualifier_ = value; }
-  const std::string& qualifier() const { return qualifier_; }
+  // NOTE: This doesn't reset |target_|.
+  URLRequestPtr TakeTargetURLRequest() { return target_url_request_.Pass(); }
 
   void set_services(InterfaceRequest<ServiceProvider> value) {
     services_ = value.Pass();
@@ -68,9 +58,6 @@ class ConnectToApplicationParams {
   }
   ServiceProviderPtr TakeExposedServices() { return exposed_services_.Pass(); }
 
-  void set_filter(const CapabilityFilter& value) { filter_ = value; }
-  const CapabilityFilter& filter() const { return filter_; }
-
   void set_on_application_end(const base::Closure& value) {
     on_application_end_ = value;
   }
@@ -87,21 +74,17 @@ class ConnectToApplicationParams {
 
  private:
   // It may be null (i.e., is_null() returns true) which indicates that there is
-  // no originator (e.g., for the first application or in tests).
-  Identity originator_identity_;
-  // Should be ignored if |originator_identity_| is null.
-  CapabilityFilter originator_filter_;
-  // The URL of the application that is being connected to.
-  GURL app_url_;
+  // no source (e.g., for the first application or in tests).
+  Identity source_;
+  // The identity of the application being connected to.
+  Identity target_;
   // The URL request to fetch the application. It may contain more information
   // than |app_url_| (e.g., headers, request body). When it is taken, |app_url_|

[yzshen1, 2015/09/18 06:07:54]

Please update |app_ur_| related comments.

   // remains unchanged.
-  URLRequestPtr app_url_request_;
-  // Please see the comments in identity.h for the exact meaning of qualifier.
-  std::string qualifier_;
+  URLRequestPtr target_url_request_;
+
   InterfaceRequest<ServiceProvider> services_;
   ServiceProviderPtr exposed_services_;
-  CapabilityFilter filter_;
   base::Closure on_application_end_;
   Shell::ConnectToApplicationCallback connect_callback_;