21.09.25. chromium issue_1252242 패치 도전1(fail 수정)

Nayeon Kim·2021년 9월 25일
2

chromium

목록 보기
7/10

issue 1252242(https://bugs.chromium.org/p/chromium/issues/detail?id=1252242#c_ts1632393184)는
content/shell/browser/shell_content_browser_client.cc 파일에서 getChannel()함수의 return값을 바꿈으로써, 새롭게 생겨날 특수 사항들이 정상적으로 적용될 수 있도록 해야 하는 이슈였다.
기존 GetChannel() 함수는 Channel::STABLE을 반환했는데, 이는 STABLE 채널을 허용하지 않는 행위들이 추가됐을 때 제대로 적용되지 않을 수 있다는 문제점을 가지고 있어서 이 함수의 반환값을 Channel::UNKNOWN으로 수정함으로써 특정 행위가 어떤 채널을 허용하고 있든 모두 제대로 적용될 수 있도록 해야했다.

사실 패치 내용 자체가 한 파일 내 특정 함수 return값 바꾸는 게 다라서, 매우 간단한 문제일 줄 알았으나.
아직 권한이 없는 나를 대신해 멘토님께서 돌려주신 CQ dry run 과정에서 일부 환경에 fail이 나는 듯 했다.

->모두 초록불이 들어와야 정상

멘토님 recommend대로, browser_tests 빌드를 거친 후, 실행시켜 fail 내역을 확인하는 중이다.
(어제부터 밤새 돌렸는데, 끝날 기미가 안 보임.. 노트북으로 버티고 있었는데 사양 높은 컴퓨터 필요할 듯)

ninja -C out/Default browser_tests

out/Default/browser_tests

로 browser_tests를 실행할 수 있는데, 실행을 하면 cmd에 14000개 넘는 테스트가 계속 올라오고, 브라우저들이 떴다가 사라지고, 무슨 삑삑거리는 소리도 나다가 노래소리도 나고 종소리도 나고(자다가 놀라서 일어난;;) 아마 오늘 안에도 끝이 날 수 있을지 모르겠다. 노트북 열이 많이 올라와서 팍 꺼지지나 않았으면.

running browser_tests 관련 자세한 내용은 https://www.chromium.org/developers/testing/browser-tests 에서 확인할 수 있는데,(멘토님이 참고 자료로 자주 보내주시는데, chromium 관련해서 생기는 웬만한 궁금증은 chromium.org에서 해결할 수 있을 듯 하다.)
다시 찬찬히 읽어 보니, 이 테스트 실행은 좋은 컴퓨터에서도 상당한 시간이 걸리는 작업이라고 함. --test-launcher-bot-mode, --test-launcher-jobs=20 등 다양한 옵션을 추가할 수 있는데 사용해보진 못했다.

[7983/14120] SitePerProcessPrintBrowserTest.MultipagePrint (TIMED OUT)
Too many badly broken tests (1412), exiting now.

약 8000개쯤 테스트가 돌다가 failed test가 많아서 자동으로 종료됐다.
다음은 어떻게 해야 할지 대책을 세워야겠음.

profile
Department of Computer Science

0개의 댓글