catchpoint/WebPageTest.RUM-SpeedIndex

Reports `firstPaint` as SpeedIndex in current Chrome

Closed this issue · 1 comments

lyio commented

The Regex var re = /url\((http.*)\)/ig; does not account for additional " that Chrome has in the object returned by window.getComputedStyles.

The value of the field background-image is url("https://localhost/Fioport.Vermarktung.Demo/Fioport.Vermarktung.Frontend/assets/img/login_1920x1080.jpg") in Chrome 52 and 53 whereas it is url(https://localhost/Fioport.Vermarktung.Demo/Fioport.Vermarktung.Frontend/assets/img/login_1920x1080.jpg) in Edge 25 (and I guess older versions of Chrome).

I replaced the " with an empty string like this CheckElement(el, matches[1].replace('"', '')); and got similar results in Edge and Chrome.

I am reluctant to open a PR for this as this project has not seen any updates in a long time.

Fixed with your PR. Thanks.