Remove TODOs and update method comments

components/safe_browsing/base_ui_manager.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/safe_browsing/base_ui_manager.h"
 
 #include "base/bind.h"
 #include "base/callback.h"
 #include "base/i18n/rtl.h"
 #include "base/metrics/histogram_macros.h"
@@ skipping 81 matching lines @@
 }
 
 }  // namespace
 
 namespace safe_browsing {
 
 BaseUIManager::BaseUIManager() {}
 
 void BaseUIManager::StopOnIOThread(bool shutdown) {
   DCHECK_CURRENTLY_ON(BrowserThread::IO);
-  // TODO(ntfschr): implement this once SafeBrowsingService is componentized
   return;
 }
 
 BaseUIManager::~BaseUIManager() {}
 
 bool BaseUIManager::IsWhitelisted(const UnsafeResource& resource) {
   NavigationEntry* entry = nullptr;
   if (resource.is_subresource) {
     entry = resource.GetNavigationEntryForResource();
   }
@@ skipping 121 matching lines @@
   UMA_HISTOGRAM_LONG_TIMES("SB2.Delay", time);
   return;
 }
 
 // A safebrowsing hit is sent after a blocking page for malware/phishing
 // or after the warning dialog for download urls, only for
 // UMA || extended_reporting users.
 void BaseUIManager::MaybeReportSafeBrowsingHit(
     const HitReport& hit_report) {
   DCHECK_CURRENTLY_ON(BrowserThread::UI);
-  // TODO(ntfschr): implement this once we support reporting in WebView
   return;
 }
 
 void BaseUIManager::ReportSafeBrowsingHitOnIOThread(
     const HitReport& hit_report) {
   DCHECK_CURRENTLY_ON(BrowserThread::IO);
-  // TODO(ntfschr): implement this once we support reporting in WebView
   return;
 }
 
 // If the user had opted-in to send ThreatDetails, this gets called
 // when the report is ready.
 void BaseUIManager::SendSerializedThreatDetails(
     const std::string& serialized) {
   DCHECK_CURRENTLY_ON(BrowserThread::IO);
-  // TODO(ntfschr): implement this once we support reporting in WebView
   return;
 }
 
 // Record this domain in the given WebContents as either whitelisted or
 // pending whitelisting (if an interstitial is currently displayed). If an
 // existing WhitelistUrlSet does not yet exist, create a new WhitelistUrlSet.
 void BaseUIManager::AddToWhitelistUrlSet(const GURL& whitelist_url,
                                          WebContents* web_contents,
                                          bool pending,
                                          SBThreatType threat_type) {
@@ skipping 75 matching lines @@
   if (resource.is_subresource) {
     NavigationEntry* entry = resource.GetNavigationEntryForResource();
     if (!entry)
       return GURL();
     return entry->GetURL().GetWithEmptyPath();
   }
   return resource.url.GetWithEmptyPath();
 }
 
 }  // namespace safe_browsing