[Offline pages] Removing obsolete TODOs as part of PE fixit

components/offline_pages/core/background/request_queue_unittest.cc

 // Copyright 2016 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/offline_pages/core/background/request_queue.h"
 
 #include <memory>
 #include <utility>
 
 #include "base/bind.h"
@@ skipping 67 matching lines @@
   }
 
   int32_t total_expired_requests() { return total_expired_requests_; }
 
  private:
   BackgroundSavePageResult last_request_expiration_status_;
   SavePageRequest last_expired_request_;
   int32_t total_expired_requests_;
 };
 
-// TODO(fgorski): Add tests for store failures in add/remove/get.
 class RequestQueueTest : public testing::Test {
  public:
   RequestQueueTest();
   ~RequestQueueTest() override;
 
   // Test overrides.
   void SetUp() override;
 
   void PumpLoop();
 
@@ skipping 475 matching lines @@
   // Doing a get should show no entries left in the queue since the expired
   // request has been removed.
   queue()->GetRequests(
       base::Bind(&RequestQueueTest::GetRequestsDone, base::Unretained(this)));
   this->PumpLoop();
   ASSERT_EQ(GetRequestsResult::SUCCESS, this->last_get_requests_result());
   ASSERT_TRUE(this->last_requests().empty());
 }
 
 }  // namespace offline_pages