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

Unified Diff: chrome/browser/resources/print_preview/settings/other_options_settings.js

Issue 294923005: Add media size capability to PDF printer. (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src
Patch Set: Add a comment to the new query stopping code. Created 6 years, 7 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/print_preview/settings/other_options_settings.js
diff --git a/chrome/browser/resources/print_preview/settings/other_options_settings.js b/chrome/browser/resources/print_preview/settings/other_options_settings.js
index ac9f0d14527748eb66b5a4c54ecb1c1db7c3e300..4b935395990d04ba8af5fc8527b36eccf39a317d 100644
--- a/chrome/browser/resources/print_preview/settings/other_options_settings.js
+++ b/chrome/browser/resources/print_preview/settings/other_options_settings.js
@@ -235,6 +235,16 @@ cr.define('print_preview', function() {
this.duplexTicketItem_.isCapabilityAvailable() ||
this.cssBackgroundTicketItem_.isCapabilityAvailable() ||
this.selectionOnlyTicketItem_.isCapabilityAvailable()) {
+ setIsVisible(this.headerFooterContainer_,
+ this.headerFooterTicketItem_.isCapabilityAvailable());
+ setIsVisible(this.fitToPageContainer_,
+ this.fitToPageTicketItem_.isCapabilityAvailable());
+ setIsVisible(this.duplexContainer_,
+ this.duplexTicketItem_.isCapabilityAvailable());
+ setIsVisible(this.cssBackgroundContainer_,
+ this.cssBackgroundTicketItem_.isCapabilityAvailable());
+ setIsVisible(this.selectionOnlyContainer_,
+ this.selectionOnlyTicketItem_.isCapabilityAvailable());
fadeInOption(this.getElement());
} else {
fadeOutOption(this.getElement());
@@ -294,8 +304,6 @@ cr.define('print_preview', function() {
* @private
*/
onDuplexChange_: function() {
- setIsVisible(this.duplexContainer_,
- this.duplexTicketItem_.isCapabilityAvailable());
this.duplexCheckbox_.checked = this.duplexTicketItem_.getValue();
this.updateContainerState_();
},
@@ -306,8 +314,6 @@ cr.define('print_preview', function() {
* @private
*/
onFitToPageChange_: function() {
- setIsVisible(this.fitToPageContainer_,
- this.fitToPageTicketItem_.isCapabilityAvailable());
this.fitToPageCheckbox_.checked = this.fitToPageTicketItem_.getValue();
this.updateContainerState_();
},
@@ -318,8 +324,6 @@ cr.define('print_preview', function() {
* @private
*/
onCssBackgroundChange_: function() {
- setIsVisible(this.cssBackgroundContainer_,
- this.cssBackgroundTicketItem_.isCapabilityAvailable());
this.cssBackgroundCheckbox_.checked =
this.cssBackgroundTicketItem_.getValue();
this.updateContainerState_();
@@ -331,8 +335,6 @@ cr.define('print_preview', function() {
* @private
*/
onSelectionOnlyChange_: function() {
- setIsVisible(this.selectionOnlyContainer_,
- this.selectionOnlyTicketItem_.isCapabilityAvailable());
this.selectionOnlyCheckbox_.checked =
this.selectionOnlyTicketItem_.getValue();
this.updateContainerState_();
@@ -344,8 +346,6 @@ cr.define('print_preview', function() {
* @private
*/
onHeaderFooterChange_: function() {
- setIsVisible(this.headerFooterContainer_,
- this.headerFooterTicketItem_.isCapabilityAvailable());
this.headerFooterCheckbox_.checked =
this.headerFooterTicketItem_.getValue();
this.updateContainerState_();

Powered by Google App Engine
This is Rietveld 408576698