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

Unified Diff: chrome/browser/resources/mobile_setup.js

Issue 5072001: Improved activation logic to cover few more edge cases. Removed button that w... (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src/
Patch Set: '' Created 10 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 side-by-side diff with in-line comments
Download patch
« no previous file with comments | « chrome/browser/resources/mobile_setup.html ('k') | no next file » | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: chrome/browser/resources/mobile_setup.js
===================================================================
--- chrome/browser/resources/mobile_setup.js (revision 66030)
+++ chrome/browser/resources/mobile_setup.js (working copy)
@@ -46,14 +46,6 @@
window.addEventListener('message', function(e) {
self.onMessageReceived_(e);
});
- $('cheat').addEventListener('click', function(e) {
- console.log('calling setTransactionStatus from cheat.onclick');
- if (self.faked_transaction_)
- return;
- $('paymentForm').classList.add('hidden');
- chrome.send('setTransactionStatus', ['OK']);
- self.faked_transaction_ = true;
- });
$(frame_name).addEventListener('load', function(e) {
// Flip the visibility of the payment page only after the frame is
// fully loaded.
@@ -62,6 +54,7 @@
$('finalMessage').classList.add('hidden');
$('errorMessage').classList.add('hidden');
$('systemStatus').classList.add('hidden');
+ $('canvas').classList.add('hidden');
$('paymentForm').classList.remove('hidden');
}
});
@@ -104,6 +97,7 @@
switch(new_state) {
case MobileSetup.PLAN_ACTIVATION_LOADING:
case MobileSetup.PLAN_ACTIVATION_START:
+ case MobileSetup.PLAN_ACTIVATION_RECONNECTING:
$('statusHeader').textContent =
MobileSetup.localStrings_.getString('connecting_header');
$('errorMessage').textContent = '';
@@ -111,9 +105,9 @@
$('finalMessage').classList.add('hidden');
$('errorMessage').classList.add('hidden');
$('systemStatus').classList.remove('hidden');
+ $('canvas').classList.remove('hidden');
break;
case MobileSetup.PLAN_ACTIVATION_INITIATING_ACTIVATION:
- case MobileSetup.PLAN_ACTIVATION_RECONNECTING:
$('statusHeader').textContent =
MobileSetup.localStrings_.getString('activating_header');
$('errorMessage').textContent = '';
@@ -121,18 +115,21 @@
$('finalMessage').classList.add('hidden');
$('errorMessage').classList.add('hidden');
$('systemStatus').classList.remove('hidden');
+ $('canvas').classList.remove('hidden');
break;
case MobileSetup.PLAN_ACTIVATION_SHOWING_PAYMENT:
$('paymentForm').classList.add('hidden');
$('finalMessage').classList.add('hidden');
$('errorMessage').classList.add('hidden');
$('systemStatus').classList.remove('hidden');
+ $('canvas').classList.remove('hidden');
this.loadPaymentFrame_(deviceInfo);
break;
case MobileSetup.PLAN_ACTIVATION_DONE:
$('statusHeader').textContent = '';
$('errorMessage').classList.add('hidden');
$('systemStatus').classList.add('hidden');
+ $('canvas').classList.add('hidden');
$('paymentForm').classList.remove('hidden');
$('finalMessage').classList.remove('hidden');
break;
@@ -142,6 +139,7 @@
$('errorMessage').textContent = deviceInfo.error;
$('paymentForm').classList.add('hidden');
$('finalMessage').classList.add('hidden');
+ $('canvas').classList.add('hidden');
$('errorMessage').classList.remove('hidden');
$('systemStatus').classList.remove('hidden');
break;
« no previous file with comments | « chrome/browser/resources/mobile_setup.html ('k') | no next file » | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698