21.09.07. chromium issue_1246835 패치 시도(해결중)

Nayeon Kim·2021년 9월 7일
2

chromium

목록 보기
4/10

지금 크로미움 3번째 빌드를 하게 생긴 처참한 마음을 안고 GoodFirstBug(첫 패치로 시도해보기 좋은 버그들이랄까) 중 issue_1246835를 보고 해봐도 괜찮겠다는 생각이 들어 코드를 수정해봤다.

issue 내용을 보아하니,
썸네일이나 프리뷰로 보여지는 이미지가 resizing될 때 현재는 RESIZE_GOOD을 사용하여 이미지를 보여주고 있는데, cpu에 영향이 거의 안 가고 유저들이 명확히 차이를 느낄 수 있다면 RESIZE_BEST로 퀄리티를 높이는 건 어떨지 권유하고 있음.

사실 available 상태가 아니라 untriaged 상태기도 하고(아직 이 이슈에 대해 패치를 진행해야 될지 결정이 안된 거 같음), CPU 성능 테스트같은 걸 내가 할 수 있나 싶어서 선뜻 건드리기가 망설어졌는데
뭐, 일단은 이 패치 과정을 이해하는 게 먼저라 실습해보는 생각으로 코드 수정해서 GERRIT에 업로드까지 완료했다.

issue를 살펴봐도 되는지 issue 작성자에게 댓글까지 남겨둠 하하..
일단 크롬 빌드나 다시 하러 가보자..

C:\Users\SAMSUNG>cd c:\chromium\src

c:\chromium\src>git status
On branch media
Changes to be committed:
  (use "git restore --staged <file>..." to unstage)
        modified:   chrome/browser/media/webrtc/current_tab_desktop_media_list.cc
        modified:   chrome/browser/ui/views/extensions/extension_uninstall_dialog_view.c
        modified:   chrome/browser/ui/views/extensions/print_job_confirmation_dialog_view.cc
        modified:   chrome/renderer/chrome_render_frame_observer.cc


c:\chromium\src>git commit -a
hint: Waiting for your editor to close the file... unix2dos: converting file C:/chromium/src/.git/COMMIT_EDITMSG to DOS format...
dos2unix: converting file C:/chromium/src/.git/COMMIT_EDITMSG to Unix format...
Aborting commit due to empty commit message.

c:\chromium\src>git status
Refresh index: 100% (376707/376707), done.
On branch media
Changes to be committed:
  (use "git restore --staged <file>..." to unstage)
        modified:   chrome/browser/media/webrtc/current_tab_desktop_media_list.cc
        modified:   chrome/browser/ui/views/extensions/extension_uninstall_dialog_view.cc
        modified:   chrome/browser/ui/views/extensions/print_job_confirmation_dialog_view.cc
        modified:   chrome/renderer/chrome_render_frame_observer.cc

Changes not staged for commit:
  (use "git add <file>..." to update what will be committed)
  (use "git restore <file>..." to discard changes in working directory)
        modified:   AUTHORS


c:\chromium\src>git add AUTHORS

c:\chromium\src>git status
On branch media
Changes to be committed:
  (use "git restore --staged <file>..." to unstage)
        modified:   AUTHORS
        modified:   chrome/browser/media/webrtc/current_tab_desktop_media_list.cc
        modified:   chrome/browser/ui/views/extensions/extension_uninstall_dialog_view.cc
        modified:   chrome/browser/ui/views/extensions/print_job_confirmation_dialog_view.cc
        modified:   chrome/renderer/chrome_render_frame_observer.cc


c:\chromium\src>git commit -a
hint: Waiting for your editor to close the file... unix2dos: converting file C:/chromium/src/.git/COMMIT_EDITMSG to DOS format...
dos2unix: converting file C:/chromium/src/.git/COMMIT_EDITMSG to Unix format...
[media e4b0256f7471] GetDisplayMedia: upper quality of media-picker
 5 files changed, 5 insertions(+), 4 deletions(-)

c:\chromium\src>git push origin HEAD:refs/for/main
Enumerating objects: 28, done.
Counting objects: 100% (28/28), done.
Delta compression using up to 8 threads
Compressing objects: 100% (15/15), done.
Writing objects: 100% (15/15), 3.30 KiB | 140.00 KiB/s, done.
Total 15 (delta 13), reused 0 (delta 0), pack-reused 0
remote: Resolving deltas: 100% (13/13)
remote: Counting objects: 402703, done
remote: Processing changes: refs: 1, new: 1, done
remote: warning: pushing without Change-Id is deprecated
remote:
remote: SUCCESS
remote:
remote:   https://chromium-review.googlesource.com/c/chromium/src/+/3145150 GetDisplayMedia: upper quality of media-picker [NEW]
remote:
To https://chromium.googlesource.com/chromium/src.git
 * [new reference]             HEAD -> refs/for/main
profile
Department of Computer Science

0개의 댓글