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

Issue 617974 link

Starred by 1 user

Issue metadata

Status: WontFix
Owner:
Closed: Oct 2016
Cc:
Components:
EstimatedDays: ----
NextAction: ----
OS: ----
Pri: 2
Type: Bug



Sign in to add a comment

Investigate AsyncLayoutInflater for background layout inflation.

Project Member Reported by lizeb@chromium.org, Jun 7 2016

Issue description

Currently, WarmupManager inflates the view hierarchy for Custom Tabs in the background, with a workaround when it throws an exception.

There is now AsyncLayoutInflater in the Android Support library. This should be a safer alternative.

Investigate using it.

 

Comment 1 by lizeb@chromium.org, Oct 18 2016

Status: Assigned (was: Untriaged)
It's not possible to use it as of now at least.

Reason:
android.view.InflateException: <merge /> can be used only with a valid ViewGroup root and attachRoot=true.

and from the AsyncLayoutInflater doc:
NOTE that the inflated View hierarchy is NOT added to the parent. It is equivalent to calling inflate(int, ViewGroup, boolean) with attachToRoot set to false. Callers will likely want to call addView(View) in the AsyncLayoutInflater.OnInflateFinishedListener callback at a minimum.

Comment 2 by lizeb@chromium.org, Oct 18 2016

Status: WontFix (was: Assigned)

Sign in to add a comment