print preview css: fix labels

chrome/browser/resources/print_preview/copies_settings.js

 // Copyright (c) 2011 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.
 
 cr.define('print_preview', function() {
   'use strict';
 
   /**
    * Creates a CopiesSettings object.
    * @constructor
    */
   function CopiesSettings() {
     this.copiesOption_ = $('copies-option');
     this.textfield_ = $('copies');
     this.incrementButton_ = $('increment');
     this.decrementButton_ = $('decrement');
     // Minimum allowed value for number of copies.
     this.minValue_ = 1;
     // Maximum allowed value for number of copies.
     this.maxValue_ = 999;
     this.collateOption_ = $('collate-option');
     this.collateCheckbox_ = $('collate');
     this.hint_ = $('copies-hint');
     this.twoSidedCheckbox_ = $('two-sided');
-    this.twoSidedOption_ = $('two-sided-div');
+    this.twoSidedOption_ = $('two-sided-option');
 
     // Constant values matches printing::DuplexMode enum. Not using const
     // keyword because it is not allowed by JS strict mode.
     this.SIMPLEX = 0;
     this.LONG_EDGE = 1;
     this.UNKNOWN_DUPLEX_MODE = -1;
     this.addEventListeners_();
   }
 
   cr.addSingletonGetter(CopiesSettings);
@@ skipping 147 matching lines @@
     onDecrementButtonClicked_: function() {
       this.onButtonClicked_(-1);
     },
 
     /**
      * Takes care of showing/hiding the collate option.
      * @private
      */
     showHideCollateOption_: function() {
       this.collateOption_.hidden = this.numberOfCopies <= 1;
-      // TODO(aayushkumar): Remove aria-hidden attribute once elements within
-      // the hidden attribute are no longer read out by a screen-reader.
-      // (Currently a bug in webkit).
-      this.collateOption_.setAttribute('aria-hidden',
-                                       this.collateOption_.hidden);
     },
 
     /*
      * Takes care of showing/hiding the two sided option and also updates the
      * default state of the checkbox.
      * @param {number} defaultDuplexValue Specifies the default duplex value.
      * @private
      */
      updateTwoSidedOption_: function(defaultDuplexValue) {
       // On Windows, some printers don't specify their duplex values in the
       // printer schema. If the printer duplex value is UNKNOWN_DUPLEX_MODE,
       // hide the two sided option in preview tab UI.
       // Ref bug: http://crbug.com/89204
       this.twoSidedOption_.hidden =
           (defaultDuplexValue == this.UNKNOWN_DUPLEX_MODE);
-      this.twoSidedOption_.setAttribute('aria-hidden',
-                                        this.twoSidedOption_.hidden);
+
       if (!this.twoSidedOption_.hidden)
         this.twoSidedCheckbox_.checked = !!defaultDuplexValue;
      },
 
     /**
      * Updates the state of the increment/decrement buttons based on the current
      * |textfield_| value.
      * @private
      */
     updateButtonsState_: function() {
       if (!this.isValid()) {
         this.textfield_.classList.add('invalid');
         this.incrementButton_.disabled = false;
         this.decrementButton_.disabled = false;
         fadeInElement(this.hint_);
       } else {
         this.textfield_.classList.remove('invalid');
         this.incrementButton_.disabled = this.numberOfCopies == this.maxValue_;
         this.decrementButton_.disabled = this.numberOfCopies == this.minValue_;
         fadeOutElement(this.hint_);
       }
       this.hint_.setAttribute('aria-hidden', this.isValid());
     }
   };
 
   return {
     CopiesSettings: CopiesSettings,
   };
 });