Commit 1353bd52 authored by wnwen's avatar wnwen Committed by Commit bot

Fix NPE when favicon is updated.

BUG=567299

Review URL: https://codereview.chromium.org/1508813002

Cr-Commit-Position: refs/heads/master@{#364494}
parent 1bcb1dae
...@@ -2102,6 +2102,7 @@ public class Tab implements ViewGroup.OnHierarchyChangeListener, ...@@ -2102,6 +2102,7 @@ public class Tab implements ViewGroup.OnHierarchyChangeListener,
@CalledByNative @CalledByNative
protected void onFaviconAvailable(Bitmap icon) { protected void onFaviconAvailable(Bitmap icon) {
if (icon == null) return;
String url = getUrl(); String url = getUrl();
boolean pageUrlChanged = !url.equals(mFaviconUrl); boolean pageUrlChanged = !url.equals(mFaviconUrl);
// This method will be called multiple times if the page has more than one favicon. // This method will be called multiple times if the page has more than one favicon.
......
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