Commit ab09c6a4 authored by Tomasz Wiszkowski's avatar Tomasz Wiszkowski Committed by Commit Bot

Fix glitch with jumping status icon / url text on tablets.

This change addresses problem where no placeholder is reserved on tablets.
Text would appear with no placeholder for status/security icon on first load
and would be followed by presenting the icon within a second, which looks like
a glitch.

Bug: 930306
Change-Id: I20117d4b6bb3fd51c0940de224b5cdd623b8a640
Reviewed-on: https://chromium-review.googlesource.com/c/1461329
Commit-Queue: Ender <ender@google.com>
Auto-Submit: Ender <ender@google.com>
Reviewed-by: default avatarTheresa <twellington@chromium.org>
Cr-Commit-Position: refs/heads/master@{#630611}
parent 2f78a453
...@@ -66,6 +66,10 @@ public class StatusViewCoordinator implements View.OnClickListener { ...@@ -66,6 +66,10 @@ public class StatusViewCoordinator implements View.OnClickListener {
*/ */
public void setToolbarDataProvider(ToolbarDataProvider toolbarDataProvider) { public void setToolbarDataProvider(ToolbarDataProvider toolbarDataProvider) {
mToolbarDataProvider = toolbarDataProvider; mToolbarDataProvider = toolbarDataProvider;
// Update status immediately after receiving the data provider to avoid initial presence
// glitch on tablet devices. This glitch would be typically seen upon launch of app, right
// before the landing page is presented to the user.
updateStatusIcon();
} }
/** /**
......
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