New issue
Advanced search Search tips

Issue 682982 link

Starred by 2 users

Issue metadata

Status: Archived
Owner: ----
Closed: Jan 10
EstimatedDays: ----
NextAction: ----
OS: ----
Pri: 3
Type: Feature



Sign in to add a comment

Replace usage of internals and testRunner in LayoutTests with SimTests

Project Member Reported by esprehn@chromium.org, Jan 20 2017

Issue description

Most LayoutTests which use internals or testRunner methods (besides the test system ones like dumpAsText()) should be rewritten as unit tests using SimTests. 
 
Project Member

Comment 2 by bugdroid1@chromium.org, Mar 23 2017

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

commit 415f9bab260d050f3471daae4282a267aa954094
Author: esprehn <esprehn@chromium.org>
Date: Thu Mar 23 07:53:03 2017

Switch Shadow DOM V0 <content> FeatureSet tests to SimTest.

This lets us remove three methods from internals. Once we have this new SimTest
we can also port lots of other tests and remove more internals methods specific
to testing shadow dom.

BUG= 682982 

Review-Url: https://codereview.chromium.org/2647843002
Cr-Commit-Position: refs/heads/master@{#459018}

[delete] https://crrev.com/dee8e0a41990efbaec2435faff764baee5b894d9/third_party/WebKit/LayoutTests/fast/dom/shadow/shadow-select-attribute-featureset-expected.txt
[delete] https://crrev.com/dee8e0a41990efbaec2435faff764baee5b894d9/third_party/WebKit/LayoutTests/fast/dom/shadow/shadow-select-attribute-featureset.html
[modify] https://crrev.com/415f9bab260d050f3471daae4282a267aa954094/third_party/WebKit/Source/core/dom/Element.h
[modify] https://crrev.com/415f9bab260d050f3471daae4282a267aa954094/third_party/WebKit/Source/core/testing/Internals.cpp
[modify] https://crrev.com/415f9bab260d050f3471daae4282a267aa954094/third_party/WebKit/Source/core/testing/Internals.h
[modify] https://crrev.com/415f9bab260d050f3471daae4282a267aa954094/third_party/WebKit/Source/core/testing/Internals.idl
[modify] https://crrev.com/415f9bab260d050f3471daae4282a267aa954094/third_party/WebKit/Source/web/BUILD.gn
[add] https://crrev.com/415f9bab260d050f3471daae4282a267aa954094/third_party/WebKit/Source/web/tests/ShadowDOMV0Test.cpp

Status: Archived (was: Untriaged)
Archiving P3s older than 1 year with no owner or component.

Sign in to add a comment