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

Side by Side Diff: remoting/webapp/crd/js/host_settings.js

Issue 687873003: Allow the background page to get an OAuth token for apps v1. (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: Move jsonParseSafe into base namespace. Created 6 years, 1 month 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/crd/js/host_list.js ('k') | remoting/webapp/crd/js/it2me_helpee_channel.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 (c) 2012 The Chromium Authors. All rights reserved. 1 // Copyright (c) 2012 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 * Class handling saving and restoring of per-host options. 7 * Class handling saving and restoring of per-host options.
8 */ 8 */
9 9
10 'use strict'; 10 'use strict';
(...skipping 65 matching lines...) Expand 10 before | Expand all | Expand 10 after
76 remoting.HostSettings.loadInternal_ = function(hostId, callback) { 76 remoting.HostSettings.loadInternal_ = function(hostId, callback) {
77 /** 77 /**
78 * @param {Object.<string>} allHosts The current options for all hosts. 78 * @param {Object.<string>} allHosts The current options for all hosts.
79 * @return {void} Nothing. 79 * @return {void} Nothing.
80 */ 80 */
81 var onDone = function(allHosts) { 81 var onDone = function(allHosts) {
82 var result = {}; 82 var result = {};
83 try { 83 try {
84 var hosts = allHosts[remoting.HostSettings.KEY_]; 84 var hosts = allHosts[remoting.HostSettings.KEY_];
85 if (hosts) { 85 if (hosts) {
86 result = jsonParseSafe(hosts); 86 result = base.jsonParseSafe(hosts);
87 if (typeof(result) != 'object') { 87 if (typeof(result) != 'object') {
88 console.error("Error loading host settings: Not an object"); 88 console.error("Error loading host settings: Not an object");
89 result = {}; 89 result = {};
90 } else if (/** @type {Object} */ (result).hasOwnProperty(hostId) && 90 } else if (/** @type {Object} */ (result).hasOwnProperty(hostId) &&
91 typeof(result[hostId]) == 'object') { 91 typeof(result[hostId]) == 'object') {
92 callback(result[hostId], result); 92 callback(result[hostId], result);
93 return; 93 return;
94 } 94 }
95 } 95 }
96 } catch (err) { 96 } catch (err) {
97 var typedErr = /** @type {*} */ (err); 97 var typedErr = /** @type {*} */ (err);
98 console.error('Error loading host settings:', typedErr); 98 console.error('Error loading host settings:', typedErr);
99 } 99 }
100 callback({}, /** @type {Object} */ (result)); 100 callback({}, /** @type {Object} */ (result));
101 }; 101 };
102 chrome.storage.local.get(remoting.HostSettings.KEY_, onDone); 102 chrome.storage.local.get(remoting.HostSettings.KEY_, onDone);
103 }; 103 };
104 104
105 /** @type {string} @private */ 105 /** @type {string} @private */
106 remoting.HostSettings.KEY_ = 'remoting-host-options'; 106 remoting.HostSettings.KEY_ = 'remoting-host-options';
OLDNEW
« no previous file with comments | « remoting/webapp/crd/js/host_list.js ('k') | remoting/webapp/crd/js/it2me_helpee_channel.js » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698