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

Side by Side Diff: chrome/browser/ui/views/frame/browser_frame.cc

Issue 2127143002: Integrate audible tab tracking into desktop engagement service. (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: Created 4 years, 5 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 unified diff | Download patch
« no previous file with comments | « chrome/browser/ui/tab_helpers.cc ('k') | chrome/browser/ui/views/frame/browser_view.cc » ('j') | no next file with comments »
Toggle Intra-line Diffs ('i') | Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
OLDNEW
1 // Copyright (c) 2012 The Chromium Authors. All rights reserved. 1 // Copyright (c) 2012 The Chromium Authors. All rights reserved.
2 // Use of this source code is governed by a BSD-style license that can be 2 // Use of this source code is governed by a BSD-style license that can be
3 // found in the LICENSE file. 3 // found in the LICENSE file.
4 4
5 #include "chrome/browser/ui/views/frame/browser_frame.h" 5 #include "chrome/browser/ui/views/frame/browser_frame.h"
6 6
7 #include <utility> 7 #include <utility>
8 8
9 #include "base/command_line.h" 9 #include "base/command_line.h"
10 #include "base/debug/leak_annotations.h" 10 #include "base/debug/leak_annotations.h"
(...skipping 197 matching lines...) Expand 10 before | Expand all | Expand 10 after
208 208
209 void BrowserFrame::OnNativeWidgetActivationChanged(bool active) { 209 void BrowserFrame::OnNativeWidgetActivationChanged(bool active) {
210 if (active) { 210 if (active) {
211 // When running under remote desktop, if the remote desktop client is not 211 // When running under remote desktop, if the remote desktop client is not
212 // active on the users desktop, then none of the windows contained in the 212 // active on the users desktop, then none of the windows contained in the
213 // remote desktop will be activated. However, NativeWidget::Activate() will 213 // remote desktop will be activated. However, NativeWidget::Activate() will
214 // still bring this browser window to the foreground. We explicitly set 214 // still bring this browser window to the foreground. We explicitly set
215 // ourselves as the last active browser window to ensure that we get treated 215 // ourselves as the last active browser window to ensure that we get treated
216 // as such by the rest of Chrome. 216 // as such by the rest of Chrome.
217 BrowserList::SetLastActive(browser_view_->browser()); 217 BrowserList::SetLastActive(browser_view_->browser());
218 } else {
219 BrowserList::NotifyBrowserNoLongerActive(browser_view_->browser());
218 } 220 }
219 Widget::OnNativeWidgetActivationChanged(active); 221 Widget::OnNativeWidgetActivationChanged(active);
220 } 222 }
221 223
222 void BrowserFrame::OnNativeWidgetWorkspaceChanged() { 224 void BrowserFrame::OnNativeWidgetWorkspaceChanged() {
223 chrome::SaveWindowWorkspace(browser_view_->browser(), GetWorkspace()); 225 chrome::SaveWindowWorkspace(browser_view_->browser(), GetWorkspace());
224 Widget::OnNativeWidgetWorkspaceChanged(); 226 Widget::OnNativeWidgetWorkspaceChanged();
225 } 227 }
226 228
227 void BrowserFrame::ShowContextMenuForView(views::View* source, 229 void BrowserFrame::ShowContextMenuForView(views::View* source,
(...skipping 40 matching lines...) Expand 10 before | Expand all | Expand 10 after
268 menu_model_builder_.reset( 270 menu_model_builder_.reset(
269 new SystemMenuModelBuilder(browser_view_, browser_view_->browser())); 271 new SystemMenuModelBuilder(browser_view_, browser_view_->browser()));
270 menu_model_builder_->Init(); 272 menu_model_builder_->Init();
271 } 273 }
272 return menu_model_builder_->menu_model(); 274 return menu_model_builder_->menu_model();
273 } 275 }
274 276
275 views::View* BrowserFrame::GetNewAvatarMenuButton() { 277 views::View* BrowserFrame::GetNewAvatarMenuButton() {
276 return browser_frame_view_->GetProfileSwitcherView(); 278 return browser_frame_view_->GetProfileSwitcherView();
277 } 279 }
OLDNEW
« no previous file with comments | « chrome/browser/ui/tab_helpers.cc ('k') | chrome/browser/ui/views/frame/browser_view.cc » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698