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

Unified Diff: remoting/webapp/host_native_messaging.js

Issue 132793007: Download the host components when user tries to enable Me2Me host. (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src
Patch Set: Created 6 years, 11 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 | « remoting/webapp/host_list.js ('k') | remoting/webapp/host_setup_dialog.js » ('j') | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: remoting/webapp/host_native_messaging.js
diff --git a/remoting/webapp/host_native_messaging.js b/remoting/webapp/host_native_messaging.js
index 0d48b004091a97e14a077d26f1a4b41296147f86..3fc847343081cc7f7795aebd9062b4ebe637b684 100644
--- a/remoting/webapp/host_native_messaging.js
+++ b/remoting/webapp/host_native_messaging.js
@@ -365,11 +365,13 @@ remoting.HostNativeMessaging.prototype.onDisconnect_ = function() {
console.error('Native Message port disconnected');
// Notify the error-handlers of any requests that are still outstanding.
- for (var id in this.pendingReplies_) {
- this.pendingReplies_[/** @type {number} */(id)].onError(
+ var pendingReplies = this.pendingReplies_;
+ this.pendingReplies_ = {};
+
+ for (var id in pendingReplies) {
+ pendingReplies[/** @type {number} */(id)].onError(
remoting.Error.UNEXPECTED);
}
- this.pendingReplies_ = {};
}
/**
« no previous file with comments | « remoting/webapp/host_list.js ('k') | remoting/webapp/host_setup_dialog.js » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698