GetAllSavableResourceLinksForCurrentPage shouldn't be permitted to modify the array of savable sche…

webkit/glue/dom_operations.cc

 // Copyright (c) 2012 The Chromium Authors. All rights reserved.
 // Use of this source code is governed by a BSD-style license that can be
 // found in the LICENSE file.
 
 #include "webkit/glue/dom_operations.h"
 
 #include <set>
 
 #include "base/compiler_specific.h"
 #include "base/logging.h"
@@ skipping 88 matching lines @@
   result->resources_list->push_back(u);
   // Insert referrer for above new resource link.
   result->referrer_urls_list->push_back(GURL());
   result->referrer_policies_list->push_back(WebKit::WebReferrerPolicyDefault);
 }
 
 // Get all savable resource links from current WebFrameImpl object pointer.
 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();
 
   // If url of current frame is invalid, ignore it.
   if (!current_frame_url.is_valid())
     return;
 
   // If url of current frame is not a savable protocol, ignore it.
   bool is_valid_protocol = false;
   for (int i = 0; savable_schemes[i] != NULL; ++i) {
@@ skipping 69 matching lines @@
       !StartsWithASCII(value.utf8(), "javascript:", false))
     return value;
 
   return WebString();
 }
 
 // Get all savable resource links from current webview, include main
 // 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;
 
   std::set<GURL> resources_set;
   std::set<GURL> frames_set;
   std::vector<WebFrame*> frames;
   SavableResourcesUniqueCheck unique_check(&resources_set,
                                            &frames_set,
                                            &frames);
@@ skipping 118 matching lines @@
     if (!element.hasTagName("meta"))
       continue;
     WebString value = element.getAttribute(attribute_name);
     if (value.isNull() || value != attribute_value)
       continue;
     meta_elements->push_back(element);
   }
 }
 
 }  // webkit_glue