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

Side by Side Diff: content/public/browser/web_contents_delegate.h

Issue 128053002: Make WebContentsDelegate::OpenColorChooser return NULL on failure (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src
Patch Set: Created 6 years, 11 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 | « content/browser/web_contents/web_contents_impl.cc ('k') | no next file » | 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) 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 #ifndef CONTENT_PUBLIC_BROWSER_WEB_CONTENTS_DELEGATE_H_ 5 #ifndef CONTENT_PUBLIC_BROWSER_WEB_CONTENTS_DELEGATE_H_
6 #define CONTENT_PUBLIC_BROWSER_WEB_CONTENTS_DELEGATE_H_ 6 #define CONTENT_PUBLIC_BROWSER_WEB_CONTENTS_DELEGATE_H_
7 7
8 #include <set> 8 #include <set>
9 #include <string> 9 #include <string>
10 10
(...skipping 317 matching lines...) Expand 10 before | Expand all | Expand 10 after
328 // Invoked when navigating to a pending entry. When invoked the 328 // Invoked when navigating to a pending entry. When invoked the
329 // NavigationController has configured its pending entry, but it has not yet 329 // NavigationController has configured its pending entry, but it has not yet
330 // been committed. 330 // been committed.
331 virtual void DidNavigateToPendingEntry(WebContents* source) {} 331 virtual void DidNavigateToPendingEntry(WebContents* source) {}
332 332
333 // Returns a pointer to a service to manage JavaScript dialogs. May return 333 // Returns a pointer to a service to manage JavaScript dialogs. May return
334 // NULL in which case dialogs aren't shown. 334 // NULL in which case dialogs aren't shown.
335 virtual JavaScriptDialogManager* GetJavaScriptDialogManager(); 335 virtual JavaScriptDialogManager* GetJavaScriptDialogManager();
336 336
337 // Called when color chooser should open. Returns the opened color chooser. 337 // Called when color chooser should open. Returns the opened color chooser.
338 // Ownership of the returned pointer is transferred to the caller. 338 // Returns NULL if we failed to open the color chooser (e.g. when there is a
339 // ColorChooserDialog already open on Windows). Ownership of the returned
340 // pointer is transferred to the caller.
339 virtual ColorChooser* OpenColorChooser( 341 virtual ColorChooser* OpenColorChooser(
340 WebContents* web_contents, 342 WebContents* web_contents,
341 SkColor color, 343 SkColor color,
342 const std::vector<ColorSuggestion>& suggestions); 344 const std::vector<ColorSuggestion>& suggestions);
343 345
344 // Called when a file selection is to be done. 346 // Called when a file selection is to be done.
345 virtual void RunFileChooser(WebContents* web_contents, 347 virtual void RunFileChooser(WebContents* web_contents,
346 const FileChooserParams& params) {} 348 const FileChooserParams& params) {}
347 349
348 // Request to enumerate a directory. This is equivalent to running the file 350 // Request to enumerate a directory. This is equivalent to running the file
(...skipping 125 matching lines...) Expand 10 before | Expand all | Expand 10 after
474 // Called when |this| is no longer the WebContentsDelegate for |source|. 476 // Called when |this| is no longer the WebContentsDelegate for |source|.
475 void Detach(WebContents* source); 477 void Detach(WebContents* source);
476 478
477 // The WebContents that this is currently a delegate for. 479 // The WebContents that this is currently a delegate for.
478 std::set<WebContents*> attached_contents_; 480 std::set<WebContents*> attached_contents_;
479 }; 481 };
480 482
481 } // namespace content 483 } // namespace content
482 484
483 #endif // CONTENT_PUBLIC_BROWSER_WEB_CONTENTS_DELEGATE_H_ 485 #endif // CONTENT_PUBLIC_BROWSER_WEB_CONTENTS_DELEGATE_H_
OLDNEW
« no previous file with comments | « content/browser/web_contents/web_contents_impl.cc ('k') | no next file » | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698