Remove unnecessary TODO

The response needs to be handled within this method anyway
adding a class for handling it would be nice, but doesn't
address any current issue. If portability is important in the
future a new class can be created at that time.

Bug: chromium:169883262
Change-Id: Idd8d8505ae003dc71f3b57b1d1877f6b4f903d4f
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2583507
Commit-Queue: Zach Trudo <zatrudo@google.com>
Reviewed-by: Leonid Baraz <lbaraz@chromium.org>
Cr-Commit-Position: refs/heads/master@{#836749}

chrome/browser/policy/messaging_layer/upload/record_handler_impl.cc

@@ -217,7 +217,6 @@ void RecordHandlerImpl::ReportUploader::HandleSuccessfulUpload() {
   //    }
   //    "encryptionSettings": ... // EncryptionSettings proto
   //  }
-  // TODO(b/169883262): Factor out the decoding into a separate class.
   const base::Value* last_succeed_uploaded_record =
       last_response_.FindDictKey("lastSucceedUploadedRecord");
   if (last_succeed_uploaded_record != nullptr) {