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

Side by Side Diff: chrome/renderer/render_view.h

Issue 6627063: Ignore JavaScript messages (alert/confirm/prompt) during unload handlers. (Closed) Base URL: http://git.chromium.org/git/chromium.git@trunk
Patch Set: Created 9 years, 9 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 (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_RENDERER_RENDER_VIEW_H_ 5 #ifndef CHROME_RENDERER_RENDER_VIEW_H_
6 #define CHROME_RENDERER_RENDER_VIEW_H_ 6 #define CHROME_RENDERER_RENDER_VIEW_H_
7 #pragma once 7 #pragma once
8 8
9 #include <deque> 9 #include <deque>
10 #include <map> 10 #include <map>
(...skipping 764 matching lines...) Expand 10 before | Expand all | Expand 10 after
775 ThumbnailScore* score); 775 ThumbnailScore* score);
776 776
777 // Capture a snapshot of a view. This is used to allow an extension 777 // Capture a snapshot of a view. This is used to allow an extension
778 // to get a snapshot of a tab using chrome.tabs.captureVisibleTab(). 778 // to get a snapshot of a tab using chrome.tabs.captureVisibleTab().
779 bool CaptureSnapshot(WebKit::WebView* view, SkBitmap* snapshot); 779 bool CaptureSnapshot(WebKit::WebView* view, SkBitmap* snapshot);
780 780
781 bool RunJavaScriptMessage(int type, 781 bool RunJavaScriptMessage(int type,
782 const std::wstring& message, 782 const std::wstring& message,
783 const std::wstring& default_value, 783 const std::wstring& default_value,
784 const GURL& frame_url, 784 const GURL& frame_url,
785 bool unload_handler_being_run,
785 std::wstring* result); 786 std::wstring* result);
786 787
787 // Sends a message and runs a nested message loop. 788 // Sends a message and runs a nested message loop.
788 bool SendAndRunNestedMessageLoop(IPC::SyncMessage* message); 789 bool SendAndRunNestedMessageLoop(IPC::SyncMessage* message);
789 790
790 // Adds search provider from the given OpenSearch description URL as a 791 // Adds search provider from the given OpenSearch description URL as a
791 // keyword search. 792 // keyword search.
792 void AddGURLSearchProvider(const GURL& osd_url, 793 void AddGURLSearchProvider(const GURL& osd_url,
793 const ViewHostMsg_PageHasOSDD_Type& provider_type); 794 const ViewHostMsg_PageHasOSDD_Type& provider_type);
794 795
(...skipping 659 matching lines...) Expand 10 before | Expand all | Expand 10 after
1454 // bunch of stuff, you should probably create a helper class and put your 1455 // bunch of stuff, you should probably create a helper class and put your
1455 // data and methods on that to avoid bloating RenderView more. You can use 1456 // data and methods on that to avoid bloating RenderView more. You can use
1456 // the Observer interface to filter IPC messages and receive frame change 1457 // the Observer interface to filter IPC messages and receive frame change
1457 // notifications. 1458 // notifications.
1458 // --------------------------------------------------------------------------- 1459 // ---------------------------------------------------------------------------
1459 1460
1460 DISALLOW_COPY_AND_ASSIGN(RenderView); 1461 DISALLOW_COPY_AND_ASSIGN(RenderView);
1461 }; 1462 };
1462 1463
1463 #endif // CHROME_RENDERER_RENDER_VIEW_H_ 1464 #endif // CHROME_RENDERER_RENDER_VIEW_H_
OLDNEW

Powered by Google App Engine
This is Rietveld 408576698