New issue
Advanced search Search tips

Issue 806581 link

Starred by 3 users
Status: Fixed
Owner:
aaronhk@chromium.org
Closed: Aug 31
Cc:
fs...@chromium.org
Components:
EstimatedDays: ----
NextAction: ----
OS: Mac
Pri: 3
Type: Bug



Sign in to add a comment

Cleanup Path2D constructor

Project Member Reported by annevank...@gmail.com, Jan 28 2018 
As per https://github.com/whatwg/html/pull/3409 the IDL can be simplified.

Comment 1 by junov@chromium.org, Feb 19 2018

Owner: fs...@chromium.org
Status: Assigned (was: Unconfirmed)

Comment 2 by fs...@chromium.org, Aug 13

Cc: fs...@chromium.org
Labels: StarterBug
Owner: ----
Status: Available (was: Assigned)

Comment 3 by fs...@chromium.org, Aug 20

Owner: aaronhk@chromium.org
Status: Assigned (was: Available)
Project Member

Comment 4 by bugdroid1@chromium.org, Aug 25 

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

commit 52aca360084f3f7c6a9c73c66f38bb99fd3cf966
Author: Aaron Krajeski <aaronhk@google.com>
Date: Sat Aug 25 03:19:03 2018

Change Path2D interface to take string or path

Bug:  806581 
Change-Id: I26467c5fc2c9fc9ecf918a5229a97650bd29a5e5
Reviewed-on: https://chromium-review.googlesource.com/1185370
Reviewed-by: Fernando Serboncini <fserb@chromium.org>
Reviewed-by: Jeremy Roman <jbroman@chromium.org>
Commit-Queue: Jeremy Roman <jbroman@chromium.org>
Cr-Commit-Position: refs/heads/master@{#586113}
[modify] https://crrev.com/52aca360084f3f7c6a9c73c66f38bb99fd3cf966/third_party/blink/renderer/bindings/modules/v8/generated.gni
[modify] https://crrev.com/52aca360084f3f7c6a9c73c66f38bb99fd3cf966/third_party/blink/renderer/modules/canvas/canvas2d/path_2d.h
[modify] https://crrev.com/52aca360084f3f7c6a9c73c66f38bb99fd3cf966/third_party/blink/renderer/modules/canvas/canvas2d/path_2d.idl
►

Sign in to add a comment