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

Unified Diff: ui/views/focus/focus_manager.cc

Issue 2826423003: Expand Chrome OS ARC support to create one tree source per package (Closed)
Patch Set: One tree per app. Created 3 years, 8 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
Index: ui/views/focus/focus_manager.cc
diff --git a/ui/views/focus/focus_manager.cc b/ui/views/focus/focus_manager.cc
index 740a4c7279a83d8ee4be3c189ddac6430092dec5..1dacb75a7163e9ed24344d96dbef48ae1c0a1fa8 100644
--- a/ui/views/focus/focus_manager.cc
+++ b/ui/views/focus/focus_manager.cc
@@ -315,8 +315,12 @@ void FocusManager::SetKeyboardAccessible(bool keyboard_accessible) {
void FocusManager::SetFocusedViewWithReason(
View* view, FocusChangeReason reason) {
- if (focused_view_ == view)
+ if (focused_view_ == view) {
+ LOG(ERROR) << "equal focus!";
yawano 2017/04/25 09:29:23 nit: Please remove logs which you have added for d
David Tseng 2017/04/25 22:48:03 Acknowledged.
return;
+ }
+
+ LOG(ERROR) << "setting focus!";
// TODO(oshima|achuith): This is to diagnose crbug.com/687232.
// Change this to DCHECK once it's resolved.
CHECK(!view || ContainsView(view));
@@ -344,6 +348,7 @@ void FocusManager::SetFocusedViewWithReason(
View* old_focused_view = focused_view_;
focused_view_ = view;
+ LOG(ERROR) << "focus set!";
if (old_focused_view) {
old_focused_view->RemoveObserver(this);
old_focused_view->Blur();

Powered by Google App Engine
This is Rietveld 408576698