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

Side by Side Diff: webkit/glue/webview_delegate.h

Issue 246079: More WebView / WebViewDelegate cleanup. (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src/
Patch Set: '' Created 11 years, 2 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
« no previous file with comments | « webkit/glue/webview.h ('k') | webkit/glue/webview_impl.h » ('j') | no next file with comments »
Toggle Intra-line Diffs ('i') | Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
OLDNEW
1 // Copyright (c) 2006-2009 The Chromium Authors. All rights reserved. 1 // Copyright (c) 2006-2009 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 // WebCore provides hooks for several kinds of functionality, allowing separate 5 // WebCore provides hooks for several kinds of functionality, allowing separate
6 // classes termed "delegates" to receive notifications (in the form of direct 6 // classes termed "delegates" to receive notifications (in the form of direct
7 // function calls) when certain events are about to occur or have just occurred. 7 // function calls) when certain events are about to occur or have just occurred.
8 // In some cases, the delegate implements the needed functionality; in others, 8 // In some cases, the delegate implements the needed functionality; in others,
9 // the delegate has some control over the behavior but doesn't actually 9 // the delegate has some control over the behavior but doesn't actually
10 // implement it. For example, the UI delegate is responsible for showing a 10 // implement it. For example, the UI delegate is responsible for showing a
(...skipping 52 matching lines...) Expand 10 before | Expand all | Expand 10 after
63 63
64 private: 64 private:
65 DISALLOW_COPY_AND_ASSIGN(WebFileChooserCallback); 65 DISALLOW_COPY_AND_ASSIGN(WebFileChooserCallback);
66 }; 66 };
67 67
68 // TODO(darin): Eliminate WebViewDelegate in favor of WebViewClient. 68 // TODO(darin): Eliminate WebViewDelegate in favor of WebViewClient.
69 class WebViewDelegate : public WebKit::WebViewClient { 69 class WebViewDelegate : public WebKit::WebViewClient {
70 public: 70 public:
71 // WebView additions ------------------------------------------------------- 71 // WebView additions -------------------------------------------------------
72 72
73 // Notifies how many matches have been found so far, for a given request_id.
74 // |final_update| specifies whether this is the last update (all frames have
75 // completed scoping).
76 virtual void ReportFindInPageMatchCount(int count, int request_id,
77 bool final_update) {
78 }
79
80 // Notifies the browser what tick-mark rect is currently selected. Parameter
81 // |request_id| lets the recipient know which request this message belongs to,
82 // so that it can choose to ignore the message if it has moved on to other
83 // things. |selection_rect| is expected to have coordinates relative to the
84 // top left corner of the web page area and represent where on the screen the
85 // selection rect is currently located.
86 virtual void ReportFindInPageSelection(int request_id,
87 int active_match_ordinal,
88 const WebKit::WebRect& selection) {
89 }
90
91 // Returns whether this WebView was opened by a user gesture. 73 // Returns whether this WebView was opened by a user gesture.
92 virtual bool WasOpenedByUserGesture() const { 74 virtual bool WasOpenedByUserGesture() const {
93 return true; 75 return true;
94 } 76 }
95 77
96 // Called by ChromeClientImpl::focus() if accessibility on the renderer side 78 // Called by ChromeClientImpl::focus() if accessibility on the renderer side
97 // is enabled, and a focus change has occurred. Will retrieve the id of the 79 // is enabled, and a focus change has occurred. Will retrieve the id of the
98 // input AccessibilityObject and send it through IPC for handling on the 80 // input AccessibilityObject and send it through IPC for handling on the
99 // browser side. 81 // browser side.
100 virtual void FocusAccessibilityObject(WebCore::AccessibilityObject* acc_obj) { 82 virtual void FocusAccessibilityObject(WebCore::AccessibilityObject* acc_obj) {
(...skipping 75 matching lines...) Expand 10 before | Expand all | Expand 10 after
176 const GURL& page_url, 158 const GURL& page_url,
177 const GURL& frame_url, 159 const GURL& frame_url,
178 const ContextMenuMediaParams& media_params, 160 const ContextMenuMediaParams& media_params,
179 const std::wstring& selection_text, 161 const std::wstring& selection_text,
180 const std::wstring& misspelled_word, 162 const std::wstring& misspelled_word,
181 int edit_flags, 163 int edit_flags,
182 const std::string& security_info, 164 const std::string& security_info,
183 const std::string& frame_charset) { 165 const std::string& frame_charset) {
184 } 166 }
185 167
186 // InspectorClient ---------------------------------------------------------
187
188 virtual void UpdateInspectorSettings(const std::wstring& raw_settings) { }
189 168
190 // DevTools ---------------------------------------------------------------- 169 // DevTools ----------------------------------------------------------------
191 170
192 virtual WebDevToolsAgentDelegate* GetWebDevToolsAgentDelegate() { 171 virtual WebDevToolsAgentDelegate* GetWebDevToolsAgentDelegate() {
193 return NULL; 172 return NULL;
194 } 173 }
195 174
196 protected: 175 protected:
197 ~WebViewDelegate() { } 176 ~WebViewDelegate() { }
198 }; 177 };
199 178
200 #endif // WEBKIT_GLUE_WEBVIEW_DELEGATE_H_ 179 #endif // WEBKIT_GLUE_WEBVIEW_DELEGATE_H_
OLDNEW
« no previous file with comments | « webkit/glue/webview.h ('k') | webkit/glue/webview_impl.h » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698