[GitHub] rnlw commented on issue #1537: DNS prefetch function uses "chrome.loadTimes" instead of the API Paint Timing

2018-10-14 Thread GitBox
rnlw commented on issue #1537: DNS prefetch function uses "chrome.loadTimes" instead of the API Paint Timing URL: https://github.com/apache/incubator-pagespeed-ngx/issues/1537#issuecomment-429604276 OK so, after 6 month, i think that i found the filter who cause this. I updated

[GitHub] injust commented on a change in pull request #1802: Prefer Paint Timing API over chrome.loadTimes()

2018-10-14 Thread GitBox
injust commented on a change in pull request #1802: Prefer Paint Timing API over chrome.loadTimes() URL: https://github.com/apache/incubator-pagespeed-mod/pull/1802#discussion_r224996956 ## File path: net/instaweb/rewriter/add_instrumentation.js ## @@ -95,6 +95,12 @@

[GitHub] injust opened a new pull request #1802: Prefer Paint Timing API over chrome.loadTimes()

2018-10-14 Thread GitBox
injust opened a new pull request #1802: Prefer Paint Timing API over chrome.loadTimes() URL: https://github.com/apache/incubator-pagespeed-mod/pull/1802 Fixes apache/incubator-pagespeed-ngx#1537 This is an automated message

[GitHub] injust commented on a change in pull request #1802: Prefer Paint Timing API over chrome.loadTimes()

2018-10-14 Thread GitBox
injust commented on a change in pull request #1802: Prefer Paint Timing API over chrome.loadTimes() URL: https://github.com/apache/incubator-pagespeed-mod/pull/1802#discussion_r224996956 ## File path: net/instaweb/rewriter/add_instrumentation.js ## @@ -95,6 +95,12 @@

[GitHub] injust edited a comment on issue #1537: add_instrumentation filter uses "chrome.loadTimes" instead of the API Paint Timing

2018-10-14 Thread GitBox
injust edited a comment on issue #1537: add_instrumentation filter uses "chrome.loadTimes" instead of the API Paint Timing URL: https://github.com/apache/incubator-pagespeed-ngx/issues/1537#issuecomment-370173577 The `insert_dns_prefetch` filter should not exhibit this behaviour. All it

[GitHub] rnlw edited a comment on issue #1537: add_instrumentation filter uses "chrome.loadTimes" instead of the API Paint Timing

2018-10-14 Thread GitBox
rnlw edited a comment on issue #1537: add_instrumentation filter uses "chrome.loadTimes" instead of the API Paint Timing URL: https://github.com/apache/incubator-pagespeed-ngx/issues/1537#issuecomment-429604276 OK so, after 6 month, i think that i found the filter who cause this. I

[GitHub] jmarantz commented on a change in pull request #1802: Prefer Paint Timing API over chrome.loadTimes()

2018-10-14 Thread GitBox
jmarantz commented on a change in pull request #1802: Prefer Paint Timing API over chrome.loadTimes() URL: https://github.com/apache/incubator-pagespeed-mod/pull/1802#discussion_r224992690 ## File path: net/instaweb/rewriter/add_instrumentation.js ## @@ -95,6 +95,12 @@

[GitHub] jmarantz commented on a change in pull request #1802: Prefer Paint Timing API over chrome.loadTimes()

2018-10-14 Thread GitBox
jmarantz commented on a change in pull request #1802: Prefer Paint Timing API over chrome.loadTimes() URL: https://github.com/apache/incubator-pagespeed-mod/pull/1802#discussion_r224992690 ## File path: net/instaweb/rewriter/add_instrumentation.js ## @@ -95,6 +95,12 @@

[GitHub] jmarantz commented on a change in pull request #1802: Prefer Paint Timing API over chrome.loadTimes()

2018-10-14 Thread GitBox
jmarantz commented on a change in pull request #1802: Prefer Paint Timing API over chrome.loadTimes() URL: https://github.com/apache/incubator-pagespeed-mod/pull/1802#discussion_r224992740 ## File path: net/instaweb/rewriter/add_instrumentation.js ## @@ -95,6 +95,12 @@

[GitHub] jmarantz commented on a change in pull request #1802: Prefer Paint Timing API over chrome.loadTimes()

2018-10-14 Thread GitBox
jmarantz commented on a change in pull request #1802: Prefer Paint Timing API over chrome.loadTimes() URL: https://github.com/apache/incubator-pagespeed-mod/pull/1802#discussion_r224992613 ## File path: net/instaweb/rewriter/add_instrumentation.js ## @@ -95,6 +95,12 @@

[GitHub] injust commented on a change in pull request #1802: Prefer Paint Timing API over chrome.loadTimes()

2018-10-14 Thread GitBox
injust commented on a change in pull request #1802: Prefer Paint Timing API over chrome.loadTimes() URL: https://github.com/apache/incubator-pagespeed-mod/pull/1802#discussion_r225034179 ## File path: net/instaweb/rewriter/add_instrumentation.js ## @@ -71,11 +71,11 @@

[GitHub] jmarantz commented on a change in pull request #1802: Prefer Paint Timing API over chrome.loadTimes()

2018-10-14 Thread GitBox
jmarantz commented on a change in pull request #1802: Prefer Paint Timing API over chrome.loadTimes() URL: https://github.com/apache/incubator-pagespeed-mod/pull/1802#discussion_r225014034 ## File path: net/instaweb/rewriter/add_instrumentation.js ## @@ -71,11 +71,11 @@

[GitHub] jmarantz commented on a change in pull request #1802: Prefer Paint Timing API over chrome.loadTimes()

2018-10-14 Thread GitBox
jmarantz commented on a change in pull request #1802: Prefer Paint Timing API over chrome.loadTimes() URL: https://github.com/apache/incubator-pagespeed-mod/pull/1802#discussion_r225014000 ## File path: net/instaweb/rewriter/add_instrumentation.js ## @@ -71,11 +71,11 @@