Stop FaviconHandler from querying GetActiveFaviconValidity() part 2/2

components/favicon/core/favicon_handler_unittest.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 "components/favicon/core/favicon_handler.h"
 
 #include<set>
 #include<vector>
 
 #include "base/memory/scoped_ptr.h"
@@ skipping 211 matching lines @@
                   << "should never be called in tests.";
     return -1;
   }
 
   bool IsOffTheRecord() override { return false; }
 
   bool IsBookmarked(const GURL& url) override { return false; }
 
   GURL GetActiveURL() override { return url_; }
 
-  bool GetActiveFaviconValidity() override { return favicon_validity_; }
+  bool GetActiveFaviconValidity() { return favicon_validity_; }
 
   void SetActiveFaviconValidity(bool favicon_validity) override {
     favicon_validity_ = favicon_validity;
   }
 
   GURL GetActiveFaviconURL() override { return favicon_url_; }
 
   void SetActiveFaviconURL(const GURL& favicon_url) override {
     favicon_url_ = favicon_url;
   }
@@ skipping 1507 matching lines @@
   EXPECT_FALSE(driver1.update_active_favicon());
   EXPECT_EQ(3u, driver1.num_favicon_available());
 }
 
 INSTANTIATE_TEST_CASE_P(FaviconHandlerTestActiveFaviconValidityTrueOrFalse,
                         FaviconHandlerActiveFaviconValidityParamTest,
                         ::testing::Bool());
 
 }  // namespace
 }  // namespace favicon