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

Side by Side Diff: chrome/browser/resources/settings/printing_page/cups_printers.js

Issue 2380753004: [CUPS] Implement the Webui handler for the printers auto discovery. (Closed)
Patch Set: Address michaelpg@'s comments. Created 4 years, 2 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 unified diff | Download patch
OLDNEW
1 // Copyright 2016 The Chromium Authors. All rights reserved. 1 // Copyright 2016 The Chromium Authors. All rights reserved.
2 // Use of this source code is governed by a BSD-style license that can be 2 // Use of this source code is governed by a BSD-style license that can be
3 // found in the LICENSE file. 3 // found in the LICENSE file.
4 4
5 /** 5 /**
6 * @fileoverview 'settings-cups-printers' is a component for showing CUPS 6 * @fileoverview 'settings-cups-printers' is a component for showing CUPS
7 * Printer settings subpage (chrome://md-settings/cupsPrinters). It is used to 7 * Printer settings subpage (chrome://md-settings/cupsPrinters). It is used to
8 * set up legacy & non-CloudPrint printers on ChromeOS by leveraging CUPS (the 8 * set up legacy & non-CloudPrint printers on ChromeOS by leveraging CUPS (the
9 * unix printing system) and the many open source drivers built for CUPS. 9 * unix printing system) and the many open source drivers built for CUPS.
10 */ 10 */
(...skipping 20 matching lines...) Expand all
31 this.updateCupsPrintersList_(); 31 this.updateCupsPrintersList_();
32 this.addWebUIListener('on-add-cups-printer', this.onAddPrinter_.bind(this)); 32 this.addWebUIListener('on-add-cups-printer', this.onAddPrinter_.bind(this));
33 }, 33 },
34 34
35 /** 35 /**
36 * @param {boolean} success 36 * @param {boolean} success
37 * @param {string} printerName 37 * @param {string} printerName
38 * @private 38 * @private
39 */ 39 */
40 onAddPrinter_: function(success, printerName) { 40 onAddPrinter_: function(success, printerName) {
41 if (!success) 41 if (success) {
42 return; 42 this.updateCupsPrintersList_();
43 43 var message = this.$.addPrinterDoneMessage;
44 this.updateCupsPrintersList_(); 44 message.textContent = loadTimeData.getStringF(
45 var message = this.$.addPrinterMessage; 45 'printerAddedSuccessfulMessage', printerName);
46 message.textContent = loadTimeData.getStringF( 46 } else {
47 'printerAddedSuccessfulMessage', printerName); 47 var message = this.$.addPrinterErrorMessage;
48 }
48 message.hidden = false; 49 message.hidden = false;
49 window.setTimeout(function() { 50 window.setTimeout(function() {
50 message.hidden = true; 51 message.hidden = true;
51 }, 3000); 52 }, 3000);
52 }, 53 },
53 54
54 /** @private */ 55 /** @private */
55 updateCupsPrintersList_: function() { 56 updateCupsPrintersList_: function() {
56 settings.CupsPrintersBrowserProxyImpl.getInstance(). 57 settings.CupsPrintersBrowserProxyImpl.getInstance().
57 getCupsPrintersList().then(this.printersChanged_.bind(this)); 58 getCupsPrintersList().then(this.printersChanged_.bind(this));
58 }, 59 },
59 60
60 /** 61 /**
61 * @param {!CupsPrintersList} cupsPrintersList 62 * @param {!CupsPrintersList} cupsPrintersList
62 * @private 63 * @private
63 */ 64 */
64 printersChanged_: function(cupsPrintersList) { 65 printersChanged_: function(cupsPrintersList) {
65 this.printers = cupsPrintersList.printerList; 66 this.printers = cupsPrintersList.printerList;
66 }, 67 },
67 68
68 /** @private */ 69 /** @private */
69 onAddPrinterTap_: function() { 70 onAddPrinterTap_: function() {
70 this.$.addPrinterDialog.open(); 71 this.$.addPrinterDialog.open();
72 this.$.addPrinterErrorMessage.hidden = true;
71 }, 73 },
72 }); 74 });
OLDNEW

Powered by Google App Engine
This is Rietveld 408576698