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

Unified Diff: ui/views/widget/widget_accelerator_handling_delegate.cc

Issue 2604793002: Allow Widget to pass accelerator handling to its parent.
Patch Set: Created 4 years 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/widget/widget_accelerator_handling_delegate.cc
diff --git a/ui/views/widget/widget_accelerator_handling_delegate.cc b/ui/views/widget/widget_accelerator_handling_delegate.cc
new file mode 100644
index 0000000000000000000000000000000000000000..46bd333680e2235d83e86d045236ae942e77eaef
--- /dev/null
+++ b/ui/views/widget/widget_accelerator_handling_delegate.cc
@@ -0,0 +1,72 @@
+// Copyright (c) 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/widget/widget_accelerator_handling_delegate.h"
+
+#include "ui/views/focus/focus_manager_delegate.h"
+#include "ui/views/widget/widget.h"
+
+namespace views {
+
+namespace {
+
+class ForwardingFocusManagerDelegate : public FocusManagerDelegate {
+ public:
+ explicit ForwardingFocusManagerDelegate(
+ WidgetAcceleratorHandlingDelegate* owner)
+ : owner_(owner->GetWeakPtr()) {}
+ ~ForwardingFocusManagerDelegate() override = default;
+
+ // Implements FocusManagerDelegate:
+ bool ProcessAccelerator(const ui::Accelerator& accelerator) override {
+ if (!owner_)
+ return false;
+ Widget* target = owner_->accelerator_target();
+ if (!target)
+ return false;
+ FocusManager* target_focus_manager = target->GetFocusManager();
+ if (!target_focus_manager)
+ return false;
+ return target_focus_manager->ProcessAccelerator(accelerator);
+ }
+
+ private:
+ base::WeakPtr<WidgetAcceleratorHandlingDelegate> owner_;
+
+ DISALLOW_COPY_AND_ASSIGN(ForwardingFocusManagerDelegate);
+};
+
+} // namespace
+
+WidgetAcceleratorHandlingDelegate::WidgetAcceleratorHandlingDelegate(
+ Widget* accelerator_target)
+ : accelerator_target_(accelerator_target),
+ accelerator_target_observer_(this),
+ weak_factory_(this) {
+ DCHECK(accelerator_target_);
+ accelerator_target_observer_.Add(accelerator_target_);
+ accelerator_target_->AddHandleAcceleratorsOverride();
+}
+
+WidgetAcceleratorHandlingDelegate::~WidgetAcceleratorHandlingDelegate() {
+ accelerator_target_->RemoveHandleAcceleratorsOverride();
+}
+
+base::WeakPtr<WidgetAcceleratorHandlingDelegate>
+WidgetAcceleratorHandlingDelegate::GetWeakPtr() {
+ return weak_factory_.GetWeakPtr();
+}
+
+std::unique_ptr<FocusManagerDelegate>
+WidgetAcceleratorHandlingDelegate::CreateFocusManagerDelegate() {
+ return base::MakeUnique<ForwardingFocusManagerDelegate>(this);
+}
+
+void WidgetAcceleratorHandlingDelegate::OnWidgetDestroying(Widget* widget) {
+ DCHECK_EQ(accelerator_target_, widget);
+ accelerator_target_ = nullptr;
+ accelerator_target_observer_.Remove(widget);
+}
+
+} // namespace views

Powered by Google App Engine
This is Rietveld 408576698