OSSCA Chromium

기여 목록 / 8460623

[web_apps] Remove the Adobe Express OEM-to-default install migration

BrowsercleanupMerged

ChromeOS에서 Adobe Express 앱의 설치 출처를 kOem에서 kApsDefault로 옮기던 일회성 마이그레이션을 통째로 지웠습니다. «~M134에 제거»라는 TODO가 있었고 트리는 M156입니다. 7파일 −225. 2026-09-29에 머지됐습니다(496b52acef84a). mek@와 dmurph@의 +1을 받았고, 첫 CQ가 봇 할당 시간 초과(«Task expired»)로 떨어져 재요청 뒤 통과했습니다.

문제 설명

  • issue url: 내부 버그 b/314865744 (공개 crbug 없음)
  • kApsDefault가 지원되기 전에는 Adobe Express를 우회로 kOem으로 설치했습니다. 2023-12 CL(tsergeant@)이 지원 이후 이를 kApsDefault로 옮기는 마이그레이션을 넣었고, WebAppProvider가 sync bridge 준비 직후 매번 실행했습니다.
  • 앱을 가진 프로필은 이미 한 번씩 마이그레이션을 거쳤으므로 더는 필요 없는 코드였습니다.

해결 내용

  • chrome/browser/web_applications/ash/migrations/ 디렉터리(.h/.cc/browsertest/BUILD.gn/DEPS)를 지웠습니다.
  • web_app_provider.cc에서 include와 호출을, web_applications/BUILD.gn에서 참조 세 곳을 지웠습니다.
  • 그 과정에서 이 타깃 때문에만 있던 allow_circular_includes_from(TODO b/332804822)도 함께 사라졌습니다.
-#if BUILDFLAG(IS_CHROMEOS)
-  web_app::migrations::MigrateAdobeExpressFromOemInstallToDefault(
-      sync_bridge_.get());
-#endif  // BUILDFLAG(IS_CHROMEOS)

발굴 과정

만료 주석 스윕에서 ChromeOS 전용 파일만 추린 후보 V~Z 중 하나입니다. 처음엔 «테스트 없음, S 크기»로 봤지만 파일을 열어 보니 마이그레이션 전용 browsertest와 빌드 타깃이 따로 있어 M 크기의 디렉터리 삭제가 됐습니다. web_applications OWNERS는 리뷰어 풀 주소라 그 주소를 넣어 자동 배정(mek@)을 받았습니다. OSSCA #545.

테스트 방법

  • 이 경로를 검사하는 테스트는 함께 지운 browsertest뿐이라, 빌드 그래프와 컴파일로 검증했습니다.
  • 리눅스 박스에서 target_os="chromeos"로 gn gen(끊긴 참조 없음), gn check //chrome/browser/web_applications/* 통과, unit_tests 링크 통과(web_app_provider.cc 재컴파일, 빌드 그래프에 ash/migrations 참조 0).
  • browser_tests는 4코어 박스에서 너무 무거워 돌리지 않았습니다.
  • presubmit 0 경고

배운 점

  • 디렉터리를 지우는 CL은 코드보다 빌드 그래프가 핵심입니다. gn gen과 gn check, 링크까지 확인합니다.
  • OWNERS에 리뷰어 풀 주소가 있고 개인은 LAST_RESORT_SUGGESTION이면 풀 주소를 넣습니다. gwsq가 담당자를 배정해 줍니다.

참고 자료

목록으로