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

Unified Diff: reference_extension/client.js

Issue 6176006: Removing isInstalledComplete (not needed). (Closed) Base URL: http://git.chromium.org/git/entd.git@master
Patch Set: Created 9 years, 11 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 side-by-side diff with in-line comments
Download patch
« no previous file with comments | « no previous file | no next file » | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: reference_extension/client.js
diff --git a/reference_extension/client.js b/reference_extension/client.js
index 9d1eec26cb5d545b2c8770873f47759a48100fec..bab4f056e9b1c423615ba608dc696d9e6d778cfa 100644
--- a/reference_extension/client.js
+++ b/reference_extension/client.js
@@ -277,7 +277,6 @@ function loadCertificates() {
client.installCert =
function installCert(cert, variables) {
var key = cert.key;
- var isInstallComplete = false;
// Called for any kind of error from the enterprise daemon.
function onerror(retval) {
@@ -325,11 +324,8 @@ function installCert(cert, variables) {
onerror);
} else if (cert.state == 'stop:ready') {
$('#cert-status').find('.cert-install').attr('status', 'green');
- if (!isInstallComplete) {
- isInstallComplete = true;
- client.showSuccess('Your certificate has been installed',
- 'Certificate Installed');
- }
+ client.showSuccess('Your certificate has been installed',
+ 'Certificate Installed');
return;
} else if (cert.state == 'stop:error') {
client.showCertDetails(cert);
« no previous file with comments | « no previous file | no next file » | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698