New issue
Advanced search Search tips

Issue 893469 link

Starred by 2 users

Issue metadata

Status: Fixed
Owner:
Closed: Oct 17
Cc:
Components:
EstimatedDays: ----
NextAction: ----
OS: ----
Pri: ----
Type: ----

Blocked on:
issue v8:8267
issue v8:8300



Sign in to add a comment

Harmony await optimization

Project Member Reported by mslekova@chromium.org, Oct 9

Issue description

Feature description: This change is supposed to reflect a recent Normative change in the spec, reducing the number of ticks "await" takes on the microtask queue from 3 to 1.

Eng owner: mslekova@chromium.org
Product owner: hablich@chromium.org

Design doc: https://docs.google.com/document/d/1kL08cz4lR6gO5b2FATNK3QAfS8t-6K6kdk88U-n8tug/edit

Are you planning on experimenting before launch? Yes
Any new strings? No
Any implications for Google webservices (i.e. sync, translate)? No
Binary size? No change
Do the existing perf tests exercise all aspects of your new feature(s)? Yes

Note: This bug is a copy of https://bugs.chromium.org/p/v8/issues/detail?id=8267 used for referencing in the finch experiment config. 
 
Blockedon: v8:8276
Blockedon: v8:8267
Blockedon: -v8:8276
Blockedon: v8:8300
Project Member

Comment 5 by bugdroid1@chromium.org, Oct 16

The following revision refers to this bug:
  https://chromium.googlesource.com/chromium/src.git/+/4bbfdb7f7387670c9e7ce4bbccdbe616d738e44b

commit 4bbfdb7f7387670c9e7ce4bbccdbe616d738e44b
Author: Maya Lekova <mslekova@chromium.org>
Date: Tue Oct 16 08:15:03 2018

[async-await] Ship await optimization behind a flag

This adds both finch and about:flags flag.

Intent to ship discussions:
https://groups.google.com/forum/#!topic/v8-users/ML7gOgK8Dsg
https://groups.google.com/a/chromium.org/forum/#!topic/blink-dev/74QxqINq4g0

Bug:  v8:8267 ,  chromium:893469 
Change-Id: Ib5a84519d48b9b82604875ff269f0e04977914d5
Reviewed-on: https://chromium-review.googlesource.com/c/1268200
Commit-Queue: Maya Lekova <mslekova@chromium.org>
Reviewed-by: Jochen Eisinger <jochen@chromium.org>
Reviewed-by: Michael Hablich <hablich@chromium.org>
Reviewed-by: Kentaro Hara <haraken@chromium.org>
Cr-Commit-Position: refs/heads/master@{#599909}
[modify] https://crrev.com/4bbfdb7f7387670c9e7ce4bbccdbe616d738e44b/chrome/browser/about_flags.cc
[modify] https://crrev.com/4bbfdb7f7387670c9e7ce4bbccdbe616d738e44b/chrome/browser/flag_descriptions.cc
[modify] https://crrev.com/4bbfdb7f7387670c9e7ce4bbccdbe616d738e44b/chrome/browser/flag_descriptions.h
[modify] https://crrev.com/4bbfdb7f7387670c9e7ce4bbccdbe616d738e44b/content/public/common/content_features.cc
[modify] https://crrev.com/4bbfdb7f7387670c9e7ce4bbccdbe616d738e44b/content/public/common/content_features.h
[modify] https://crrev.com/4bbfdb7f7387670c9e7ce4bbccdbe616d738e44b/content/renderer/render_process_impl.cc
[modify] https://crrev.com/4bbfdb7f7387670c9e7ce4bbccdbe616d738e44b/tools/metrics/histograms/enums.xml

Labels: Launch-Status-Approval-Requested
Labels: -Launch-Status-Approval-Requested
Status: Fixed (was: Assigned)
Both the server and the client side of the config are submitted.

Sign in to add a comment