Commit dddb9eae authored by arthursonzogni's avatar arthursonzogni Committed by Commit Bot

Cleanup NavigationLoaderOnUI comments.

NavigationLoaderOnUI landed. Some comments about it are still there.
This patch remove them.

This disambiguate the fact this variable can be safely used:
NavigationURLLoaderImpl::URLLoaderRequestController::browser_context_

Bug: 824840
Change-Id: Icb6dc5050a604878531958acbc0fdc2a7739d420
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2120717
Auto-Submit: Arthur Sonzogni <arthursonzogni@chromium.org>
Commit-Queue: Kinuko Yasuda <kinuko@chromium.org>
Reviewed-by: default avatarKinuko Yasuda <kinuko@chromium.org>
Cr-Commit-Position: refs/heads/master@{#753552}
parent 2ad372ba
...@@ -18,9 +18,9 @@ ...@@ -18,9 +18,9 @@
namespace content { namespace content {
// When NavigationLoaderOnUI is enabled, the CrossOriginReadBlockingChecker // The CrossOriginReadBlockingChecker lives on the UI thread, but blobs must be
// lives on the UI thread, but blobs must be read on IO. This class handles all // read on IO. This class handles all blob access for
// blob access for CrossOriginReadBlockingChecker. // CrossOriginReadBlockingChecker.
class CrossOriginReadBlockingChecker::BlobIOState { class CrossOriginReadBlockingChecker::BlobIOState {
public: public:
BlobIOState(base::WeakPtr<CrossOriginReadBlockingChecker> checker, BlobIOState(base::WeakPtr<CrossOriginReadBlockingChecker> checker,
......
...@@ -1242,7 +1242,6 @@ class NavigationURLLoaderImpl::URLLoaderRequestController ...@@ -1242,7 +1242,6 @@ class NavigationURLLoaderImpl::URLLoaderRequestController
// Counts the time overhead of all the hops from the UI to the IO threads. // Counts the time overhead of all the hops from the UI to the IO threads.
base::TimeDelta ui_to_io_time_; base::TimeDelta ui_to_io_time_;
// Only used when NavigationLoaderOnUI is enabled:
BrowserContext* browser_context_; BrowserContext* browser_context_;
network::mojom::URLResponseHeadPtr head_; network::mojom::URLResponseHeadPtr head_;
......
Markdown is supported
0%
or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment