Send a correctly-formatted response if there are no host options saved.

Send a correctly-formatted response if there are no host options saved.

Previously, the response would be simply {}, which is interpreted as an
error by the import code.

Review-Url: https://codereview.chromium.org/2689593005
Cr-Commit-Position: refs/heads/master@{#450246}
Committed: https://chromium.googlesource.com/chromium/src/+/f143318ec3d651b2cdee9a5edb7b011afbdb1510

[Jamie, 2017-02-10 18:57:36]

jamiewalch@chromium.org changed reviewers:
+ kelvinp@chromium.org

[Jamie, 2017-02-10 18:57:37]

PTAL

[kelvinp, 2017-02-13 19:58:19]

lgtm

https://codereview.chromium.org/2689593005/diff/1/remoting/webapp/crd/js/opti...
File remoting/webapp/crd/js/options_export.js (right):

https://codereview.chromium.org/2689593005/diff/1/remoting/webapp/crd/js/opti...
remoting/webapp/crd/js/options_export.js:23:
chrome.storage.local.get('remoting-host-options', function(options) {
put 'remoting-host-options' as a variable?

[Jamie, 2017-02-14 02:48:01]

The CQ bit was checked by jamiewalch@chromium.org

[Jamie, 2017-02-14 02:48:01]

The patchset sent to the CQ was uploaded after l-g-t-m from kelvinp@chromium.org
Link to the patchset: https://codereview.chromium.org/2689593005/#ps20001
(title: "Reviewer feedback.")

[Jamie, 2017-02-14 02:48:17]

FYI

https://codereview.chromium.org/2689593005/diff/1/remoting/webapp/crd/js/opti...
File remoting/webapp/crd/js/options_export.js (right):

https://codereview.chromium.org/2689593005/diff/1/remoting/webapp/crd/js/opti...
remoting/webapp/crd/js/options_export.js:23:
chrome.storage.local.get('remoting-host-options', function(options) {
On 2017/02/13 19:58:19, kelvinp wrote:
> put 'remoting-host-options' as a variable?

Done.

[commit-bot: I haz the power, 2017-02-14 02:49:11]

CQ is trying da patch. Follow status at
 
https://chromium-cq-status.appspot.com/v2/patch-status/codereview.chromium.or...

[commit-bot: I haz the power, 2017-02-14 04:34:02]

CQ is committing da patch.

Bot data: {"patchset_id": 20001, "attempt_start_ts": 1487040481279880,
"parent_rev": "f435bbbcaa33ff17a3da0704284f0e6ca4130c57", "commit_rev":
"f143318ec3d651b2cdee9a5edb7b011afbdb1510"}

[commit-bot: I haz the power, 2017-02-14 04:34:41]

Description was changed from

==========
Send a correctly-formatted response if there are no host options saved.

Previously, the response would be simply {}, which is interpreted as an
error by the import code.
==========

to

==========
Send a correctly-formatted response if there are no host options saved.

Previously, the response would be simply {}, which is interpreted as an
error by the import code.

Review-Url: https://codereview.chromium.org/2689593005
Cr-Commit-Position: refs/heads/master@{#450246}
Committed:
https://chromium.googlesource.com/chromium/src/+/f143318ec3d651b2cdee9a5edb7b...
==========

[commit-bot: I haz the power, 2017-02-14 04:34:43]

Committed patchset #2 (id:20001) as
https://chromium.googlesource.com/chromium/src/+/f143318ec3d651b2cdee9a5edb7b...