Remove some TODOs which have been addressed.

Bug: None
Change-Id: I40b36e01f42da2dd0e21d5395155073feb356355
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1955659
Commit-Queue: Peng Huang <penghuang@chromium.org>
Commit-Queue: Vasiliy Telezhnikov <vasilyt@chromium.org>
Auto-Submit: Peng Huang <penghuang@chromium.org>
Reviewed-by: Vasiliy Telezhnikov <vasilyt@chromium.org>
Cr-Commit-Position: refs/heads/master@{#722507}

components/viz/service/display_embedder/skia_output_surface_impl_on_gpu.cc

@@ -231,15 +231,11 @@ class SkiaOutputSurfaceImplOnGpu::ScopedPromiseImageAccess {
     begin_semaphores_.reserve(image_contexts_.size());
     // We may need one more space for the swap buffer semaphore.
     end_semaphores_.reserve(image_contexts_.size() + 1);
-    // TODO(penghuang): gather begin read access semaphores from shared images.
-    // https://crbug.com/944194
     impl_on_gpu_->BeginAccessImages(image_contexts_, &begin_semaphores_,
                                     &end_semaphores_);
   }
 
   ~ScopedPromiseImageAccess() {
-    // TODO(penghuang): end shared image access with meaningful semaphores.
-    // https://crbug.com/944194
     impl_on_gpu_->EndAccessImages(image_contexts_);
   }
 
@@ -324,7 +320,6 @@ scoped_refptr<gpu::SyncPointClientState> CreateSyncPointClientState(
 std::unique_ptr<gpu::SharedImageRepresentationFactory>
 CreateSharedImageRepresentationFactory(SkiaOutputSurfaceDependency* deps,
                                        gpu::MemoryTracker* memory_tracker) {
-  // TODO(https://crbug.com/899905): Use a real MemoryTracker, not nullptr.
   return std::make_unique<gpu::SharedImageRepresentationFactory>(
       deps->GetSharedImageManager(), memory_tracker);
 }