Remove unused SecureHash include from FileSystemAccessManagerImpl
MERGED2026contentfile_system_accesscleanup
2026. 9. 5.
zbnerd
zbnerdChromium의 File System Access 구현에서 사용하지 않는crypto/secure_hash.h include를 제거했습니다. 이 변경은 2026년 9월 9일
Chromium main에 병합되었습니다.
문제 설명
content/browser/file_system_access/file_system_access_manager_impl.cc에는crypto::SecureHash를 사용하는 코드가 없는데도crypto/secure_hash.h include가 남아 있었습니다.
이처럼 사용하지 않는 include는 불필요한 헤더 의존성을 만듭니다.
이번 CL은 crypto/secure_hash.h를 제거하기 위한 정리 작업의 일부로,
Chromium Issue 372283556과
연결되어 있습니다.
해결 내용
FileSystemAccessManagerImpl 구현 파일의 include 목록에서 다음 한 줄을
삭제했습니다.
-#include "crypto/secure_hash.h"최종 패치는 한 파일에서 한 줄을 제거하는 변경입니다.
테스트 방법
CL의 커밋 메시지와 Gerrit 실행 기록에 남은 검증은 다음과 같습니다.
git cl presubmit --upload로 업로드 전 presubmit 검사를 실행했습니다.- 기존
out/Default의 빌드 옵션을 재사용한 Clang 구문 검사로 대상 파일을
확인했습니다. - 2026년 9월 5일 Patch Set 2의
CQ dry run이
통과했습니다. - 2026년 9월 9일 Patch Set 3의
CQ 실행을
거쳐, 자동 rebase된 Patch Set 4가 main에 병합되었습니다.
배운 점
- 오래된 API를 정리할 때는 실제 사용 코드를 이전하는 작업뿐 아니라,
사용하지 않는 include를 제거하는 작업도 필요합니다. - include를 제거할 때는 해당 파일의 심볼 사용 여부를 확인하고,
구문 검사와 CQ로 변경을 검증할 수 있습니다. - 기존 빌드 설정을 재사용하면 대상 파일에 맞는 옵션으로 구문 검사를
수행할 수 있습니다.