athena: A simpler implementation of WindowListProvider.

athena/wm/window_list_provider_impl.cc

+// Copyright 2014 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 "athena/wm/window_list_provider_impl.h"
+
+#include "ui/aura/window.h"
+
+namespace athena {
+
+WindowListProviderImpl::WindowListProviderImpl(aura::Window* container)
+    : container_(container) {
+  CHECK(container_);
+}
+
+WindowListProviderImpl::~WindowListProviderImpl() {
+}
+
+aura::Window::Windows WindowListProviderImpl::GetWindowList() const {
+  aura::Window::Windows list;
+  const aura::Window::Windows& container_children = container_->children();
+  for (aura::Window::Windows::const_iterator iter = container_children.begin();
+       iter != container_children.end();
+       ++iter) {
+    if ((*iter)->type() == ui::wm::WINDOW_TYPE_NORMAL)
+      list.push_back(*iter);
+  }
+  return list;
+}
+
+void WindowListProviderImpl::MoveToFront(aura::Window* window) {
+  CHECK_EQ(container_, window->parent());
+  container_->StackChildAtTop(window);

[oshima, 2014/08/14 22:07:05]

window moves to top when activation changes. Would it be possible to eliminate
some of MoveToFront in SplitViewController where makes sense?

[mfomitchev, 2014/08/14 22:19:08]

In practice I think we could get rid of all of those that exist now in TOT - the
window that you are dragging from the bezel gets the focus and thus becomes
activated. That said, if it's a no-op, is there harm in doing explicitly rather
than relying on focus changes to activate the appropriate windows? Or were you
suggesting to use wm::ActivateWindow instead of  MoveToFront()?

[sadrul, 2014/08/14 23:47:22]

Removed MoveToFront() from the interface

+}
+
+}  // namespace athena