Skip to content

  • Projects
  • Groups
  • Snippets
  • Help
    • Loading...
    • Help
    • Contribute to GitLab
  • Sign in / Register
V
V8
  • Project
    • Project
    • Details
    • Activity
    • Cycle Analytics
  • Repository
    • Repository
    • Files
    • Commits
    • Branches
    • Tags
    • Contributors
    • Graph
    • Compare
    • Charts
  • Issues 0
    • Issues 0
    • List
    • Board
    • Labels
    • Milestones
  • Merge Requests 0
    • Merge Requests 0
  • CI / CD
    • CI / CD
    • Pipelines
    • Jobs
    • Schedules
    • Charts
  • Wiki
    • Wiki
  • Snippets
    • Snippets
  • Members
    • Members
  • Collapse sidebar
  • Activity
  • Graph
  • Charts
  • Create a new issue
  • Jobs
  • Commits
  • Issue Boards
  • Linshizhi
  • V8
  • Repository

Switch branch/tag
  • v8
  • test
  • mjsunit
  • es6
  • computed-property-names-classes.js
Find file
BlameHistoryPermalink
  • Timothy Gu's avatar
    Install class "name" accessor before methods · 224b659c
    Timothy Gu authored Jun 23, 2021
    https://github.com/tc39/ecma262/pull/1490 changed the spec so that the
    "name" property of a class should be installed after "length" but before
    "prototype". This CL adapts accordingly.
    
    After this change, there is now no need for the separate code path to
    set the "name" accessor at runtime. Delete the relevant runtime code as
    well.
    
    Bug: v8:8771
    Change-Id: I8f809b45bf209c899cf5df76d0ebf6d9a45a6d4e
    Reviewed-on: https://chromium-review.googlesource.com/c/v8/v8/+/2974772
    Commit-Queue: Timothy Gu <timothygu@chromium.org>
    Reviewed-by: 's avatarMarja Hölttä <marja@chromium.org>
    Cr-Commit-Position: refs/heads/master@{#75340}
    224b659c
computed-property-names-classes.js 15 KB
EditWeb IDE

Replace computed-property-names-classes.js

Attach a file by drag & drop or click to upload


Cancel
A new branch will be created in your fork and a new merge request will be started.