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

Side by Side Diff: chrome/browser/ui/apps/app_info_dialog.h

Issue 899173002: Replace Webstore link with app info dialog link in chrome://apps (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: Reformat Created 5 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 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 #ifndef CHROME_BROWSER_UI_APPS_APP_INFO_DIALOG_H_ 5 #ifndef CHROME_BROWSER_UI_APPS_APP_INFO_DIALOG_H_
6 #define CHROME_BROWSER_UI_APPS_APP_INFO_DIALOG_H_ 6 #define CHROME_BROWSER_UI_APPS_APP_INFO_DIALOG_H_
7 7
8 #include "base/callback_forward.h" 8 #include "base/callback_forward.h"
9 #include "ui/gfx/native_widget_types.h" 9 #include "ui/gfx/native_widget_types.h"
10 10
11 class Profile; 11 class Profile;
12 12
13 namespace extensions { 13 namespace extensions {
14 class Extension; 14 class Extension;
15 } 15 }
16 16
17 namespace gfx { 17 namespace gfx {
18 class Rect; 18 class Rect;
19 class Size; 19 class Size;
20 } 20 }
21 21
22 // Used for UMA to track where the App Info dialog is launched from. 22 // Used for UMA to track where the App Info dialog is launched from.
23 enum AppInfoLaunchSource { 23 enum AppInfoLaunchSource {
24 FROM_APP_LIST, // Launched from the app list context menu. 24 FROM_APP_LIST, // Launched from the app list context menu.
25 FROM_EXTENSIONS_PAGE, // Launched from the chrome://extensions page. 25 FROM_EXTENSIONS_PAGE, // Launched from the chrome://extensions page.
sashab 2015/02/09 11:47:07 Almost forgot! We'll need a new Launch Source for
26 NUM_LAUNCH_SOURCES, 26 NUM_LAUNCH_SOURCES,
27 }; 27 };
28 28
29 // Size of native window container for app info dialog
sashab 2015/02/09 11:47:07 Nit: . at end of comment. Also sentence tense: "Re
30 gfx::Size GetAppInfoNativeDialogSize();
31
sashab 2015/02/09 11:47:08 I'd be happy to put ShouldShowAppInfoDialog here.
29 // Shows the chrome app information as a frameless window for the given |app| 32 // Shows the chrome app information as a frameless window for the given |app|
30 // and |profile| at the given |app_list_bounds|. Appears 'inside' the app list. 33 // and |profile| at the given |app_list_bounds|. Appears 'inside' the app list.
31 void ShowAppInfoInAppList(gfx::NativeWindow parent, 34 void ShowAppInfoInAppList(gfx::NativeWindow parent,
32 const gfx::Rect& app_list_bounds, 35 const gfx::Rect& app_list_bounds,
33 Profile* profile, 36 Profile* profile,
34 const extensions::Extension* app, 37 const extensions::Extension* app,
35 const base::Closure& close_callback); 38 const base::Closure& close_callback);
36 39
37 // Shows the chrome app information in a native dialog box of the given |size|. 40 // Shows the chrome app information in a native dialog box of the given |size|.
38 void ShowAppInfoInNativeDialog(gfx::NativeWindow parent, 41 void ShowAppInfoInNativeDialog(gfx::NativeWindow parent,
39 const gfx::Size& size, 42 const gfx::Size& size,
40 Profile* profile, 43 Profile* profile,
41 const extensions::Extension* app, 44 const extensions::Extension* app,
42 const base::Closure& close_callback); 45 const base::Closure& close_callback);
43 46
44 #endif // CHROME_BROWSER_UI_APPS_APP_INFO_DIALOG_H_ 47 #endif // CHROME_BROWSER_UI_APPS_APP_INFO_DIALOG_H_
OLDNEW

Powered by Google App Engine
This is Rietveld 408576698