Cleanup of static lists of schemes & origins that are created at startup.

content/renderer/savable_resources.cc

 // Copyright (c) 2013 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 "content/renderer/savable_resources.h"
 
 #include <set>
 
 #include "base/compiler_specific.h"
 #include "base/logging.h"
 #include "base/strings/string_util.h"
+#include "content/public/common/url_utils.h"
 #include "content/renderer/web_frame_utils.h"
 #include "third_party/WebKit/public/platform/WebString.h"
 #include "third_party/WebKit/public/platform/WebVector.h"
 #include "third_party/WebKit/public/web/WebDocument.h"
 #include "third_party/WebKit/public/web/WebElement.h"
 #include "third_party/WebKit/public/web/WebElementCollection.h"
 #include "third_party/WebKit/public/web/WebInputElement.h"
 #include "third_party/WebKit/public/web/WebLocalFrame.h"
 #include "third_party/WebKit/public/web/WebNode.h"
 #include "third_party/WebKit/public/web/WebView.h"
@@ skipping 67 matching lines @@
   if (!element_url.SchemeIsHTTPOrHTTPS() &&
       !element_url.SchemeIs(url::kFileScheme))
     return;
 
   result->resources_list->push_back(element_url);
 }
 
 }  // namespace
 
 bool GetSavableResourceLinksForFrame(WebFrame* current_frame,
-                                     SavableResourcesResult* result,
-                                     const char** savable_schemes) {
+                                     SavableResourcesResult* result) {
   // 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 false;
 
   // 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) {
-    if (current_frame_url.SchemeIs(savable_schemes[i])) {
-      is_valid_protocol = true;
-      break;
-    }
-  }
-  if (!is_valid_protocol)
+  if (!IsSavableURL(current_frame_url))
     return false;
 
   // Get current using document.
   WebDocument current_doc = current_frame->document();
   // Go through all descent nodes.
   WebElementCollection all = current_doc.all();
   // Go through all elements in this frame.
   for (WebElement element = all.firstItem(); !element.isNull();
        element = all.nextItem()) {
     GetSavableResourceLinkForElement(element,
@@ skipping 47 matching lines @@
   // otherwise return NULL.
   if (!value.isNull() && !value.isEmpty() &&
       !base::StartsWith(value.utf8(), "javascript:",
                         base::CompareCase::INSENSITIVE_ASCII))
     return value;
 
   return WebString();
 }
 
 }  // namespace content