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

Unified Diff: webkit/glue/dom_operations.cc

Issue 10445025: GetAllSavableResourceLinksForCurrentPage shouldn't be permitted to modify the array of savable sche… (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src
Patch Set: updated to remove cast added in r139253 Created 8 years, 7 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 side-by-side diff with in-line comments
Download patch
« no previous file with comments | « webkit/glue/dom_operations.h ('k') | no next file » | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: webkit/glue/dom_operations.cc
diff --git a/webkit/glue/dom_operations.cc b/webkit/glue/dom_operations.cc
index cedf58ec04a7e6eea913b057e6bc1f9beb62cac1..4a791c66984c943466a58ab02167fa4316dd2959 100644
--- a/webkit/glue/dom_operations.cc
+++ b/webkit/glue/dom_operations.cc
@@ -106,7 +106,7 @@ void GetSavableResourceLinkForElement(
void GetAllSavableResourceLinksForFrame(WebFrame* current_frame,
SavableResourcesUniqueCheck* unique_check,
webkit_glue::SavableResourcesResult* result,
- const char** savable_schemes) {
+ const char* const* savable_schemes) {
// Get current frame's URL.
GURL current_frame_url = current_frame->document().url();
@@ -196,7 +196,7 @@ WebString GetSubResourceLinkFromElement(const WebElement& element) {
// frame and sub-frame
bool GetAllSavableResourceLinksForCurrentPage(WebView* view,
const GURL& page_url, SavableResourcesResult* result,
- const char** savable_schemes) {
+ const char* const* savable_schemes) {
WebFrame* main_frame = view->mainFrame();
if (!main_frame)
return false;
« no previous file with comments | « webkit/glue/dom_operations.h ('k') | no next file » | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698