OSSCA Chromium

기여 목록 / 8457503

[DevTools] Remove blocked-mixed-content test after crrev.com/c/8457645

DevToolstestMerged

CL 8457645에서 unit test로 옮긴 기존 blocked-mixed-content 웹 테스트를 제거했습니다.

문제 설명

unit test로 옮긴 뒤에도 기존 Chromium 웹 테스트가 남아 있었습니다.

해결 내용

blocked-mixed-content.js와 blocked-mixed-content-expected.txt를 제거했습니다.

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

diff --git a/third_party/blink/web_tests/http/tests/devtools/security/blocked-mixed-content-expected.txt b/third_party/blink/web_tests/http/tests/devtools/security/blocked-mixed-content-expected.txt
deleted file mode 100644
index b5df46a3..0000000
--- a/third_party/blink/web_tests/http/tests/devtools/security/blocked-mixed-content-expected.txt
+++ /dev/null
@@ -1,24 +0,0 @@
-Tests active mixed content blocking in the security panel.
-
-<DIV class=security-explanation security-explanation-info >
-    <DEVTOOLS-ICON role=presentation name=indeterminate-question-box class=security-property security-property-info >
-        <#document-fragment >
-            <STYLE >
-            </STYLE>
-            <SPAN style=--icon-url: var(--image-file-indeterminate-question-box); >
-            </SPAN>
-        </#document-fragment>
-    </DEVTOOLS-ICON>
-    <DIV class=security-explanation-text >
-        <DIV class=security-explanation-title >
-Blocked mixed content
-        </DIV>
-        <DIV >
-Your page requested non-secure resources that were blocked.
-        </DIV>
-        <BUTTON class=security-mixed-content devtools-link text-button link-style role=link tabindex=0 >
-View 1 request in Network panel
-        </BUTTON>
-    </DIV>
-</DIV>
-
diff --git a/third_party/blink/web_tests/http/tests/devtools/security/blocked-mixed-content.js b/third_party/blink/web_tests/http/tests/devtools/security/blocked-mixed-content.js
deleted file mode 100644
index d6eb1c9..0000000
--- a/third_party/blink/web_tests/http/tests/devtools/security/blocked-mixed-content.js
+++ /dev/null
@@ -1,34 +0,0 @@
-// Copyright 2017 The Chromium Authors
-// Use of this source code is governed by a BSD-style license that can be
-// found in the LICENSE file.
-
-import {TestRunner} from 'test_runner';
-import {SecurityTestRunner} from 'security_test_runner';
-
-import * as SDK from 'devtools/core/sdk/sdk.js';
-import * as Security from 'devtools/panels/security/security.js';
-
-(async function() {
-  TestRunner.addResult(`Tests active mixed content blocking in the security panel.\n`);
-  await TestRunner.showPanel('security');
-
-  TestRunner.mainTarget.model(Security.SecurityModel.SecurityModel)
-      .dispatchEventToListeners(
-        Security.SecurityModel.Events.VisibleSecurityStateChanged,
-        new Security.SecurityModel.PageVisibleSecurityState(
-          Protocol.Security.SecurityState.Neutral, /* certificateSecurityState= */ null,
-          /* safetyTipInfo= */ null, /* securityStateIssueIds= */ ['scheme-is-not-cryptographic']));
-
-  var request = SDK.NetworkRequest.NetworkRequest.create(
-      0, 'http://foo.test', 'https://foo.test', 0, 0, null);
-  request.setBlockedReason(Protocol.Network.BlockedReason.MixedContent);
-  request.mixedContentType = 'blockable';
-  SecurityTestRunner.dispatchRequestFinished(request);
-
-  var explanations =
-      Security.SecurityPanel.SecurityPanel.instance().mainView.contentElement.getElementsByClassName('security-explanation');
-  for (var i = 0; i < explanations.length; i++)
-    TestRunner.dumpDeepInnerHTML(explanations[i]);
-
-  TestRunner.completeTest();
-})();

테스트 방법

CQ를 통해 2026년 9월 25일(한국 시간) Chromium main에 병합되었습니다.

배운 점

테스트 migration 후에는 기존 테스트와 기대 결과 파일도 함께 정리해야 합니다.

참고 자료