Skip to content

Commit 0b75fa2

Browse files
TFSThiagoBR98TFSThiagoBR98
authored
Merge pull request #122 from TFSThiagoBR98/fix-77
Fix Events and Styles String Mutation, closes #77
2 parents 882fa5a + 3102398 commit 0b75fa2

10 files changed

+718
-150
lines changed

‎build/WebIDL.py‎

Lines changed: 13 additions & 5 deletions
Original file line numberDiff line numberDiff line change
@@ -1,4 +1,10 @@
1-
# from http://mxr.mozilla.org/mozilla-central/source/dom/bindings/parser/WebIDL.py
1+
## JavascriptSubtitlesOctopus
2+
## Patched to:
3+
## - add integer pointers (IntPtr)
4+
## - add [Owner] Extended attribute
5+
## From https://github.com/emscripten-core/emscripten/blob/f36f9fcaf83db93e6a6d0f0cdc47ab6379ade139/third_party/WebIDL.py
6+
7+
# from https://hg.mozilla.org/mozilla-central/file/tip/dom/bindings/parser/WebIDL.py
28
# rev 501baeb3a034
39

410
# This Source Code Form is subject to the terms of the Mozilla Public
@@ -2858,6 +2864,7 @@ def handleExtendedAttribute(self, attr):
28582864
identifier == "AvailableIn" or
28592865
identifier == "Const" or
28602866
identifier == "Value" or
2867+
identifier == "Owner" or
28612868
identifier == "BoundsChecked" or
28622869
identifier == "NewObject"):
28632870
# Known attributes that we don't need to do anything with here
@@ -2933,7 +2940,7 @@ def addExtendedAttributes(self, attrs):
29332940
self.enforceRange = True
29342941
elif identifier == "TreatNonCallableAsNull":
29352942
self._allowTreatNonCallableAsNull = True
2936-
elif identifier in ['Ref', 'Const']:
2943+
elif identifier in ['Ref', 'Const', 'Owner']:
29372944
# ok in emscripten
29382945
self._extraAttributes[identifier] = True
29392946
else:
@@ -4923,11 +4930,12 @@ def p_error(self, p):
49234930

49244931
def __init__(self, outputdir='', lexer=None):
49254932
Tokenizer.__init__(self, outputdir, lexer)
4926-
self.parser = yacc.yacc(module=self,
4933+
self.parser = yacc.yacc(debug=0,
4934+
module=self,
49274935
outputdir=outputdir,
49284936
tabmodule='webidlyacc',
4929-
errorlog=yacc.NullLogger(),
4930-
picklefile='WebIDLGrammar.pkl')
4937+
write_tables=0,
4938+
errorlog=yacc.NullLogger())
49314939
self._globalScope = IDLScope(BuiltinLocation("<Global Scope>"), None, None)
49324940
self._installBuiltins(self._globalScope)
49334941
self._productions = []

‎build/patches/webidl_binder/0001-WebIDL-Add-headers-and-make-it-work-under-SubtitlesO.patch‎

Lines changed: 87 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -0,0 +1,87 @@
1+
From d9b7e2738213d631810ace5e5f47cc8bc210fa8c Mon Sep 17 00:00:00 2001
2+
From: =?UTF-8?q?Thiago=20Fran=C3=A7a=20da=20Silva?=
3+
<tfsthiagobr98@outlook.com>
4+
Date: Fri, 18 Feb 2022 21:18:04 -0300
5+
Subject: [PATCH 1/4] WebIDL: Add headers and make it work under
6+
SubtitlesOctopus project
7+
8+
---
9+
build/WebIDL.py | 3 +++
10+
build/tempfiles.py | 3 +++
11+
build/webidl_binder.py | 20 ++++++++++----------
12+
3 files changed, 16 insertions(+), 10 deletions(-)
13+
14+
diff --git a/build/WebIDL.py b/build/WebIDL.py
15+
index 14689cb..8892616 100644
16+
--- a/build/WebIDL.py
17+
+++ b/build/WebIDL.py
18+
@@ -1,3 +1,6 @@
19+
+## JavascriptSubtitlesOctopus
20+
+## From https://github.com/emscripten-core/emscripten/blob/f36f9fcaf83db93e6a6d0f0cdc47ab6379ade139/third_party/WebIDL.py
21+
+
22+
# from https://hg.mozilla.org/mozilla-central/file/tip/dom/bindings/parser/WebIDL.py
23+
# rev 501baeb3a034
24+
25+
diff --git a/build/tempfiles.py b/build/tempfiles.py
26+
index e1c9dcc..6487516 100644
27+
--- a/build/tempfiles.py
28+
+++ b/build/tempfiles.py
29+
@@ -1,3 +1,6 @@
30+
+## JavascriptSubtitlesOctopus
31+
+## From https://github.com/emscripten-core/emscripten/blob/c834ef7d69ccb4100239eeba0b0f6573fed063bc/tools/tempfiles.py
32+
+
33+
# Copyright 2013 The Emscripten Authors. All rights reserved.
34+
# Emscripten is available under two separate licenses, the MIT license and the
35+
# University of Illinois/NCSA Open Source License. Both these licenses can be
36+
diff --git a/build/webidl_binder.py b/build/webidl_binder.py
37+
index da864f8..687a5ba 100644
38+
--- a/build/webidl_binder.py
39+
+++ b/build/webidl_binder.py
40+
@@ -1,3 +1,6 @@
41+
+## JavascriptSubtitlesOctopus
42+
+## From https://github.com/emscripten-core/emscripten/blob/f36f9fcaf83db93e6a6d0f0cdc47ab6379ade139/tools/webidl_binder.py
43+
+
44+
# Copyright 2014 The Emscripten Authors. All rights reserved.
45+
# Emscripten is available under two separate licenses, the MIT license and the
46+
# University of Illinois/NCSA Open Source License. Both these licenses can be
47+
@@ -10,15 +13,12 @@ https://emscripten.org/docs/porting/connecting_cpp_and_javascript/WebIDL-Binder.
48+
49+
import os
50+
import sys
51+
+from tempfiles import try_delete
52+
53+
-__scriptdir__ = os.path.dirname(os.path.abspath(__file__))
54+
-__rootdir__ = os.path.dirname(__scriptdir__)
55+
-sys.path.append(__rootdir__)
56+
-
57+
-from tools import shared, utils
58+
+def path_from_root(*pathelems):
59+
+ return os.path.join(os.path.join('/', 'emsdk', 'upstream', 'emscripten'), *pathelems)
60+
61+
-sys.path.append(utils.path_from_root('third_party'))
62+
-sys.path.append(utils.path_from_root('third_party/ply'))
63+
+sys.path.append(path_from_root('third_party', 'ply'))
64+
65+
import WebIDL
66+
67+
@@ -50,14 +50,14 @@ class Dummy:
68+
input_file = sys.argv[1]
69+
output_base = sys.argv[2]
70+
71+
-shared.try_delete(output_base + '.cpp')
72+
-shared.try_delete(output_base + '.js')
73+
+try_delete(output_base + '.cpp')
74+
+try_delete(output_base + '.js')
75+
76+
p = WebIDL.Parser()
77+
p.parse(r'''
78+
interface VoidPtr {
79+
};
80+
-''' + utils.read_file(input_file))
81+
+''' + open(input_file).read())
82+
data = p.finish()
83+
84+
interfaces = {}
85+
--
86+
2.35.1
87+

‎build/patches/webidl_binder/0002-WebIDL-Implement-Integer-Pointer-type-IntPtr.patch‎

Lines changed: 101 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -0,0 +1,101 @@
1+
From 2e7bb7592195c94853c4e3b6718d47677e1555bf Mon Sep 17 00:00:00 2001
2+
From: =?UTF-8?q?Thiago=20Fran=C3=A7a=20da=20Silva?=
3+
<tfsthiagobr98@outlook.com>
4+
Date: Fri, 18 Feb 2022 21:19:20 -0300
5+
Subject: [PATCH 2/4] WebIDL: Implement Integer Pointer type (IntPtr)
6+
7+
This allows compatibility with `changed` parameter of the `renderImage` function
8+
---
9+
build/WebIDL.py | 15 +++++++++++++++
10+
build/webidl_binder.py | 4 ++++
11+
2 files changed, 19 insertions(+)
12+
13+
diff --git a/build/WebIDL.py b/build/WebIDL.py
14+
index 8892616..d140f8f 100644
15+
--- a/build/WebIDL.py
16+
+++ b/build/WebIDL.py
17+
@@ -1,4 +1,6 @@
18+
## JavascriptSubtitlesOctopus
19+
+## Patched to:
20+
+## - add integer pointers (IntPtr)
21+
## From https://github.com/emscripten-core/emscripten/blob/f36f9fcaf83db93e6a6d0f0cdc47ab6379ade139/third_party/WebIDL.py
22+
23+
# from https://hg.mozilla.org/mozilla-central/file/tip/dom/bindings/parser/WebIDL.py
24+
@@ -2165,6 +2167,7 @@ class IDLBuiltinType(IDLType):
25+
'date',
26+
'void',
27+
# Funny stuff
28+
+ 'IntPtr',
29+
'ArrayBuffer',
30+
'ArrayBufferView',
31+
'Int8Array',
32+
@@ -2184,6 +2187,7 @@ class IDLBuiltinType(IDLType):
33+
Types.short: IDLType.Tags.int16,
34+
Types.unsigned_short: IDLType.Tags.uint16,
35+
Types.long: IDLType.Tags.int32,
36+
+ Types.IntPtr: IDLType.Tags.int32,
37+
Types.unsigned_long: IDLType.Tags.uint32,
38+
Types.long_long: IDLType.Tags.int64,
39+
Types.unsigned_long_long: IDLType.Tags.uint64,
40+
@@ -2380,6 +2384,9 @@ BuiltinTypes = {
41+
IDLBuiltinType.Types.any:
42+
IDLBuiltinType(BuiltinLocation("<builtin type>"), "Any",
43+
IDLBuiltinType.Types.any),
44+
+ IDLBuiltinType.Types.IntPtr:
45+
+ IDLBuiltinType(BuiltinLocation("<builtin type>"), "IntPtr",
46+
+ IDLBuiltinType.Types.IntPtr),
47+
IDLBuiltinType.Types.domstring:
48+
IDLBuiltinType(BuiltinLocation("<builtin type>"), "String",
49+
IDLBuiltinType.Types.domstring),
50+
@@ -3622,6 +3629,7 @@ class Tokenizer(object):
51+
"...": "ELLIPSIS",
52+
"::": "SCOPE",
53+
"Date": "DATE",
54+
+ "IntPtr": "INTPTR",
55+
"DOMString": "DOMSTRING",
56+
"ByteString": "BYTESTRING",
57+
"any": "ANY",
58+
@@ -4507,6 +4515,7 @@ class Parser(Tokenizer):
59+
| DATE
60+
| DOMSTRING
61+
| BYTESTRING
62+
+ | INTPTR
63+
| ANY
64+
| ATTRIBUTE
65+
| BOOLEAN
66+
@@ -4573,6 +4582,12 @@ class Parser(Tokenizer):
67+
"""
68+
p[0] = self.handleModifiers(BuiltinTypes[IDLBuiltinType.Types.any], p[2])
69+
70+
+ def p_SingleTypeIntPtrType(self, p):
71+
+ """
72+
+ SingleType : INTPTR TypeSuffixStartingWithArray
73+
+ """
74+
+ p[0] = self.handleModifiers(BuiltinTypes[IDLBuiltinType.Types.IntPtr], p[2])
75+
+
76+
def p_UnionType(self, p):
77+
"""
78+
UnionType : LPAREN UnionMemberType OR UnionMemberType UnionMemberTypes RPAREN
79+
diff --git a/build/webidl_binder.py b/build/webidl_binder.py
80+
index 687a5ba..e9a56e5 100644
81+
--- a/build/webidl_binder.py
82+
+++ b/build/webidl_binder.py
83+
@@ -1,4 +1,6 @@
84+
## JavascriptSubtitlesOctopus
85+
+## Patched to:
86+
+## - add integer pointers (IntPtr)
87+
## From https://github.com/emscripten-core/emscripten/blob/f36f9fcaf83db93e6a6d0f0cdc47ab6379ade139/tools/webidl_binder.py
88+
89+
# Copyright 2014 The Emscripten Authors. All rights reserved.
90+
@@ -337,6 +339,8 @@ def type_to_c(t, non_pointing=False):
91+
ret = 'bool'
92+
elif t == 'Any' or t == 'VoidPtr':
93+
ret = 'void*'
94+
+ elif t == 'IntPtr':
95+
+ ret = 'int*'
96+
elif t in interfaces:
97+
ret = (interfaces[t].getExtendedAttribute('Prefix') or [''])[0] + t + ('' if non_pointing else '*')
98+
else:
99+
--
100+
2.35.1
101+

‎build/patches/webidl_binder/0003-WebIDL-Implement-ByteString-type.patch‎

Lines changed: 35 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -0,0 +1,35 @@
1+
From ea9c45b10d807966510711da723ea1ae558efd45 Mon Sep 17 00:00:00 2001
2+
From: =?UTF-8?q?Thiago=20Fran=C3=A7a=20da=20Silva?=
3+
<tfsthiagobr98@outlook.com>
4+
Date: Mon, 14 Feb 2022 22:27:57 -0300
5+
Subject: [PATCH 3/4] WebIDL: Implement ByteString type
6+
7+
This allows compatibility with `bitmap` attribute of the ASS_Image struct
8+
---
9+
build/webidl_binder.py | 3 +++
10+
1 file changed, 3 insertions(+)
11+
12+
diff --git a/build/webidl_binder.py b/build/webidl_binder.py
13+
index e9a56e5..faedf10 100644
14+
--- a/build/webidl_binder.py
15+
+++ b/build/webidl_binder.py
16+
@@ -1,6 +1,7 @@
17+
## JavascriptSubtitlesOctopus
18+
## Patched to:
19+
## - add integer pointers (IntPtr)
20+
+## - implement ByteString type
21+
## From https://github.com/emscripten-core/emscripten/blob/f36f9fcaf83db93e6a6d0f0cdc47ab6379ade139/tools/webidl_binder.py
22+
23+
# Copyright 2014 The Emscripten Authors. All rights reserved.
24+
@@ -327,6 +328,8 @@ def type_to_c(t, non_pointing=False):
25+
ret = 'char'
26+
elif t == 'Octet':
27+
ret = 'unsigned char'
28+
+ elif t == 'ByteString':
29+
+ ret = 'unsigned char*'
30+
elif t == 'Void':
31+
ret = 'void'
32+
elif t == 'String':
33+
--
34+
2.35.1
35+

0 commit comments

Comments
 (0)