Remove some unused code in PhishingDOMFeatureExtractor.

BUG=none
TEST=PhishingDOMFeatureExtractorTest


Review URL: http://codereview.chromium.org/8384023

git-svn-id: svn://svn.chromium.org/chrome/trunk/src@108028 0039d316-1c4b-4281-b951-d872f2087c98

chrome/renderer/safe_browsing/phishing_dom_feature_extractor.cc

@@ -170,11 +170,7 @@ void PhishingDOMFeatureExtractor::ExtractFeaturesWithTimeout() {
     } else {
       // We just moved to a new frame, so update our frame state
       // and advance to the first element.
-      if (!ResetFrameData()) {
-        // Nothing in this frame, move on to the next one.
-        DLOG(WARNING) << "No content in frame, skipping";
-        continue;
-      }
+      ResetFrameData();
       cur_node = cur_frame_data_->elements.firstItem();
     }
 
@@ -381,7 +377,7 @@ void PhishingDOMFeatureExtractor::Clear() {
   cur_document_.reset();
 }
 
-bool PhishingDOMFeatureExtractor::ResetFrameData() {
+void PhishingDOMFeatureExtractor::ResetFrameData() {
   DCHECK(!cur_document_.isNull());
   DCHECK(!cur_frame_data_.get());
 
@@ -390,7 +386,6 @@ bool PhishingDOMFeatureExtractor::ResetFrameData() {
   cur_frame_data_->domain =
       net::RegistryControlledDomainService::GetDomainAndRegistry(
           cur_document_.url());
-  return true;
 }
 
 WebKit::WebDocument PhishingDOMFeatureExtractor::GetNextDocument() {

chrome/renderer/safe_browsing/phishing_dom_feature_extractor.h

@@ -106,8 +106,8 @@ class PhishingDOMFeatureExtractor {
   void Clear();
 
   // Called after advancing |cur_document_| to update the state in
-  // |cur_frame_data_|.  Returns true if the state was updated successfully.
-  bool ResetFrameData();
+  // |cur_frame_data_|.
+  void ResetFrameData();
 
   // Returns the next document in frame-traversal order from cur_document_.
   // If there are no more documents, returns a null WebDocument.