New issue
Advanced search Search tips

Issue 800691 link

Starred by 2 users

Issue metadata

Status: Fixed
Owner:
Closed: Apr 2018
Components:
EstimatedDays: ----
NextAction: ----
OS: Android
Pri: 3
Type: Bug



Sign in to add a comment

enable perfetto builds in chrome also for is_android

Project Member Reported by primiano@chromium.org, Jan 10 2018

Issue description

I guess just an oversight. I was looking at build logs and couldn't figure out why perfetto didn't show up. Turns out that right now the root BUILD.gn says:

  if (is_mac || is_linux) {
    deps += [ "//third_party/perfetto:all" ]
  }

I guess there is an is_android missing :)

 
Yep turns out there's still some BUILD.gn changes needed in perfetto to make this work; I figured landing in third_party first and then fixing up those after would be good.
(Specifically it's the Android test_task_runner which doesn't compile).
Ahh makes sense if it was deliberate. No worries then, it's not urgent.
P.s. I did split out the Android task runner yesterday off base. I think we can exclude it from the unit test via ifdef that on chrome. That is only meant to integrate in Android services that must use a looper. Chrome is going to have its own task runner that wraps base's. Or if we want to build perfetto standalone we can just use the UnixTaskRunner which uses no external libraries 
Status: Fixed (was: Untriaged)

Sign in to add a comment