Settings People Revamp: Simplify Change Picture browsertests.

chrome/browser/resources/settings/people_page/change_picture.js

 // Copyright 2015 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.exportPath('settings_test');
 
 /** @type {boolean} */
 settings_test.changePictureNotifyForTest;
 
 /**
@@ skipping 26 matching lines @@
      * True if the user has a plugged-in webcam.
      * @private {boolean}
      */
     cameraPresent_: {
       type: Boolean,
       value: false,
     },
 
     /**
      * The currently selected item. This property is bound to the iron-selector
-     * and never directly assigned.
-     * @private {settings.ChangePictureImageElement}
+     * and never directly assigned. This may be undefined momentarily as
+     * the selection changes due to iron-selector implementation details.
+     * @private {?settings.ChangePictureImageElement}
      */
-    selectedItem_: {
-      type: settings.ChangePictureImageElement,
-      notify: !!settings_test.changePictureNotifyForTest,
-    },
+    selectedItem_: Object,
 
     /**
      * The url of the 'old' image, which is the existing image sourced from
      * the camera, a file, or a deprecated default image. It defaults to an
      * empty string instead of undefined, because Polymer bindings don't behave
      * as expected with undefined properties.
      * @private {string}
      */
     oldImageUrl_: {
       type: String,
@@ skipping 16 matching lines @@
     defaultImages_: {
       type: Array,
       value: function() { return []; },
     },
 
     /**
      * The fallback image to be selected when the user discards the 'old' image.
      * This may be null if the user started with the 'old' image.
      * @private {?settings.ChangePictureImageElement}
      */
-    fallbackImage_: {
-      type: settings.ChangePictureImageElement,
-      value: null,
-    },
+    fallbackImage_: Object,
 
     /**
      * Type of the last selected icon. This is used to jump back to the camera
      * after the user discards a newly taken photo.
      * @private {string}
      */
     lastSelectedImageType_: {
       type: String,
       value: '',
     },
@@ skipping 211 matching lines @@
     if (this.fallbackImage_ != null) {
       this.selectImage_(this.fallbackImage_);
       return;
     }
 
     // If the user has not chosen an image since opening the subpage and
     // discards the current photo, select the first default image.
     assert(this.defaultImages_.length > 0);
     this.browserProxy_.selectDefaultImage(this.defaultImages_[0].url);
 
-    announceAccessibleMessage(
-        loadTimeData.getString('photoDiscardAccessibleText'));
+    announceAccessibleMessage(this.i18n('photoDiscardAccessibleText'));
   },
 
   /**
    * @param {string} oldImageUrl
    * @return {boolean} True if there is no old image and the old image icon
    *     should be hidden.
    * @private
    */
   isOldImageHidden_: function(oldImageUrl) { return oldImageUrl.length == 0; },
 
@@ skipping 63 matching lines @@
    */
   getAuthorWebsite_: function(selectedItem, defaultImages) {
     if (!this.isAuthorCreditShown_(selectedItem))
       return '';
 
     assert(selectedItem.dataset.defaultImageIndex !== null &&
            selectedItem.dataset.defaultImageIndex < defaultImages.length);
     return defaultImages[selectedItem.dataset.defaultImageIndex].website;
   },
 });