catapult-project / catapult

Deprecated Catapult GitHub. Please instead use http://crbug.com "Speed>Benchmarks" component for bugs and https://chromium.googlesource.com/catapult for downloading and editing source code..
https://chromium.googlesource.com/catapult
BSD 3-Clause "New" or "Revised" License
1.93k stars 564 forks source link

Remove old path for getting URL from navigation start event #4358

Open deepanjanroy opened 6 years ago

deepanjanroy commented 6 years ago

See crbug.com/824761 for more context. We now dump the URL directly in the navigation start event. Currently in catapult we're handling both the new way of getting the URL as well as the old way of getting it from FrameLoader snapshot. The required change in chromium will hit stable in M67. Let's wait one extra version and remove the old path when M68 hits stable.