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

Unified Diff: chrome/browser/ui/views/frame/browser_frame.cc

Issue 2939943004: Fix duplicate logging of ActiveBrowserChanged action on views. (Closed)
Patch Set: Fix ChromeVisibilityObserverBrowserTest on windows. Created 3 years, 6 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
« no previous file with comments | « chrome/browser/ui/views/frame/browser_frame.h ('k') | no next file » | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: chrome/browser/ui/views/frame/browser_frame.cc
diff --git a/chrome/browser/ui/views/frame/browser_frame.cc b/chrome/browser/ui/views/frame/browser_frame.cc
index 34168ebdf5bd2cf52ff9150c2cd30d14035c4af6..9c4367f34c552f9ab7f008804bb18a4791adede2 100644
--- a/chrome/browser/ui/views/frame/browser_frame.cc
+++ b/chrome/browser/ui/views/frame/browser_frame.cc
@@ -205,21 +205,6 @@ void BrowserFrame::SchedulePaintInRect(const gfx::Rect& rect) {
}
}
-void BrowserFrame::OnNativeWidgetActivationChanged(bool active) {
- if (active) {
- // When running under remote desktop, if the remote desktop client is not
- // active on the users desktop, then none of the windows contained in the
- // remote desktop will be activated. However, NativeWidget::Activate() will
- // still bring this browser window to the foreground. We explicitly set
- // ourselves as the last active browser window to ensure that we get treated
- // as such by the rest of Chrome.
- BrowserList::SetLastActive(browser_view_->browser());
- } else {
- BrowserList::NotifyBrowserNoLongerActive(browser_view_->browser());
- }
- Widget::OnNativeWidgetActivationChanged(active);
-}
-
void BrowserFrame::OnNativeWidgetWorkspaceChanged() {
chrome::SaveWindowWorkspace(browser_view_->browser(), GetWorkspace());
#if !defined(OS_CHROMEOS) && defined(USE_X11)
« no previous file with comments | « chrome/browser/ui/views/frame/browser_frame.h ('k') | no next file » | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698