All graphs for this bug:
  https://chromeperf.appspot.com/group_report?bug_id=827602

(For debugging:) Original alerts at time of bug-filing:
  https://chromeperf.appspot.com/group_report?sid=bb42189d84253c08c22fe70bdfd17ece28019dbc8595f67fc69da46d745cee26


Bot(s) for this bug's original alert(s):

android-nexus5

📍 Pinpoint job started.
https://pinpoint-dot-chromeperf.appspot.com/job/15ddff07440000

📍 Found significant differences after each of 3 commits.
https://pinpoint-dot-chromeperf.appspot.com/job/15ddff07440000

Revert "Android: Add code to parse the CFI info added to the apk" by fdoray@chromium.org
https://chromium.googlesource.com/chromium/src/+/88328b6800f987ba9f532842862021cb48875618

Revert "Enable extracting unwind table on official builds without channel" by agrieve@chromium.org
https://chromium.googlesource.com/chromium/src/+/a2474b6490ea9cfda76528059fbf23369e7d281f

Do not change media notification icon multiple times by ssid@chromium.org
https://chromium.googlesource.com/chromium/src/+/a91837ee530b963d208e9f08f5a657e649a5323e

Understanding performance regressions:
  http://g.co/ChromePerformanceRegressions

hm it's mostly the media icon CL:
https://chromium.googlesource.com/chromium/src/+/a91837ee530b963d208e9f08f5a657e649a5323e

+media benchmark owners. I ran this test locally with and without the CL. I cannot reproduce the regression running locally on nexus 5.

For both cases the time to start was ~110ms. The benchmark graphs shows it regressed from 110 to ~200ms.

I ran on ChromePublic.apk with gn args:
ffmpeg_branding = "Chrome"
is_chrome_branded = true
is_component_build = false
is_debug = false
is_official_build = true
proprietary_codecs = true
symbol_level = 1
target_cpu = "arm"
target_os = "android"
use_goma = true

Actually I think I figured the issue. It was just timing difference because I made the notification async.. I will fix it.

Great. Feel free to follow up with liberato@ with questions.

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

commit 42ffcfbbe189e6395e799657c4e6ce163a4edc22
Author: Siddhartha <ssid@chromium.org>
Date: Fri Apr 20 22:53:07 2018

Media notification: If the current tab does not have favicon, don't try to fetch

If a page does not have any favicon, do not try to fetch an icon from
large icon bridge and wait for callback to update the notification icon.
We know if the page has favicon or not by tracking if onFaviconUpdated
was ever called.

BUG= 827602 

Change-Id: I3356470ca7088871540360621ee3140a3304621b
Reviewed-on: https://chromium-review.googlesource.com/1000314
Reviewed-by: Mounir Lamouri <mlamouri@chromium.org>
Commit-Queue: Siddhartha S <ssid@chromium.org>
Cr-Commit-Position: refs/heads/master@{#552506}
[modify] https://crrev.com/42ffcfbbe189e6395e799657c4e6ce163a4edc22/chrome/android/java/src/org/chromium/chrome/browser/media/ui/MediaSessionTabHelper.java
[modify] https://crrev.com/42ffcfbbe189e6395e799657c4e6ce163a4edc22/chrome/android/junit/src/org/chromium/chrome/browser/media/ui/MediaNotificationFaviconTest.java

Most graphs came down except some stories on youtube seem to be disabled / not running. So, can't verify.