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

Side by Side Diff: chrome/browser/ui/views/apps/app_info_dialog/app_info_dialog_views.cc

Issue 273223002: views: Make view::Views::GetPreferredSize() const. (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src
Patch Set: More compile fix for ToT Created 6 years, 7 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 | Annotate | Revision Log
OLDNEW
1 // Copyright 2014 The Chromium Authors. All rights reserved. 1 // Copyright 2014 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/apps/app_info_dialog/app_info_dialog_views.h" 5 #include "chrome/browser/ui/views/apps/app_info_dialog/app_info_dialog_views.h"
6 6
7 #include "chrome/browser/ui/views/apps/app_info_dialog/app_info_manage_tab.h" 7 #include "chrome/browser/ui/views/apps/app_info_dialog/app_info_manage_tab.h"
8 #include "chrome/browser/ui/views/apps/app_info_dialog/app_info_permissions_tab. h" 8 #include "chrome/browser/ui/views/apps/app_info_dialog/app_info_permissions_tab. h"
9 #include "chrome/browser/ui/views/apps/app_info_dialog/app_info_summary_tab.h" 9 #include "chrome/browser/ui/views/apps/app_info_dialog/app_info_summary_tab.h"
10 #include "chrome/browser/ui/views/constrained_window_views.h" 10 #include "chrome/browser/ui/views/constrained_window_views.h"
(...skipping 38 matching lines...) Expand 10 before | Expand all | Expand 10 after
49 } 49 }
50 50
51 AppInfoDialog::~AppInfoDialog() {} 51 AppInfoDialog::~AppInfoDialog() {}
52 52
53 bool AppInfoDialog::Cancel() { 53 bool AppInfoDialog::Cancel() {
54 if (!close_callback_.is_null()) 54 if (!close_callback_.is_null())
55 close_callback_.Run(); 55 close_callback_.Run();
56 return true; 56 return true;
57 } 57 }
58 58
59 gfx::Size AppInfoDialog::GetPreferredSize() { 59 gfx::Size AppInfoDialog::GetPreferredSize() const {
60 // These numbers represent the size of the view, not the total size of the 60 // These numbers represent the size of the view, not the total size of the
61 // dialog. The actual dialog will be slightly taller (have a larger height) 61 // dialog. The actual dialog will be slightly taller (have a larger height)
62 // than what is specified here. 62 // than what is specified here.
63 static const int kDialogWidth = 360; 63 static const int kDialogWidth = 360;
64 static const int kDialogHeight = 360; 64 static const int kDialogHeight = 360;
65 return gfx::Size(kDialogWidth, kDialogHeight); 65 return gfx::Size(kDialogWidth, kDialogHeight);
66 } 66 }
67 67
68 int AppInfoDialog::GetDialogButtons() const { return ui::DIALOG_BUTTON_NONE; } 68 int AppInfoDialog::GetDialogButtons() const { return ui::DIALOG_BUTTON_NONE; }
69 69
70 ui::ModalType AppInfoDialog::GetModalType() const { 70 ui::ModalType AppInfoDialog::GetModalType() const {
71 return ui::MODAL_TYPE_WINDOW; 71 return ui::MODAL_TYPE_WINDOW;
72 } 72 }
OLDNEW

Powered by Google App Engine
This is Rietveld 408576698