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

Unified Diff: remoting/webapp/app_remoting/js/feedback_consent.js

Issue 1179873005: [AppRemoting] Break out AppRemoting shared module (re-land). (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: Created 5 years, 6 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
Index: remoting/webapp/app_remoting/js/feedback_consent.js
diff --git a/remoting/webapp/app_remoting/js/feedback_consent.js b/remoting/webapp/app_remoting/js/feedback_consent.js
index 9dce2383a0a891e5f732b02b64cc48b45e924415..2195ea50c7b6b68408f67e65209872c6fb6ff49a 100644
--- a/remoting/webapp/app_remoting/js/feedback_consent.js
+++ b/remoting/webapp/app_remoting/js/feedback_consent.js
@@ -5,6 +5,11 @@
'use strict';
/**
+ * @type {string} The app id (from the webstore) for this application.
+ */
+var appId = '';
+
+/**
* @type {string} The host id corresponding to the user's VM. The @pending
* place-holder instructs the Orchestrator to abandon any pending host,
* and is used if no host id is provided by the main window.
@@ -140,7 +145,7 @@ function onToken(token) {
'crashServiceReportId': crashServiceReportId
};
var uri = remoting.settings.APP_REMOTING_API_BASE_URL +
- '/applications/' + remoting.settings.getAppRemotingApplicationId() +
+ '/applications/' + appId +
'/hosts/' + hostId +
'/reportIssue';
var onDone = function(/** !remoting.Xhr.Response */ response) {
@@ -250,6 +255,7 @@ function onWindowMessage(event) {
if (event.data['hostId']) {
hostId = /** @type {string} */ (event.data['hostId']);
}
+ appId = /** @type {string} */ (event.data['appId']);
connectionStats = /** @type {string} */ (event.data['connectionStats']);
sessionId = /** @type {string} */ (event.data['sessionId']);
}
« no previous file with comments | « remoting/webapp/app_remoting/js/ar_main.js ('k') | remoting/webapp/app_remoting/manifest_common.json.jinja2 » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698