LGTM3
/Daniel
On 2023-06-07 17:37, Philip Jägenstedt wrote:
LGTM2
On Wed, Jun 7, 2023 at 5:36 PM Chris Harrelson <chris...@chromium.org>
wrote:
Thanks! LGTM1
On Wed, Jun 7, 2023 at 6:55 AM Ari Chivukula
<aric...@chromium.org> wrote:
I'm back! The code has been merged behind a flag w/ wpts:
https://chromium-review.googlesource.com/c/chromium/src/+/4538244
The spec changes have been merged too:
https://github.com/w3c/navigation-timing/pull/188
https://github.com/WICG/client-hints-infrastructure/pull/153
~ Ari Chivukula (Their/There/They're)
On Wed, Apr 19, 2023 at 12:02 PM Rick Byers
<rby...@chromium.org> wrote:
Great, thanks Ari! No worries at all, checking to make
sure the i's have been dotted and t's have been crossed is
what the API owners are here for :-)
On Wed, Apr 19, 2023 at 11:59 AM Ari Chivukula
<aric...@chromium.org> wrote:
Works for me, didn't realize that was the standard order.
~ Ari Chivukula (Their/There/They're)
On Wed, Apr 19, 2023 at 11:58 AM Rick Byers
<rby...@chromium.org> wrote:
Hi Ari,
The documented process
<https://www.chromium.org/blink/launching-features/#new-feature-prepare-to-ship>
is
to land features and specs before requesting
approval to ship, as that gives the best possible
chance for wide review and eventual interop. But
we unfortunately often find ourselves in the
position that this is impractical - eg. spec work
blocked from landing due to lack of interest from
a 2nd implementor (WHATWG venue) or just very slow
reviews. So, if necessary, we are willing to
approve based on a PR only but it's usually
considered an exceptional situation. None of those
seem to be the case here, it's just blocked on
some impl and spec work right? In particular, we
definitely want to see WPTs landed prior to
shipping, so we're aligned with Noam on that
request. chrishtr, foolip, yoav and I just
discussed this now in the API owners meeting and
are aligned on this being standard practice
(though wouldn't be surprised if we've slipped up
occasionally).
Thanks,
Rick
On Wed, Apr 19, 2023 at 11:34 AM Ari Chivukula
<aric...@chromium.org> wrote:
That differs from my past experience (in which
a proposed spec change is sufficient to move
forward with a launch), but if you want me to
build and test it all behind a flag before I
get the go ahead here that's fine.
~ Ari Chivukula (Their/There/They're)
On Wed, Apr 19, 2023 at 11:32 AM Rick Byers
<rby...@chromium.org> wrote:
This makes a lot of sense to me. Normally
we prefer to approve shipping only after
spec PRs have landed, it looks like the PR
is blocked on some work from you including
landing the WPTs, right? I'd suggest you
get this all landed under a
status=experimental RuntimeEnabledFeature
and see if we can get the PR reviewed and
landed in short order before trying to
flip to status=stable, WDYT?
Rick
On Wed, Apr 5, 2023 at 2:46 PM 'Ari
Chivukula' via blink-dev
<blink-dev@chromium.org> wrote:
https://github.com/w3ctag/design-reviews/issues/833
On Wednesday, April 5, 2023 at
11:27:22 AM UTC-4 Alex Russell wrote:
Please file an issue with the TAG
as an FYI and let us know when
they respond. Their job is to help
us drive coherence across the
platform and spot architectural
patterns. There might be other
occurances of similar oversights
that they can help us spot and fix.
Best,
Alex
On Tuesday, March 28, 2023 at
11:15:31 AM UTC-7 Ari Chivukula wrote:
Contact emails
ari...@chromium.org,
mike...@chromium.org,
yoav...@chromium.org
Specification
https://github.com/w3c/navigation-timing/pull/188
<https://github.com/w3c/navigation-timing/pull/188>
Summary
Websites can indicate that a
particular Client Hint
<https://wicg.github.io/client-hints-infrastructure/>is
critical to the page by
including it in a
`Critical-CH` HTTP response
header. Doing so will trigger
a connection restart if the
hint listed in the
`Critical-CH` HTTP response
header could be (but wasn’t)
included in the HTTP request
initially sent.This intent
proposes adding `readonly
attribute DOMHighResTimeStamp
criticalCHRestart;` to the
`PerformanceNavigationTiming`
interface.
Blink component
Blink>PerformanceAPIs>NavigationTiming
<https://bugs.chromium.org/p/chromium/issues/list?q=component%3ABlink%3EPerformanceAPIs%3ENavigationTiming>
Motivation
It’s possible for websites to
observe a change in the
Navigation Timing
<https://github.com/w3c/navigation-timing>related
to `responseStart` and
`requestStart`, but there is
no explicit timestamp that
indicates when this occurred.
Without that it’s harder for
sites to detect the impact
that the `Critical-CH` HTTP
response header has on
connections. Developers
(including Akamai) are
interested in using this to
better understand the impact
of User Agent Client Hints /
User Agent Reduction.
`criticalCHRestart` can only
be set for top-level frames,
as subresources and iframes
ignore `Critical-CH` HTTP
response headers.
TAG review
N/A, this is a small,
incremental addition to an
existing feature for
convenience and the loaded
website has other (less
convenient) ways to detect a
Critical-CH restart.
Compatibility &
Interoperability
We would be the first to
implement if approved.
Gecko:
https://github.com/mozilla/standards-positions/issues/767
<https://github.com/mozilla/standards-positions/issues/767>
WebKit:
https://github.com/WebKit/standards-positions/issues/156
<https://github.com/WebKit/standards-positions/issues/156>
Web Developers: Requested by
Akamai
<https://github.com/w3c/navigation-timing/issues/177>
Is this feature fully tested
by web-platform-tests?
No, but it will be.
Tracking bug
https://crbug.com/1425910
<https://crbug.com/1425910>
Link to entry on the Chrome
Platform Status
https://chromestatus.com/feature/5140317616996352
<https://chromestatus.com/feature/5140317616996352>
--
You received this message because you
are subscribed to the Google Groups
"blink-dev" group.
To unsubscribe from this group and
stop receiving emails from it, send an
email to
blink-dev+unsubscr...@chromium.org.
To view this discussion on the web
visit
https://groups.google.com/a/chromium.org/d/msgid/blink-dev/0dacf540-c8be-4096-9ad1-bc7a99af630cn%40chromium.org
<https://groups.google.com/a/chromium.org/d/msgid/blink-dev/0dacf540-c8be-4096-9ad1-bc7a99af630cn%40chromium.org?utm_medium=email&utm_source=footer>.
--
You received this message because you are subscribed to the
Google Groups "blink-dev" group.
To unsubscribe from this group and stop receiving emails from
it, send an email to blink-dev+unsubscr...@chromium.org.
To view this discussion on the web visit
https://groups.google.com/a/chromium.org/d/msgid/blink-dev/CAGpy5DKthEYQEYGqoFyVWsp0xnm24QLEJfEoeZz9ZKB39PKtkg%40mail.gmail.com
<https://groups.google.com/a/chromium.org/d/msgid/blink-dev/CAGpy5DKthEYQEYGqoFyVWsp0xnm24QLEJfEoeZz9ZKB39PKtkg%40mail.gmail.com?utm_medium=email&utm_source=footer>.
--
You received this message because you are subscribed to the Google
Groups "blink-dev" group.
To unsubscribe from this group and stop receiving emails from it,
send an email to blink-dev+unsubscr...@chromium.org.
To view this discussion on the web visit
https://groups.google.com/a/chromium.org/d/msgid/blink-dev/CAOMQ%2Bw8xuuVTFCSKCyvqu-_7mnR6xpdzqbKkVd8T48S30rhJsQ%40mail.gmail.com
<https://groups.google.com/a/chromium.org/d/msgid/blink-dev/CAOMQ%2Bw8xuuVTFCSKCyvqu-_7mnR6xpdzqbKkVd8T48S30rhJsQ%40mail.gmail.com?utm_medium=email&utm_source=footer>.
--
You received this message because you are subscribed to the Google
Groups "blink-dev" group.
To unsubscribe from this group and stop receiving emails from it, send
an email to blink-dev+unsubscr...@chromium.org.
To view this discussion on the web visit
https://groups.google.com/a/chromium.org/d/msgid/blink-dev/CAARdPYfz2QM8hj_1GdYL%2BwGhey9HPjAAPcMwmJLUEkm697n2Sw%40mail.gmail.com
<https://groups.google.com/a/chromium.org/d/msgid/blink-dev/CAARdPYfz2QM8hj_1GdYL%2BwGhey9HPjAAPcMwmJLUEkm697n2Sw%40mail.gmail.com?utm_medium=email&utm_source=footer>.
--
You received this message because you are subscribed to the Google Groups
"blink-dev" group.
To unsubscribe from this group and stop receiving emails from it, send an email
to blink-dev+unsubscr...@chromium.org.
To view this discussion on the web visit
https://groups.google.com/a/chromium.org/d/msgid/blink-dev/75ec0c4b-5590-3872-4028-438543406a6f%40gmail.com.