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

Side by Side Diff: remoting/webapp/base/js/ipc.js

Issue 888323002: Improve HRD first run experience (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: Ready for Checkin Created 5 years, 10 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
« no previous file with comments | « remoting/webapp/base/js/base.js ('k') | remoting/webapp/base/js/message_window.js » ('j') | no next file with comments »
Toggle Intra-line Diffs ('i') | Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
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 /** 5 /**
6 * @fileoverview 6 * @fileoverview
7 * 7 *
8 * In Chrome Apps, some platform APIs can only be called from the background 8 * In Chrome Apps, some platform APIs can only be called from the background
9 * page (e.g. reloading a chrome.app.AppWindow). Likewise, some chrome API's 9 * page (e.g. reloading a chrome.app.AppWindow). Likewise, some chrome API's
10 * must be initiated by user interaction, which can only be called from the 10 * must be initiated by user interaction, which can only be called from the
(...skipping 71 matching lines...) Expand 10 before | Expand all | Expand 10 after
82 * @private 82 * @private
83 */ 83 */
84 base.Ipc.Request_ = function(methodName, params) { 84 base.Ipc.Request_ = function(methodName, params) {
85 this.methodName = methodName; 85 this.methodName = methodName;
86 this.params = params; 86 this.params = params;
87 }; 87 };
88 88
89 89
90 /** 90 /**
91 * @param {string} methodName 91 * @param {string} methodName
92 * @param {function(?)} handler The handler can be invoked by calling 92 * @param {function(...?)} handler The handler can be invoked by calling
93 * base.Ipc.invoke(|methodName|, arg1, arg2, ...) 93 * base.Ipc.invoke(|methodName|, arg1, arg2, ...)
94 * Async handlers that return promises are currently not supported. 94 * Async handlers that return promises are currently not supported.
95 * @return {boolean} Whether the handler is successfully registered. 95 * @return {boolean} Whether the handler is successfully registered.
96 */ 96 */
97 base.Ipc.prototype.register = function(methodName, handler) { 97 base.Ipc.prototype.register = function(methodName, handler) {
98 if (methodName in this.handlers_) { 98 if (methodName in this.handlers_) {
99 console.error('service ' + methodName + ' is already registered.'); 99 console.error('service ' + methodName + ' is already registered.');
100 return false; 100 return false;
101 } 101 }
102 this.handlers_[methodName] = handler; 102 this.handlers_[methodName] = handler;
(...skipping 75 matching lines...) Expand 10 before | Expand all | Expand 10 after
178 }; 178 };
179 179
180 base.Ipc.deleteInstance = function() { 180 base.Ipc.deleteInstance = function() {
181 if (instance_) { 181 if (instance_) {
182 instance_.dispose_(); 182 instance_.dispose_();
183 instance_ = null; 183 instance_ = null;
184 } 184 }
185 }; 185 };
186 186
187 })(); 187 })();
OLDNEW
« no previous file with comments | « remoting/webapp/base/js/base.js ('k') | remoting/webapp/base/js/message_window.js » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698