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

Unified Diff: mojo/shell/connect_to_application_params.cc

Issue 1351443004: Revert of Make CapabilityFilter be part of Identity (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: Created 5 years, 3 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 side-by-side diff with in-line comments
Download patch
« no previous file with comments | « mojo/shell/connect_to_application_params.h ('k') | mojo/shell/connect_util.h » ('j') | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: mojo/shell/connect_to_application_params.cc
diff --git a/mojo/shell/connect_to_application_params.cc b/mojo/shell/connect_to_application_params.cc
index bd0edc2313c7ece6ebe68ba05a97ca1f7e0f1e46..f1f4f2605c45152ad01a723be9780317a8e7d678 100644
--- a/mojo/shell/connect_to_application_params.cc
+++ b/mojo/shell/connect_to_application_params.cc
@@ -13,38 +13,27 @@
ConnectToApplicationParams::~ConnectToApplicationParams() {}
-void ConnectToApplicationParams::SetSource(ApplicationInstance* source) {
- if (!source) {
- source_ = Identity();
+void ConnectToApplicationParams::SetOriginatorInfo(
+ ApplicationInstance* originator) {
+ if (!originator) {
+ originator_identity_ = Identity();
+ originator_filter_.clear();
return;
}
- source_ = source->identity();
+ originator_identity_ = originator->identity();
+ originator_filter_ = originator->filter();
}
-void ConnectToApplicationParams::SetTarget(const Identity& target) {
- target_ = target;
- target_url_request_ = URLRequest::New();
- target_url_request_->url = target_.url().spec();
+void ConnectToApplicationParams::SetURLInfo(const GURL& app_url) {
+ app_url_ = app_url;
+ app_url_request_ = URLRequest::New();
+ app_url_request_->url = app_url_.spec();
}
-void ConnectToApplicationParams::SetTargetURL(const GURL& target_url) {
- target_ = Identity(target_url, target_.qualifier(), target_.filter());
- target_url_request_ = URLRequest::New();
- target_url_request_->url = target_.url().spec();
-}
-
-void ConnectToApplicationParams::SetTargetURLRequest(URLRequestPtr request) {
- Identity target = request ? Identity(GURL(request->url), target_.qualifier(),
- target_.filter())
- : Identity();
- SetTargetURLRequest(request.Pass(), target);
-}
-
-void ConnectToApplicationParams::SetTargetURLRequest(URLRequestPtr request,
- const Identity& target) {
- target_url_request_ = request.Pass();
- target_ = target;
+void ConnectToApplicationParams::SetURLInfo(URLRequestPtr app_url_request) {
+ app_url_request_ = app_url_request.Pass();
+ app_url_ = app_url_request_ ? GURL(app_url_request_->url) : GURL();
}
} // namespace shell
« no previous file with comments | « mojo/shell/connect_to_application_params.h ('k') | mojo/shell/connect_util.h » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698