Bug 153984

Summary: The Operator Dictionary lacks entries for U+1EEF0 and U+1EEF1
Product: WebKit Reporter: Frédéric Wang (:fredw) <fred.wang>
Component: MathMLAssignee: Frédéric Wang (:fredw) <fred.wang>
Status: RESOLVED FIXED    
Severity: Normal CC: commit-queue, dbarton, dr.khaled.hosny, esprehn+autocc, glenn, kondapallykalyan, mrobinson
Priority: P2    
Version: WebKit Nightly Build   
Hardware: Unspecified   
OS: Unspecified   
URL: https://lists.w3.org/Archives/Public/www-math/2016Feb/0000.html
See Also: https://bugzilla.mozilla.org/show_bug.cgi?id=1246657
Bug Depends on: 160462    
Bug Blocks: 161306    
Attachments:
Description Flags
WIP Patch
none
Patch WIP
none
Patch
darin: review+
Patch none

Frédéric Wang (:fredw)
Reported 2016-02-08 07:34:52 PST
These are mentioned on http://www.unicode.org/charts/PDF/U1EE00.pdf referred by the MathML 3 recommendation, but absent from their suggested operator dictionary. I don't think any math font supports them yet. Stretching operators The following operators stretch based on the width of the text that is displayed below or above them. 1EEF0 ARABIC MATHEMATICAL OPERATOR MEEM WITH HAH WITH TATWEEL • used in Arabic mathematics to denote summation • stretched at the tatweel → 2211 ∑ n-ary summation 1EEF1 ARABIC MATHEMATICAL OPERATOR HAH WITH DAL • used in Persian mathematics to denote limits • stretched between the hah and the dal
Attachments
WIP Patch (16.53 KB, patch)
2016-08-23 06:41 PDT, Frédéric Wang (:fredw) 		no flags
DetailsFormatted DiffDiff
Patch WIP (33.00 KB, patch)
2016-08-24 08:06 PDT, Frédéric Wang (:fredw) 		no flags
DetailsFormatted DiffDiff
Patch (39.29 KB, patch)
2016-08-25 05:54 PDT, Frédéric Wang (:fredw) 		darin: review+
DetailsFormatted DiffDiff
Patch (42.13 KB, patch)
2016-08-28 23:08 PDT, Frédéric Wang (:fredw) 		no flags
DetailsFormatted DiffDiff
Show Obsolete (2) View All Add attachment proposed patch, testcase, etc.
Frédéric Wang (:fredw)
Comment 1 2016-08-23 06:41:54 PDT
Created attachment 286710 [details] WIP Patch
Frédéric Wang (:fredw)
Comment 2 2016-08-23 06:44:37 PDT
*** Bug 122296 has been marked as a duplicate of this bug. ***
Frédéric Wang (:fredw)
Comment 3 2016-08-24 08:06:56 PDT
Created attachment 286853 [details] Patch WIP Adding a test and fixing a coding error with explicit prefix. I'll add a test for horizontal stretchiness tomorrow.
Frédéric Wang (:fredw)
Comment 4 2016-08-25 05:54:35 PDT
Created attachment 286963 [details] Patch
Darin Adler
Comment 5 2016-08-27 17:17:16 PDT
Comment on attachment 286963 [details] Patch View in context: https://bugs.webkit.org/attachment.cgi?id=286963&action=review > Source/WebCore/mathml/MathMLOperatorDictionary.cpp:1109 > +// The following two non-BMP characters are handled separately so that the arrays 'dictionary' and > +// 'horizontalOperators' can use UChar instead of UChar32 and avoid an increase of memory footprint. I don’t think this is the right call. The additional memory footprint is about 2kB. We should keep the code simple instead.
Frédéric Wang (:fredw)
Comment 6 2016-08-28 01:08:34 PDT
(In reply to comment #5) > I don’t think this is the right call. The additional memory footprint is > about 2kB. We should keep the code simple instead. OK, I'll rewrite a simpler patch then.
Frédéric Wang (:fredw)
Comment 7 2016-08-28 23:08:30 PDT
Created attachment 287248 [details] Patch
Frédéric Wang (:fredw)
Comment 8 2016-08-28 23:57:54 PDT
Committed r205111: <http://trac.webkit.org/changeset/205111>
Note You need to log in before you can comment on or make changes to this bug.