Servo: स्क्रिप्ट कोड से tinyfiledialogs का उपयोग हटाएं

@highfive : मुझे असाइन करें

अरे @ejmg! इस मुद्दे पर काम करने में आपकी रुचि के लिए धन्यवाद। यह अब आपको सौंपा गया है!

आईआरसी पर प्रति अनुरोध अनअसाइनिंग।

@highfive : मुझे असाइन करें

अरे @cdeler! इस मुद्दे पर काम करने में आपकी रुचि के लिए धन्यवाद। यह अब आपको सौंपा गया है!

हैलो @jdm ,

मेरे पास स्थानीय रूप से एक मसौदा कार्यान्वयन है। क्या आप जानते हैं, मैं इसे स्थानीय रूप से कैसे जांच सकता हूं?

@cdeler आप एक पृष्ठ चलाने का प्रयास कर सकते हैं जो navigator.permissions.request({'name': 'geolocation'}) को कॉल करता है यह सत्यापित करने के लिए कि संशोधित कोड उचित स्थानों पर प्रिंटलाइन जोड़कर सही ढंग से चलता है।

क्या आप अभी भी इस @cdeler पर काम कर रहे हैं?

मुझे इसमें दिलचस्पी है और इसमें पहले से ही देखा गया है, लेकिन यदि यह संभव हो तो II को कुछ मार्गदर्शन की आवश्यकता हो सकती है।
पहली समस्या जिसका मैं सामना कर रहा हूं वह यह है कि मैं _ports/glutin/browser.rs_ में PermissionStatusBinding::PermissionState का उपयोग कैसे कर सकता हूं

@kleinph आपको एनम को परिभाषित करने की आवश्यकता होगी, जो स्क्रिप्ट और पोर्ट/ग्लूटिन दोनों के लिए उपलब्ध है, और फिर आवश्यकतानुसार दो एनम के बीच परिवर्तित करें।

ठीक है, मैं कोशिश करूँगा कि @jdm । धन्यवाद!

स्क्रिप्ट/डीओएम से एम्बेडर को संदेश भेजने का एक उदाहरण https://github.com/servo/servo/blob/9d9fff3b0ad843286875051e6544b3d4750d6238/components/script/dom/windowproxy.rs#L264 पर पाया जाता है।

@highfive : मुझे असाइन करें

अरे @gatoWololo! इस मुद्दे पर काम करने में आपकी रुचि के लिए धन्यवाद। यह अब आपको सौंपा गया है!

मुझे यह कैसे करना है इसका एक सामान्य ज्ञान है। एक बात जो मुझे स्पष्ट नहीं है वह यह है कि संदेश को एम्बेडर को कैसे भेजा जाए। #20428 और #20429 दोनों को embedder_proxy वस्तु सीधे नक्षत्र से servo/lib.rs मिलती है। हालाँकि, यह देखते हुए कि Pemissions structs कैसे बनाए जाते हैं, अंत में 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 हो सकता है, और अनुमति कोड या तो प्रतिक्रिया प्राप्त करने पर सिंक्रोनस ब्लॉक कर सकता है, या रिसीवर को आईपीसी राउटर तक हुक कर सकता है और एक प्रतिक्रिया प्राप्त होने पर थ्रेड के इवेंट लूप में एक ईवेंट इंजेक्ट कर सकता है।

एम्बेडर के साथ संचार कैसे करें, मुझे लगता है कि ग्लोबलस्कोप में एक send_to_embedder API जोड़ना और ग्लोबलस्कोप में एम्बेडर चैनल को संग्रहीत करना समझदार होगा। फिर अनुमति कोड self.global().send_to_embedder(...) जैसी किसी चीज़ का उपयोग करने में सक्षम होना चाहिए।

एम्बेडर को संदेश में IpcSender हो सकता है

ओह समझा।

बढ़िया है। मैं यहाँ से काम करूँगा!

तो मुझे लगता है कि मेरे पास कुछ काम कर रहा है। लेकिन मैं वास्तव में इसका परीक्षण करने में सक्षम नहीं हूं। मैंने सोचा था कि यह मेरा कोड था, लेकिन सर्वो के एक संशोधित संस्करण का उपयोग करके मुझे अभी भी त्रुटि मिलती है:

[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 साथ सर्वो चलाने का प्रयास करें।

मुझे अपने कार्यान्वयन का परीक्षण करने में कठिनाई हो रही है। अनुमतियों को कॉल करने वाला एक 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

पहला बस दुर्घटनाग्रस्त हो जाता है। दूसरा दुर्घटनाग्रस्त हो जाता है लेकिन इसकी उम्मीद है? इसलिए मुझे यकीन नहीं है कि मैंने कुछ भी परीक्षण किया है।

कुछ परीक्षण भी हैं जैसे tests/html/permission-test.html जो wpt का हिस्सा नहीं हैं। तो मुझे यकीन नहीं है कि उन्हें कैसे चलाया जाए।

परीक्षण/एचटीएमएल मैन्युअल परीक्षण हैं जिन्हें आप ./mach run tests/html/permission-test.html साथ चला सकते हैं। मैंने अभी अनुमति कोड में as_window() के उपयोगों पर ध्यान दिया है (https://github.com/servo/servo/blob/fd174c54ef4fa6574ae782dacccaeccd14abb936/components/script/dom/permissions.rs#L321-L322) जो बना देगा उस कोड को किसी भी समय गैर-विंडो स्कोप में बुलाया जाता है। हमें विंडो के बजाय permission_state_invocation_results API को GlobalScope पर ले जाकर उन्हें ठीक करने की आवश्यकता है।

test-background-fetch-permission.html में विफलता को इंगित करने के लिए धन्यवाद - मैंने इसे ठीक करने के लिए #23645 दर्ज किया है।

धन्यवाद। अच्छा ऐसा है।

जो उस कोड को किसी भी समय गैर-विंडो स्कोप में बुलाए जाने पर आतंकित कर देगा। हमें अनुमति_स्टेट_इनवोकेशन_results एपीआई को विंडो के बजाय ग्लोबलस्कोप पर ले जाकर उन्हें ठीक करने की आवश्यकता है।

क्या यह एक अलग पीआर होना चाहिए या उसी का हिस्सा होना चाहिए?

यह निश्चित रूप से एक अलग प्रतिबद्धता होनी चाहिए, लेकिन यह एक अलग पीआर भी हो सकती है। मैं इस समस्या (#23057) को सत्यापित करने के लिए स्वचालित परीक्षणों के अच्छे होने की उम्मीद नहीं करता, क्योंकि अनुमति कोड बिना सिर के चलने पर संकेत देने वाले UI को दबा देता है।

# 23651 में प्रस्तावित सुधार सही रास्ते पर था, लेकिन मेरे पास कुछ समीक्षा टिप्पणियां थीं जिन्हें संबोधित करने की आवश्यकता है और लेखक अन्य चीजों पर चले गए हैं।

@peacerebel यह काम करने के लिए एक अच्छा अगला मुद्दा हो सकता है!

@PeaceRebel यह काम करने के लिए एक अच्छा अगला मुद्दा हो सकता है!

कल इस पर एक नजर डालेंगे।

@highfive मुझे असाइन करें

अरे @PeaceRebel! इस मुद्दे पर काम करने में आपकी रुचि के लिए धन्यवाद। यह अब आपको सौंपा गया है!

निष्क्रियता के कारण अनअसाइनिंग।

मैंने अपनी शाखा https://github.com/iulianR/servo/tree/issue-23057-tinifiledialogs पर मास्टर के शीर्ष पर पीआर में परिवर्तन को पुन: आधारित किया

वह महान होगा!

मैंने अपनी शाखा को एक प्रतिबद्धता के साथ अद्यतन किया जो स्ट्रिंग स्वरूपण को एम्बेडर्स को दर्शाता है।

अब मैं पीआर के दूसरे भाग को संबोधित करने की कोशिश में थोड़ा फंस गया हूं। मुझे यकीन नहीं है कि मुझे HostTrait पर एक नई विधि की आवश्यकता है, या यदि मुझे पहले से मौजूद prompt_yes_no() उपयोग/अपडेट करना चाहिए। हो सकता है कि आप मेरे लिए विधि हस्ताक्षर लिख सकें। धन्यवाद।

मुझे लगता है कि अभी के लिए प्रॉम्प्ट_यस_नो विधि का उपयोग करना ठीक लगता है।