If upload is enabled don't fallback to local save.

If upload is enabled don't fallback to local save.

We believe that sometimes offering upload and sometimes offering local save is a confusing user experience. If CVC or address isn't available this time it'll be available eventually. It only has to succeed once.

The exception is if the upload details RPC fails. I'm still doing fallback here for the reasons explained in the comment there.

BUG=535784
Committed: https://crrev.com/6201c91cf4689c2889b38f87c09269054c9bc886
Cr-Commit-Position: refs/heads/master@{#364469}

[Justin Donnelly, 2015-12-10 19:28:54]

jdonnelly@chromium.org changed reviewers:
+ estade@chromium.org

[Evan Stade, 2015-12-10 19:37:43]

lgtm

[Justin Donnelly, 2015-12-10 19:57:20]

The CQ bit was checked by jdonnelly@chromium.org

[commit-bot: I haz the power, 2015-12-10 19:59:17]

CQ is trying da patch. Follow status at
 https://chromium-cq-status.appspot.com/patch-status/1515903003/1
View timeline at
 https://chromium-cq-status.appspot.com/patch-timeline/1515903003/1

[commit-bot: I haz the power, 2015-12-10 20:50:21]

Committed patchset #1 (id:1)

[commit-bot: I haz the power, 2015-12-10 20:51:09]

Description was changed from

==========
If upload is enabled don't fallback to local save.

We believe that sometimes offering upload and sometimes offering local save is a
confusing user experience. If CVC or address isn't available this time it'll be
available eventually. It only has to succeed once.

The exception is if the upload details RPC fails. I'm still doing fallback here
for the reasons explained in the comment there.

BUG=535784
==========

to

==========
If upload is enabled don't fallback to local save.

We believe that sometimes offering upload and sometimes offering local save is a
confusing user experience. If CVC or address isn't available this time it'll be
available eventually. It only has to succeed once.

The exception is if the upload details RPC fails. I'm still doing fallback here
for the reasons explained in the comment there.

BUG=535784

Committed: https://crrev.com/6201c91cf4689c2889b38f87c09269054c9bc886
Cr-Commit-Position: refs/heads/master@{#364469}
==========

[commit-bot: I haz the power, 2015-12-10 20:51:10]

Patchset 1 (id:??) landed as
https://crrev.com/6201c91cf4689c2889b38f87c09269054c9bc886
Cr-Commit-Position: refs/heads/master@{#364469}