(Mac)Views: Widgets focus first View in traversal order if initial focus fails.

ui/views/controls/focus_ring.cc

 // Copyright 2016 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.
 
 #include "ui/views/controls/focus_ring.h"
 
 #include "ui/gfx/canvas.h"
 #include "ui/native_theme/native_theme.h"
 #include "ui/views/controls/focusable_border.h"
 
@@ skipping 18 matching lines @@
   return nullptr;
 }
 
 }  // namespace
 
 const char FocusRing::kViewClassName[] = "FocusRing";
 
 // static
 views::View* FocusRing::Install(views::View* parent,
                                 ui::NativeTheme::ColorId override_color_id) {
-  DCHECK(parent->HasFocus());

[karandeepb, 2016/12/30 21:16:04]

If it's ok, can you send a separate CL for this, since this is unrelated? 

[Patti Lor, 2017/01/03 03:48:26]

No worries, see https://codereview.chromium.org/2612463002/. I'll wait til that
one lands before landing this one.

   FocusRing* ring = GetFocusRing(parent);
   if (!ring) {
     ring = new FocusRing();
     parent->AddChildView(ring);
   }
   ring->override_color_id_ = override_color_id;
   ring->Layout();
   ring->SchedulePaint();
   return ring;
 }
@@ skipping 38 matching lines @@
 FocusRing::FocusRing()
     : override_color_id_(ui::NativeTheme::kColorId_NumColors) {
   // A layer is necessary to paint beyond the parent's bounds.
   SetPaintToLayer(true);
   layer()->SetFillsBoundsOpaquely(false);
 }
 
 FocusRing::~FocusRing() {}
 
 }  // namespace views