MD Settings: Change picture: Focus container

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.
 
 /**
  * Contains the possible types of Change Picture selections.
  * @enum {string}
  */
 var ChangePictureSelectionTypes = {
   CAMERA: 'camera',
@@ skipping 127 matching lines @@
 
   /** @protected */
   currentRouteChanged: function(newRoute) {
     if (newRoute == settings.Route.CHANGE_PICTURE) {
       this.browserProxy_.initialize();
 
       // This in needed because we manually clear the selectedItem_ property
       // when navigating away. The selector element doesn't fire its upward
       // data binding unless its selected item has changed.
       this.selectedItem_ = this.$.selector.selectedItem;
+      this.$.container.focus();

[tommycli, 2017/03/03 23:26:51]

I think this is a solid solution, but I think it's worth adding a comment as to
why we need to focus the container. (arrow keys)

[stevenjb, 2017/03/03 23:34:01]

Good call. Done.

     } else {
       // Ensure we deactivate the camera when we navigate away.
       this.selectedItem_ = null;
     }
   },
 
   /**
    * Handler for the 'default-images-changed' event.
    * @param {!Array<!settings.DefaultImage>} images
    * @private
@@ skipping 286 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;
   },
 });