New issue
Advanced search Search tips
Note: Color blocks (like █ or █) mean that a user may not be available. Tooltip shows the reason.

Issue 804302 link

Starred by 1 user
Status: Assigned
Owner:
lizeb@chromium.org
Cc:
█ mge...@chromium.org
pasko@chromium.org
EstimatedDays: ----
NextAction: ----
OS: Android
Pri: 2
Type: Bug



Sign in to add a comment

Make "early DNS" work on Android with AsyncDNS

Project Member Reported by lizeb@chromium.org, Jan 22 2018 
On Android, we issue a DNS request before the native library is loaded, to speed up cold starts with an Intent.

In the past, this was shown in local experiments to significantly reduce page load latency (see https://docs.google.com/document/d/1e2qeg7TK5ciEGJ6C2J8BWQvWScTPGnT5V3Y9WTpu8GU/edit).

This relies on the host DNS cache being shared with Chrome's, which is no longer the case with AsyncDNS.

This bug tracks the work to bring this back.
This may be very lightweight, as there is also early connection, which would trigger a DNS resolution as well.

Comment 1 by mge...@chromium.org, Jan 22 2018 

I'd expect that early requests sometimes still use the system resolver, because it takes a while on Android to get the DNS config and we fall back to the system resolver when there's no config.
Project Member

Comment 2 by bugdroid1@chromium.org, Jan 29 2018 

The following revision refers to this bug:
  https://chromium.googlesource.com/chromium/src.git/+/cbf8fac039fa32d55ec1d3e5ace8aa307d0c605f

commit cbf8fac039fa32d55ec1d3e5ace8aa307d0c605f
Author: Benoit Lize <lizeb@chromium.org>
Date: Mon Jan 29 10:01:54 2018

android: Add a trace event to track early DNS.

Also convert another trace event in WarmupManager to
TraceEvent.scoped().

Bug: 804302
Change-Id: I1e21c4ccfeb8310fe7a412d9cef308bc479320af
Reviewed-on: https://chromium-review.googlesource.com/886845
Reviewed-by: Benoit L <lizeb@chromium.org>
Reviewed-by: Egor Pasko <pasko@chromium.org>
Reviewed-by: Maria Khomenko <mariakhomenko@chromium.org>
Commit-Queue: Benoit L <lizeb@chromium.org>
Cr-Commit-Position: refs/heads/master@{#532376}
[modify] https://crrev.com/cbf8fac039fa32d55ec1d3e5ace8aa307d0c605f/chrome/android/java/src/org/chromium/chrome/browser/WarmupManager.java
►

Sign in to add a comment