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

Side by Side Diff: chrome/browser/ui/views/chrome_views_delegate.cc

Issue 2693783005: Revert of Linux Aura: Use Aura theme on all windows when 'Use Classic theme' is used (Closed)
Patch Set: Created 3 years, 10 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 (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/chrome_views_delegate.h" 5 #include "chrome/browser/ui/views/chrome_views_delegate.h"
6 6
7 #include <memory> 7 #include <memory>
8 8
9 #include "base/location.h" 9 #include "base/location.h"
10 #include "base/logging.h" 10 #include "base/logging.h"
(...skipping 432 matching lines...) Expand 10 before | Expand all | Expand 10 after
443 if (params->context) 443 if (params->context)
444 params->context = params->context->GetRootWindow(); 444 params->context = params->context->GetRootWindow();
445 DCHECK(params->parent || params->context || !params->child) 445 DCHECK(params->parent || params->context || !params->child)
446 << "Please provide a parent or context for this widget."; 446 << "Please provide a parent or context for this widget.";
447 if (!params->parent && !params->context) 447 if (!params->parent && !params->context)
448 params->context = ash::Shell::GetPrimaryRootWindow(); 448 params->context = ash::Shell::GetPrimaryRootWindow();
449 #elif defined(USE_AURA) 449 #elif defined(USE_AURA)
450 // While the majority of the time, context wasn't plumbed through due to the 450 // While the majority of the time, context wasn't plumbed through due to the
451 // existence of a global WindowParentingClient, if this window is toplevel, 451 // existence of a global WindowParentingClient, if this window is toplevel,
452 // it's possible that there is no contextual state that we can use. 452 // it's possible that there is no contextual state that we can use.
453 gfx::NativeWindow parent_or_context = 453 if (params->parent == NULL && params->context == NULL && !params->child) {
454 params->parent ? params->parent : params->context; 454 params->native_widget = new views::DesktopNativeWidgetAura(delegate);
455 void* profile = 455 } else if (use_non_toplevel_window) {
456 parent_or_context
457 ? parent_or_context->GetNativeWindowProperty(Profile::kProfileKey)
458 : nullptr;
459 if ((!params->parent && !params->context && !params->child) ||
460 !use_non_toplevel_window) {
461 views::DesktopNativeWidgetAura* native_widget =
462 new views::DesktopNativeWidgetAura(delegate);
463 params->native_widget = native_widget;
464 native_widget->SetNativeWindowProperty(Profile::kProfileKey, profile);
465 } else {
466 views::NativeWidgetAura* native_widget = 456 views::NativeWidgetAura* native_widget =
467 new views::NativeWidgetAura(delegate); 457 new views::NativeWidgetAura(delegate);
458 if (params->parent) {
459 Profile* parent_profile = reinterpret_cast<Profile*>(
460 params->parent->GetNativeWindowProperty(Profile::kProfileKey));
461 native_widget->SetNativeWindowProperty(Profile::kProfileKey,
462 parent_profile);
463 }
468 params->native_widget = native_widget; 464 params->native_widget = native_widget;
469 native_widget->SetNativeWindowProperty(Profile::kProfileKey, profile); 465 } else {
466 params->native_widget = new views::DesktopNativeWidgetAura(delegate);
470 } 467 }
471 #endif 468 #endif
472 } 469 }
473 470
474 #if defined(OS_LINUX) && !defined(OS_CHROMEOS) 471 #if defined(OS_LINUX) && !defined(OS_CHROMEOS)
475 bool ChromeViewsDelegate::WindowManagerProvidesTitleBar(bool maximized) { 472 bool ChromeViewsDelegate::WindowManagerProvidesTitleBar(bool maximized) {
476 // On Ubuntu Unity, the system always provides a title bar for maximized 473 // On Ubuntu Unity, the system always provides a title bar for maximized
477 // windows. 474 // windows.
478 views::LinuxUI* ui = views::LinuxUI::instance(); 475 views::LinuxUI* ui = views::LinuxUI::instance();
479 return maximized && ui && ui->UnityIsRunning(); 476 return maximized && ui && ui->UnityIsRunning();
(...skipping 94 matching lines...) Expand 10 before | Expand all | Expand 10 after
574 return ViewsDelegate::GetBubbleDialogMargins(); 571 return ViewsDelegate::GetBubbleDialogMargins();
575 } 572 }
576 573
577 #if !defined(USE_ASH) 574 #if !defined(USE_ASH)
578 views::Widget::InitParams::WindowOpacity 575 views::Widget::InitParams::WindowOpacity
579 ChromeViewsDelegate::GetOpacityForInitParams( 576 ChromeViewsDelegate::GetOpacityForInitParams(
580 const views::Widget::InitParams& params) { 577 const views::Widget::InitParams& params) {
581 return views::Widget::InitParams::OPAQUE_WINDOW; 578 return views::Widget::InitParams::OPAQUE_WINDOW;
582 } 579 }
583 #endif 580 #endif
OLDNEW

Powered by Google App Engine
This is Rietveld 408576698