MD Settings: Certificate manager, hooking up all dialogs.

chrome/browser/resources/settings/certificate_manager_page/certificate_delete_confirmation_dialog.js

 // Copyright 2016 The Chromium Authors. All rights reserved.
 // Use of this source code is governed by a BSD-style license that can be
 // found in the LICENSE file.
 
 /**
  * @fileoverview A confirmation dialog allowing the user to delete various types
  * of certificates.
  */
 Polymer({
   is: 'settings-certificate-delete-confirmation-dialog',
@@ skipping 68 matching lines @@
   onCancelTap_: function() {
     this.$.dialog.close();
   },
 
   /** @private */
   onOkTap_: function() {
     this.browserProxy_.deleteCertificate(this.model.id).then(function() {
       this.$.dialog.close();
     }.bind(this),
     /** @param {!CertificatesError} error */
     function(error) {

[tommycli, 2016/03/23 19:28:04]

Nit: Same comment here, the indentation confused me.

[dpapad, 2016/03/24 22:06:10]

See previous response to similar comment.

-      // TODO(dpapad): Display error here.
-    });
+      this.$.dialog.close();
+      this.dispatchEvent(
+          new CustomEvent('certificates-error', {detail: error}));
+    }.bind(this));
   },
 });