Commit ad0719c9 authored by Abhijeet Singh's avatar Abhijeet Singh Committed by Commit Bot

Remove extraneous call to RemovePreTargetHandler

QuickAnswersView never gets added as a pre-target handler, so call to
RemovePreTargetHandler in ~QuickAnswersView() is extraneous and is thus
removed.

Bug: b:150475423
Test: Tested on Chrome OS VM.
Change-Id: Id1246e27d626daa4a6da5bb32d9b1c471c410c1e
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2216531
Commit-Queue: Abhijeet Singh <siabhijeet@google.com>
Reviewed-by: default avatarXiyuan Xia <xiyuan@chromium.org>
Reviewed-by: default avatarDavid Black <dmblack@google.com>
Cr-Commit-Position: refs/heads/master@{#772440}
parent 2a6ebd7f
...@@ -137,9 +137,7 @@ QuickAnswersView::QuickAnswersView(const gfx::Rect& anchor_view_bounds, ...@@ -137,9 +137,7 @@ QuickAnswersView::QuickAnswersView(const gfx::Rect& anchor_view_bounds,
reinterpret_cast<void*>(views::MenuConfig::kMenuControllerGroupingId)); reinterpret_cast<void*>(views::MenuConfig::kMenuControllerGroupingId));
} }
QuickAnswersView::~QuickAnswersView() { QuickAnswersView::~QuickAnswersView() = default;
Shell::Get()->RemovePreTargetHandler(this);
}
const char* QuickAnswersView::GetClassName() const { const char* QuickAnswersView::GetClassName() const {
return "QuickAnswersView"; return "QuickAnswersView";
......
Markdown is supported
0%
or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment