Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(199)

Unified Diff: ui/aura/test/test_activation_client.cc

Issue 9689047: Added notion of currently active app / browser (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src
Patch Set: Remove default args from State Created 8 years, 9 months ago
Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.
Jump to:
View side-by-side diff with in-line comments
Download patch
« content/test/test_renderer_host.h ('K') | « ui/aura/test/test_activation_client.h ('k') | no next file » | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: ui/aura/test/test_activation_client.cc
diff --git a/ui/aura/test/test_activation_client.cc b/ui/aura/test/test_activation_client.cc
index 7ba0d41e0f4f8134fef95060fed62db274acad2d..74810ccbe1527c02bcf5f45cd23b065efe73f0bf 100644
--- a/ui/aura/test/test_activation_client.cc
+++ b/ui/aura/test/test_activation_client.cc
@@ -4,6 +4,7 @@
#include "ui/aura/test/test_activation_client.h"
+#include "ui/aura/client/activation_delegate.h"
#include "ui/aura/root_window.h"
#include "ui/aura/window.h"
@@ -13,8 +14,7 @@ namespace test {
////////////////////////////////////////////////////////////////////////////////
// TestActivationClient, public:
-TestActivationClient::TestActivationClient(RootWindow* root_window)
- : active_window_(NULL) {
+TestActivationClient::TestActivationClient(RootWindow* root_window) {
client::SetActivationClient(root_window, this);
}
@@ -25,22 +25,34 @@ TestActivationClient::~TestActivationClient() {
// TestActivationClient, client::ActivationClient implementation:
void TestActivationClient::ActivateWindow(Window* window) {
- if (active_window_)
- active_window_->RemoveObserver(this);
- active_window_ = window;
- active_window_->AddObserver(this);
+ Window *last_active = GetActiveWindow();
+ if (last_active == window)
+ return;
+
+ RemoveActiveWindow(window);
+ active_windows_.push_back(window);
+ window->AddObserver(this);
+ if (aura::client::GetActivationDelegate(window))
+ aura::client::GetActivationDelegate(window)->OnActivated();
+
+ if (last_active) {
+ if (aura::client::GetActivationDelegate(last_active))
+ aura::client::GetActivationDelegate(last_active)->OnLostActive();
+ }
+
}
void TestActivationClient::DeactivateWindow(Window* window) {
- if (window == active_window_) {
- if (active_window_)
- active_window_->RemoveObserver(this);
- active_window_ = NULL;
+ if (window && window == GetActiveWindow()) {
+ RemoveActiveWindow(window);
+ if (aura::client::GetActivationDelegate(window))
+ aura::client::GetActivationDelegate(window)->OnLostActive();
}
}
-
Window* TestActivationClient::GetActiveWindow() {
- return active_window_;
+ if (active_windows_.size() == 0)
+ return NULL;
+ return active_windows_.back();
}
bool TestActivationClient::CanFocusWindow(Window* window) const {
@@ -51,9 +63,26 @@ bool TestActivationClient::CanFocusWindow(Window* window) const {
// TestActivationClient, WindowObserver implementation:
void TestActivationClient::OnWindowDestroyed(Window* window) {
- if (window == active_window_) {
- window->RemoveObserver(this);
- active_window_ = NULL;
+ if (window == GetActiveWindow()) {
+ active_windows_.pop_back();
+ Window *next_active = GetActiveWindow();
+ if (next_active) {
+ if (aura::client::GetActivationDelegate(next_active))
+ aura::client::GetActivationDelegate(next_active)->OnActivated();
+ }
+ return;
+ }
+
+ RemoveActiveWindow(window);
+}
+
+void TestActivationClient::RemoveActiveWindow(Window* window) {
+ for (unsigned int i = 0; i < active_windows_.size(); ++i) {
+ if (active_windows_[i] == window) {
+ active_windows_.erase(active_windows_.begin() + i);
+ window->RemoveObserver(this);
+ return;
+ }
}
}
« content/test/test_renderer_host.h ('K') | « ui/aura/test/test_activation_client.h ('k') | no next file » | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698