Servo: 스크립트 코드에서 tinyfiledialog 사용 제거

@highfive : 나에게 할당

안녕하세요 @ejmg! 이 문제에 관심을 가져주셔서 감사합니다. 이제 당신에게 할당되었습니다!

IRC에 대한 요청에 따라 할당 해제.

@highfive : 나에게 할당

안녕하세요 @cdeler입니다! 이 문제에 관심을 가져주셔서 감사합니다. 이제 당신에게 할당되었습니다!

안녕하세요 @jdm ,

로컬에 초안 구현이 있습니다. 현지에서 어떻게 확인할 수 있는지 아세요?

@cdeler 적절한 위치에 println을 추가하여 수정된 코드가 올바르게 실행되는지 확인하기 위해 navigator.permissions.request({'name': 'geolocation'}) 를 호출하는 페이지를 실행해 볼 수 있습니다.

이 @cdeler에서 아직도 작업 중이신가요?

나는 이것에 관심이 있고 이미 그것을 조사했지만 이것이 가능한 경우 II에 약간의 지침이 필요할 수 있습니다.
내가 직면한 첫 번째 문제는 _ports/glutin/browser.rs_에서 PermissionStatusBinding::PermissionState 를 어떻게 사용할 수 있습니까?

@kleinph 스크립트와 포트/ embedder_traits 에 동등한 열거형을 정의한 다음 필요에 따라 두 열거형 간에 변환해야 합니다.

좋아요, @jdm 을 시도해

스크립트/DOM에서 임베더로 메시지를 보내는 예는 https://github.com/servo/servo/blob/9d9fff3b0ad843286875051e6544b3d4750d6238/components/script/dom/windowproxy.rs#L264에 있습니다.

@highfive : 나에게 할당

안녕하세요 @gatoWololo! 이 문제에 관심을 가져주셔서 감사합니다. 이제 당신에게 할당되었습니다!

나는 그것을하는 방법에 대한 일반적인 감각을 가지고 있습니다. 나에게 명확하지 않은 한 가지는 임베더에 메시지를 보내는 방법입니다. #20428 과 #20429 모두 servo/lib.rs 의 성좌에서 직접 embedder_proxy 개체를 얻습니다. 그러나 Pemissions 구조체가 생성되는 방식을 보면 마침내 create_constellation 도달하기 전에 꽤 많은 레이어가 있습니다.

Both of these contain Permissions:
struct WorkerNavigator
struct Navigator

WorkerNavigator is created by WorkerGlobalScope which is created by a:
- ServiceWorkerGlobalScope which is created by a ServiceWorkerManager
   which is init from servo/lib.rs script::init_service_workers(sw_senders);
or
- DedicatedWorkerGlobalScope which is created by run_worker_scope() which is called
  by Worker::Constructor() which does not seem to be called from anywhere...

Navigator is created by Window::Navigator(&self) it is not clear to me how a Window is
created.

window.send_to_embedder(msg); 도 있는 것을 보았으므로 메시지를 보낼 수 있지만 임베더로부터 응답을 받으려면 어떻게 해야 합니까?

여기 대화를 기반으로: https://github.com/servo/servo/pull/20480 embedder_proxy 전달하는 것이 합의된 방식이었습니까?

임베더에 대한 메시지는 IpcSender를 포함할 수 있으며 권한 코드는 응답 수신 시 동기적으로 차단하거나 수신기를 ipc 라우터에 연결하고 응답이 수신될 때 스레드의 이벤트 루프에 이벤트를 주입할 수 있습니다.

Embedder와 통신하는 방법은 GlobalScope에 send_to_embedder API를 추가하고 Embedder 채널을 GlobalScope에 저장하는 것이 현명하다고 생각합니다. 그런 다음 권한 코드는 self.global().send_to_embedder(...) 와 같은 것을 사용할 수 있어야 합니다.

임베더에 대한 메시지는 IpcSender를 포함할 수 있습니다.

아, 알겠습니다.

잘 들린다. 여기서부터 일할게!

그래서 뭔가 효과가 있는 것 같아요. 그러나 나는 그것을 실제로 테스트 할 수 없습니다. 내 코드라고 생각했지만 수정되지 않은 버전의 Servo를 사용하면 여전히 오류가 발생합니다.

[2019-06-25T21:25:40Z ERROR script::dom::bindings::error] Error at file:///home/gatowololo/permissionTest.html:14:9 window.navigator.permissions is undefined

여기서 permissionTest.html 는 servo/tests/html/permission-test.html 의 복사본입니다.

window.navigator.permissions is undefined

활성화해야 할 것 같습니다.

https://github.com/servo/servo/blob/e100af57a5bd95701b5310871e9909e3726539f0/resources/prefs.json#L17

--pref dom.permissions.enabled Servo를 실행해 보세요.

구현을 테스트하는 데 어려움을 겪고 있습니다. 권한을 호출하는 html 페이지를 열면 작동합니다(예!). 하지만 이 기능과 관련된 테스트를 제대로 실행할 수 없습니다. 그래서 "navigator.permissions"를 호출하는 테스트를 찾습니다.

> rg "navigator.permissions" *
tests/wpt/web-platform-tests/permissions/interfaces.any.js
24:    self.permissionStatus = await navigator.permissions.query({ name: "geolocation" });
25:    self.permissionStatus = await navigator.permissions.query({ name: "background-fetch" });
37:    Permissions: ['navigator.permissions'],
tests/wpt/web-platform-tests/permissions/test-background-fetch-permission.html
10:    return navigator.permissions.query({name:'background-fetch'}).then(function(result) {
...

그런 다음 이 wpt 테스트 중 일부를 실행하려고 합니다.

> ./mach test-wpt --pref dom.permissions.enabled --release tests/wpt/web-platform-tests/permissions/interfaces.any.js
...
0:01.87 pid:4508 [2019-06-27T15:49:30Z ERROR servo] expected a Window scope

> ./mach test-wpt --pref dom.permissions.enabled --release tests/wpt/web-platform-tests/permissions/test-background-fetch-permission.html
...
 0:01.65 pid:4793 [2019-06-27T15:51:24Z ERROR servo] assertion failed: !JS_IsExceptionPending(cx)
 0:01.65 pid:4793 Pipeline failed in hard-fail mode.  Crashing!
 0:01.69 TEST_END: CRASH, expected OK

첫 번째는 그냥 충돌합니다. 두 번째 충돌이 예상되지만 예상됩니까? 그래서 나는 내가 아무 것도 테스트하지 않았다고 확신하지 않습니다.

wpt의 일부가 아닌 tests/html/permission-test.html 와 같은 일부 테스트도 있습니다. 그래서 나는 그들을 실행하는 방법을 잘 모르겠습니다.

tests/html 은 ./mach run tests/html/permission-test.html 실행할 수 있는 수동 테스트입니다. 방금 권한 코드(https://github.com/servo/servo/blob/fd174c54ef4fa6574ae782dacccaeccd14abb936/components/script/dom/permissions.rs#L321-L322)에서 as_window() 사용을 확인했습니다 비 Window 범위에서 호출될 때마다 해당 코드 패닉이 발생합니다. permission_state_invocation_results API를 Window 대신 GlobalScope로 이동하여 수정해야 합니다.

test-background-fetch-permission.html의 오류를 지적해 주셔서 감사합니다. 수정하기 위해 #23645를 제출했습니다.

감사 해요. 알 겠어요.

비 Window 범위에서 호출될 때마다 해당 코드를 패닉 상태로 만듭니다. permission_state_invocation_results API를 Window 대신 GlobalScope로 이동하여 수정해야 합니다.

이것은 별도의 PR 또는 동일한 PR의 일부여야 합니까?

확실히 별도의 커밋이어야 하지만 별도의 PR일 수도 있습니다. 자동 테스트가 이 문제(#23057)를 확인하는 데 적합하지 않을 것이라고 생각합니다. 권한 코드가 헤드리스로 실행할 때 프롬프트 UI를 억제하기 때문입니다.

#23651에서 제안된 수정 사항은 올바른 방향으로 진행되었지만 해결해야 할 몇 가지 검토 의견이 있었고 작성자는 다른 작업으로 이동했습니다.

@peacerebel 이것은 작업하기에 좋은 다음 호가 될 것입니다!

@PeaceRebel 이것은 작업하기에 좋은 다음 호가 될 수 있습니다!

이것은 내일 살펴볼 것입니다.

@highfive 할당

안녕하세요 @PeaceRebel입니다! 이 문제에 관심을 가져주셔서 감사합니다. 이제 당신에게 할당되었습니다!

활동이 없어 할당을 취소합니다.

내 분기 https://github.com/iulianR/servo/tree/issue-23057-tinifiledialogs 의 마스터 위에 있는 PR의 변경 사항을 다시 기반으로 했습니다

그것은 좋을 것입니다!

문자열 형식을 임베더에 위임하는 커밋으로 분기를 업데이트했습니다.

이제 PR의 두 번째 부분을 다루려고 하는 데 약간 막혔습니다. HostTrait 에 대한 새 메서드가 필요한지 또는 이미 존재하는 prompt_yes_no() 메서드를 사용/업데이트해야 하는지 잘 모르겠습니다. 저를 위해 메소드 서명을 작성해 주실 수 있습니다. 감사 해요.

지금은 prompt_yes_no 메서드를 사용하는 것이 좋다고 생각합니다.