Bug 189089 - Remove PseudoElementUserAgentCustom.
Summary: Remove PseudoElementUserAgentCustom.
Status: RESOLVED FIXED
Alias: None
Product: WebKit
Classification: Unclassified
Component: CSS (show other bugs)
Version: WebKit Nightly Build
Hardware: Unspecified Unspecified
: P2 Normal
Assignee: Emilio Cobos Álvarez (:emilio)
URL:
Keywords: InRadar
Depends on:
Blocks:
 
Reported: 2018-08-29 10:29 PDT by Emilio Cobos Álvarez (:emilio)
Modified: 2018-09-04 18:24 PDT (History)
9 users (show)

See Also:


Attachments
Patch (6.21 KB, patch)
2018-08-29 11:08 PDT, Emilio Cobos Álvarez (:emilio)
rniwa: review+
commit-queue: commit-queue-
Details | Formatted Diff | Diff

Note You need to log in before you can comment on or make changes to this bug.
Description Emilio Cobos Álvarez (:emilio) 2018-08-29 10:29:08 PDT
Looks like it was added in bug 100826, but seems unused now, I couldn't find anything that set the pseudo attribute to something that wasn't -webkit- stuff, so this doesn't match anything.

I hope we can remove it without issues.
Comment 1 Emilio Cobos Álvarez (:emilio) 2018-08-29 10:53:37 PDT
Test at https://github.com/web-platform-tests/wpt/pull/12743.
Comment 2 Emilio Cobos Álvarez (:emilio) 2018-08-29 11:08:53 PDT
Created attachment 348416 [details]
Patch
Comment 3 Darin Adler 2018-09-01 22:04:04 PDT
Ryosuke, would you review this patch?
Comment 4 Myles C. Maxfield 2018-09-04 16:59:18 PDT
Looks good to me, but I'll let Ryosuke give the final review.
Comment 5 Ryosuke Niwa 2018-09-04 17:02:29 PDT
Comment on attachment 348416 [details]
Patch

Sure, this looks unused now. ::part(~) would be a different implementation anyway.
Comment 6 Ryosuke Niwa 2018-09-04 17:06:05 PDT
Note that this is a potentially Web incompatible change if some websites had been relying on these pseudo element names being valid in some of their rules. But the chance of it happening is low enough that I'm comfortable making this change given this isn't a feature ever exposed to the Web.
Comment 7 Emilio Cobos Álvarez (:emilio) 2018-09-04 18:16:35 PDT
(In reply to Ryosuke Niwa from comment #6)
> Note that this is a potentially Web incompatible change if some websites had
> been relying on these pseudo element names being valid in some of their
> rules. But the chance of it happening is low enough that I'm comfortable
> making this change given this isn't a feature ever exposed to the Web.

Right, that's why I wanted to remove it if possible sooner rather than later.

Thanks a lot for the review!
Comment 8 WebKit Commit Bot 2018-09-04 18:18:38 PDT
Comment on attachment 348416 [details]
Patch

Rejecting attachment 348416 [details] from commit-queue.

Failed to run "['/Volumes/Data/EWS/WebKit/Tools/Scripts/webkit-patch', '--status-host=webkit-queues.webkit.org', '--bot-id=webkit-cq-01', 'apply-attachment', '--no-update', '--non-interactive', 348416, '--port=mac']" exit_code: 2 cwd: /Volumes/Data/EWS/WebKit

Logging in as commit-queue@webkit.org...
Fetching: https://bugs.webkit.org/attachment.cgi?id=348416&action=edit
Fetching: https://bugs.webkit.org/show_bug.cgi?id=189089&ctype=xml&excludefield=attachmentdata
Processing 1 patch from 1 bug.
Processing patch 348416 from bug 189089.
Fetching: https://bugs.webkit.org/attachment.cgi?id=348416
Failed to run "[u'/Volumes/Data/EWS/WebKit/Tools/Scripts/svn-apply', '--force', '--reviewer', u'Ryosuke Niwa']" exit_code: 1 cwd: /Volumes/Data/EWS/WebKit

Parsed 5 diffs from patch file(s).
patching file Source/WebCore/ChangeLog
Hunk #1 FAILED at 1.
1 out of 1 hunk FAILED -- saving rejects to file Source/WebCore/ChangeLog.rej
patching file Source/WebCore/css/CSSSelector.cpp
patching file Source/WebCore/css/CSSSelector.h
patching file Source/WebCore/css/RuleSet.cpp
patching file Source/WebCore/css/parser/CSSParserSelector.h

Failed to run "[u'/Volumes/Data/EWS/WebKit/Tools/Scripts/svn-apply', '--force', '--reviewer', u'Ryosuke Niwa']" exit_code: 1 cwd: /Volumes/Data/EWS/WebKit

Full output: https://webkit-queues.webkit.org/results/9096418
Comment 9 Emilio Cobos Álvarez (:emilio) 2018-09-04 18:23:10 PDT
Committed r235652: <https://trac.webkit.org/changeset/235652>
Comment 10 Radar WebKit Bug Importer 2018-09-04 18:24:18 PDT
<rdar://problem/44121968>