기여 목록 / 8174985
Expose dragged images as a real file on demand on Windows
UIfixIn Review
윈도우에서 포토샵이나 그림판 등으로 이미지를 드래그 했을때 열리지 않던 문제를 해결했습니다
문제 설명
- issue url: https://crbug.com/40349733
- 윈도우에서 포토샵이나 그림판 등으로 이미지를 드래그 했을때 열리지 않음
해결 내용
- 근본적으로 해당 문제는 전달될때 사용되는 핸들인
CF_HDROP이 크롬의 경우 파일 스트림으로만 전달되지만, 포토샵이나 그림판은 실제 파일이 필요하기 때문에 발생하는 문제입니다. 따라서, 드래그된 이미지가 실제 파일로 존재하지 않는 경우, 임시 파일을 생성하여 전달하도록 수정했습니다. - Incognito, guest, 실패했을때 등등 상황에서의 파일 저장 케이스들을 처리하고 이에 대한 테스트 케이스를 추가했습니다.
리뷰 과정
- 코드를 넣는 위치에 대해서 리뷰어 의견이 있었고 Incognito나 Guest 프로필에서 파일 저장에 대한 리뷰가 있었습니다
- 이를 분기화하여 Incognito나 Guest 프로필에서는 임시 파일을 생성하지 않고, 드래그된 이미지를 실제 파일로 저장할 수 없는 경우에는 드래그를 실패하도록 처리했습니다.
테스트 방법
- 드래그 앤 드랍 시 정상적으로 처리가 되는지, Incognito 나 Guest 프로필에서 드래그 앤 드랍 시 실패하는지에 대한 테스트 케이스를 추가하였습니다
배운 점
- 우선순위가 낮은 이슈이고, 윈도우 서드파티에 대한 리뷰여서 리뷰가 매우 느리게 진행되었지만, 함께 진행하며 여러 케이스를 심도있게 고려하고 그에 대한 테스트 케이스를 작성해보면서 많은걸 배울 수 있었습니다.
- 특히 각 앱들간 윈도우 핸들러 공유 처리에 대해 공부할 수 있었습니다.