OSSCA Chromium

기여 목록 / 8493052

[DevTools] Disable layout tests to land crrev.com/c/8494771

DevToolstestIn Review

security state section의 Lit 전환을 반영하기 위해 보안 웹 테스트 2개의 기대값을 임시 조정했습니다.

문제 설명

CL 8494771의 DOM 변경으로 보안 웹 테스트 2개의 출력이 기존 기대 결과와 달라졌습니다.

해결 내용

TestExpectations에 두 테스트를 [ Failure Pass ]로 등록해 성공과 실패를 모두 허용하도록 했습니다. 대상은 origin-view-noncryptographic-secure-origin.js와 origin-view-then-interstitial.js입니다.

전체 코드 diff는 다음과 같습니다.

diff --git a/third_party/blink/web_tests/TestExpectations b/third_party/blink/web_tests/TestExpectations
index 31aeccd..35db3e6 100644
--- a/third_party/blink/web_tests/TestExpectations
+++ b/third_party/blink/web_tests/TestExpectations
@@ -4501,6 +4501,10 @@

 crbug.com/1380554 external/wpt/content-security-policy/wasm-unsafe-eval/postMessage-wasm-module.html [ Timeout ]

+## Disable layout tests to land crrev.com/c/8494771
+crbug.com/407750721 http/tests/devtools/security/origin-view-noncryptographic-secure-origin.js [ Failure Pass ]
+crbug.com/407750721 http/tests/devtools/security/origin-view-then-interstitial.js [ Failure Pass ]
+
 # ====== Random order flaky tests from here ======
 # These tests are flaky when run in random order, which is the default on Linux & Mac since since 2016-12-16.

테스트 방법

Gerrit에서 patchset 1의 CQ 실행 통과를 확인했습니다. 현재 리뷰 중입니다.

배운 점

DevTools 변경이 Chromium에 반영된 뒤에는 기대 결과를 갱신하고 임시 설정을 제거해야 합니다.

참고 자료