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

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

Issue 1210013007: clang/win: Fix warnings to prepare for building without -Wno-reorder. (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@clang-unsequenced
Patch Set: Created 5 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
OLDNEW
1 // Copyright 2012 The Chromium Authors. All rights reserved. 1 // Copyright 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_view.h" 5 #include "chrome/browser/ui/views/frame/browser_view.h"
6 6
7 #include <algorithm> 7 #include <algorithm>
8 8
9 #include "base/auto_reset.h" 9 #include "base/auto_reset.h"
10 #include "base/command_line.h" 10 #include "base/command_line.h"
(...skipping 443 matching lines...) Expand 10 before | Expand all | Expand 10 after
454 top_container_(nullptr), 454 top_container_(nullptr),
455 tabstrip_(nullptr), 455 tabstrip_(nullptr),
456 toolbar_(nullptr), 456 toolbar_(nullptr),
457 find_bar_host_view_(nullptr), 457 find_bar_host_view_(nullptr),
458 infobar_container_(nullptr), 458 infobar_container_(nullptr),
459 contents_web_view_(nullptr), 459 contents_web_view_(nullptr),
460 contents_container_(nullptr), 460 contents_container_(nullptr),
461 initialized_(false), 461 initialized_(false),
462 in_process_fullscreen_(false), 462 in_process_fullscreen_(false),
463 #if defined(OS_WIN) 463 #if defined(OS_WIN)
464 ticker_(0),
464 hung_window_detector_(&hung_plugin_action_), 465 hung_window_detector_(&hung_plugin_action_),
465 ticker_(0),
466 #endif 466 #endif
467 force_location_bar_focus_(false), 467 force_location_bar_focus_(false),
468 activate_modal_dialog_factory_(this) { 468 activate_modal_dialog_factory_(this) {
469 } 469 }
470 470
471 BrowserView::~BrowserView() { 471 BrowserView::~BrowserView() {
472 // All the tabs should have been destroyed already. If we were closed by the 472 // All the tabs should have been destroyed already. If we were closed by the
473 // OS with some tabs than the NativeBrowserFrame should have destroyed them. 473 // OS with some tabs than the NativeBrowserFrame should have destroyed them.
474 DCHECK_EQ(0, browser_->tab_strip_model()->count()); 474 DCHECK_EQ(0, browser_->tab_strip_model()->count());
475 475
(...skipping 2141 matching lines...) Expand 10 before | Expand all | Expand 10 after
2617 return immersive_mode_controller()->IsEnabled(); 2617 return immersive_mode_controller()->IsEnabled();
2618 } 2618 }
2619 2619
2620 views::Widget* BrowserView::GetBubbleAssociatedWidget() { 2620 views::Widget* BrowserView::GetBubbleAssociatedWidget() {
2621 return GetWidget(); 2621 return GetWidget();
2622 } 2622 }
2623 2623
2624 gfx::Rect BrowserView::GetTopContainerBoundsInScreen() { 2624 gfx::Rect BrowserView::GetTopContainerBoundsInScreen() {
2625 return top_container_->GetBoundsInScreen(); 2625 return top_container_->GetBoundsInScreen();
2626 } 2626 }
OLDNEW

Powered by Google App Engine
This is Rietveld 408576698