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

Unified Diff: chrome/browser/resources/chromeos/emulator/bluetooth_settings.js

Issue 2958193002: Device emulator: Replace usage of paper-dialog with cr-dialog. (Closed)
Patch Set: Created 3 years, 6 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
Index: chrome/browser/resources/chromeos/emulator/bluetooth_settings.js
diff --git a/chrome/browser/resources/chromeos/emulator/bluetooth_settings.js b/chrome/browser/resources/chromeos/emulator/bluetooth_settings.js
index aa04e9e34a3f8722ab8bad3611e08447af7a522a..e4d7294c9b0167ffe9bdedf71fa1cbd1360a3bfe 100644
--- a/chrome/browser/resources/chromeos/emulator/bluetooth_settings.js
+++ b/chrome/browser/resources/chromeos/emulator/bluetooth_settings.js
@@ -173,15 +173,6 @@ Polymer({
}
},
- /**
- * Called when the device edit dialog is opened. Re-validates necessary input
- * fields.
- */
- editDialogOpened: function() {
- this.validateAddress();
- this.validatePath();
- },
-
handleAddressInput: function() {
this.autoFormatAddress();
this.validateAddress();
@@ -480,7 +471,14 @@ Polymer({
var index = event.model.index;
this.currentEditIndex = index;
this.currentEditableObject = this.devices[index];
- this.$.editDialog.toggle();
+ this.$.editDialog.showModal();
+ this.validateAddress();
+ this.validatePath();
+ },
+
+ /** @private */
+ onCloseTap_: function() {
+ this.$.editDialog.close();
},
/**

Powered by Google App Engine
This is Rietveld 408576698