Remove stale TODO

This trace event is still actively used by catapult, and since network
instrument v2 project is no longer being worked on, we should keep this
event.

Change-Id: Ia61df9fd01ecbb8bdc440cf76b826fc8e7fb7d68
Reviewed-on: https://chromium-review.googlesource.com/c/1286872Reviewed-by: Matt Falkenhagen <falken@chromium.org>
Reviewed-by: Yutaka Hirano <yhirano@chromium.org>
Commit-Queue: Deepanjan Roy <dproy@chromium.org>
Cr-Commit-Position: refs/heads/master@{#601594}

third_party/blink/renderer/platform/loader/fetch/resource_fetcher.cc

@@ -788,7 +788,6 @@ Resource* ResourceFetcher::RequestResource(
       scoped_resource_load_tracker(identifier, resource_request);
   SCOPED_BLINK_UMA_HISTOGRAM_TIMER_THREAD_SAFE(
       "Blink.Fetch.RequestResourceTime");
-  // TODO(dproy): Remove this. http://crbug.com/659666
   TRACE_EVENT1("blink", "ResourceFetcher::requestResource", "url",
                UrlForTraceEvent(params.Url()));