A window should not get activated or get input focus if it's behind the lock screen.

ui/views/widget/widget.cc

 // Copyright (c) 2012 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/widget/widget.h"
 
 #include "base/logging.h"
 #include "base/macros.h"
 #include "base/message_loop/message_loop.h"
 #include "base/strings/utf_string_conversions.h"
@@ skipping 1299 matching lines @@
 const Widget* Widget::AsWidget() const {
   return this;
 }
 
 bool Widget::SetInitialFocus(ui::WindowShowState show_state) {
   View* v = widget_delegate_->GetInitiallyFocusedView();
   if (!focus_on_creation_ || show_state == ui::SHOW_STATE_INACTIVE ||
       show_state == ui::SHOW_STATE_MINIMIZED) {
     // If not focusing the window now, tell the focus manager which view to
     // focus when the window is restored.
-    if (v)
+    if (v && focus_manager_)

[xdai1, 2016/03/08 18:30:17]

focus_manager_ is null for non top level windows.

[sky, 2016/03/08 22:42:04]

A null focus_manager_ wasn't possible before, so it shouldn't be possible now. I
suspect your test needs to be updated to ensure a focus_manager_ is created
correctly.

[xdai1, 2016/03/10 01:12:57]

According to the comment in
https://code.google.com/p/chromium/codesearch#chromium/src/ui/views/widget/wi...,
a non top level widget doesn't have a focus manager. So I guess before this
change, it's impossible for a non top level widget to show with
SHOW_STATE_INACTIVE, but after this change it's possible now?

Actually a null focus_manager_ is the main reason of the builder failure in
Patch 7 (for example, more than 100 tests failed in
linux_chromium_chromeos_rel_ng because of a null focus_manager_:
https://build.chromium.org/p/tryserver.chromium.linux/builders/linux_chromium...).

       focus_manager_->SetStoredFocusView(v);
     return true;
   }
   if (v)
     v->RequestFocus();
   return !!v;
 }
 
 ////////////////////////////////////////////////////////////////////////////////
 // Widget, ui::EventSource implementation:
@@ skipping 165 matching lines @@
 
 ////////////////////////////////////////////////////////////////////////////////
 // internal::NativeWidgetPrivate, NativeWidget implementation:
 
 internal::NativeWidgetPrivate* NativeWidgetPrivate::AsNativeWidgetPrivate() {
   return this;
 }
 
 }  // namespace internal
 }  // namespace views