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

Side by Side Diff: chrome/browser/ui/webui/html_dialog_ui.h

Issue 8741010: TaskManager: Added functionality to remember the size of the task manager dialog. (Closed) Base URL: http://git.chromium.org/git/chromium.git@trunk
Patch Set: Fixed review comments. Created 9 years 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 (c) 2011 The Chromium Authors. All rights reserved. 1 // Copyright (c) 2011 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_WEBUI_HTML_DIALOG_UI_H_ 5 #ifndef CHROME_BROWSER_UI_WEBUI_HTML_DIALOG_UI_H_
6 #define CHROME_BROWSER_UI_WEBUI_HTML_DIALOG_UI_H_ 6 #define CHROME_BROWSER_UI_WEBUI_HTML_DIALOG_UI_H_
7 #pragma once 7 #pragma once
8 8
9 #include <string> 9 #include <string>
10 #include <vector> 10 #include <vector>
11 11
12 #include "base/string16.h" 12 #include "base/string16.h"
13 #include "chrome/browser/ui/webui/chrome_web_ui.h" 13 #include "chrome/browser/ui/webui/chrome_web_ui.h"
14 #include "googleurl/src/gurl.h" 14 #include "googleurl/src/gurl.h"
15 #include "ui/gfx/rect.h"
15 16
16 struct ContextMenuParams; 17 struct ContextMenuParams;
17 18
18 namespace base { 19 namespace base {
19 class ListValue; 20 class ListValue;
20 template<class T> class PropertyAccessor; 21 template<class T> class PropertyAccessor;
21 } 22 }
22 23
23 namespace gfx { 24 namespace gfx {
24 class Size; 25 class Size;
(...skipping 37 matching lines...) Expand 10 before | Expand all | Expand 10 after
62 virtual void OnCloseContents(TabContents* source, bool* out_close_dialog) = 0; 63 virtual void OnCloseContents(TabContents* source, bool* out_close_dialog) = 0;
63 64
64 // A callback to allow the delegate to dictate that the window should not 65 // A callback to allow the delegate to dictate that the window should not
65 // have a title bar. This is useful when presenting branded interfaces. 66 // have a title bar. This is useful when presenting branded interfaces.
66 virtual bool ShouldShowDialogTitle() const = 0; 67 virtual bool ShouldShowDialogTitle() const = 0;
67 68
68 // A callback to allow the delegate to inhibit context menu or show 69 // A callback to allow the delegate to inhibit context menu or show
69 // customized menu. 70 // customized menu.
70 virtual bool HandleContextMenu(const ContextMenuParams& params); 71 virtual bool HandleContextMenu(const ContextMenuParams& params);
71 72
73 // Stores the dialog bounds.
74 virtual void StoreDialogSize(gfx::Rect dialog_bounds) {}
yoshiki 2011/12/01 14:28:30 Could you add "const" modifier to dialog_bounds ar
NaveenBobbili (Motorola) 2011/12/02 07:58:11 Done.
75
72 protected: 76 protected:
73 virtual ~HtmlDialogUIDelegate() {} 77 virtual ~HtmlDialogUIDelegate() {}
74 }; 78 };
75 79
76 // Displays file URL contents inside a modal HTML dialog. 80 // Displays file URL contents inside a modal HTML dialog.
77 // 81 //
78 // This application really should not use TabContents + WebUI. It should instead 82 // This application really should not use TabContents + WebUI. It should instead
79 // just embed a RenderView in a dialog and be done with it. 83 // just embed a RenderView in a dialog and be done with it.
80 // 84 //
81 // Before loading a URL corresponding to this WebUI, the caller should set its 85 // Before loading a URL corresponding to this WebUI, the caller should set its
(...skipping 40 matching lines...) Expand 10 before | Expand all | Expand 10 after
122 // Intended to be the place to collect the settings and lockdowns 126 // Intended to be the place to collect the settings and lockdowns
123 // necessary for running external UI conponents securely (e.g., the 127 // necessary for running external UI conponents securely (e.g., the
124 // cloud print dialog). 128 // cloud print dialog).
125 class ExternalHtmlDialogUI : public HtmlDialogUI { 129 class ExternalHtmlDialogUI : public HtmlDialogUI {
126 public: 130 public:
127 explicit ExternalHtmlDialogUI(TabContents* tab_contents); 131 explicit ExternalHtmlDialogUI(TabContents* tab_contents);
128 virtual ~ExternalHtmlDialogUI(); 132 virtual ~ExternalHtmlDialogUI();
129 }; 133 };
130 134
131 #endif // CHROME_BROWSER_UI_WEBUI_HTML_DIALOG_UI_H_ 135 #endif // CHROME_BROWSER_UI_WEBUI_HTML_DIALOG_UI_H_
OLDNEW
« no previous file with comments | « chrome/browser/ui/gtk/html_dialog_gtk.cc ('k') | chrome/browser/ui/webui/task_manager_dialog.cc » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698