da7ddc04b4314de9c2adefbb6ce167bd6c3c2bc3: Bug 1032671 - Part 3. Set font.name-list.emoji for some platforms. r=jfkthame
Makoto Kato <m_kato@ga2.so-net.ne.jp> - Sun, 12 Nov 2017 16:30:23 +0900 - rev 394790
Push 56450 by m_kato@ga2.so-net.ne.jp at Mon, 04 Dec 2017 07:32:34 +0000
Bug 1032671 - Part 3. Set font.name-list.emoji for some platforms. r=jfkthame - Segoe UI Emoji for Windows 8+ - EmojiOne Mozilla for Windows 7 - Apple Color Emoji for OSX - EmojiOne Mozilla for GTK(Linux). fontconfig doesn't support emoji as family name. - Noto Color Emoji for Android MozReview-Commit-ID: GOkOFRujk93
69219385100a1b5ebced5b323bf5d5de27c76566: Bug 1032671 - Part 2. Use font.name-list.emoji preference for emoji presenration. r=jfkthame
Makoto Kato <m_kato@ga2.so-net.ne.jp> - Sun, 12 Nov 2017 16:24:58 +0900 - rev 394789
Push 56450 by m_kato@ga2.so-net.ne.jp at Mon, 04 Dec 2017 07:32:34 +0000
Bug 1032671 - Part 2. Use font.name-list.emoji preference for emoji presenration. r=jfkthame MozReview-Commit-ID: DbrGeXzCpNT
e5b43d990d7029f9666225c0fa7eddd3b751a8c8: Bug 1032671 - Part 1. Add emoji presentation API for UTR#51. r=jfkthame
Makoto Kato <m_kato@ga2.so-net.ne.jp> - Sun, 12 Nov 2017 16:06:18 +0900 - rev 394788
Push 56450 by m_kato@ga2.so-net.ne.jp at Mon, 04 Dec 2017 07:32:34 +0000
Bug 1032671 - Part 1. Add emoji presentation API for UTR#51. r=jfkthame From ICU 57, ICU supports emoji presentation as draft API. MozReview-Commit-ID: 6JgQMv4Ky9m
8295eb601d27dbb4aaa15d879bc9795a200789ee: Backed out changeset 7a2629417a47 (bug 1422662) for bustage r=backout on a CLOSED TREE
Coroiu Cristina <ccoroiu@mozilla.com> - Mon, 04 Dec 2017 09:21:51 +0200 - rev 394787
Push 56449 by ccoroiu@mozilla.com at Mon, 04 Dec 2017 07:22:19 +0000
Backed out changeset 7a2629417a47 (bug 1422662) for bustage r=backout on a CLOSED TREE
7a2629417a4707f15b8c35c8fccecee530904e73: Bug 1422662 - Move BaseMediaResource::Close() into MediaResource class. r=jwwang
Chris Pearce <cpearce@mozilla.com> - Mon, 04 Dec 2017 16:29:35 +1300 - rev 394786
Push 56448 by cpearce@mozilla.com at Mon, 04 Dec 2017 06:57:20 +0000
Bug 1422662 - Move BaseMediaResource::Close() into MediaResource class. r=jwwang For the project to export Gecko's media stack and import it into Servo, I need a way to shutdown an abstract MediaResource. So I'd like to move BaseMediaResource::Close() up into MediaResource class. MozReview-Commit-ID: 9JmxJPs02PN
944ec9b4d0ef2b6ef19e2ae9d82190867027ca46: Bug 914817 - [ja] Update icon for Rakuten searchplugin, switch SearchForm to https r=mkaply
Francesco Lodolo (:flod) <flod@lodolo.net> - Sun, 03 Dec 2017 09:30:31 +0100 - rev 394785
Push 56447 by francesco.lodolo@mozillaitalia.org at Mon, 04 Dec 2017 06:34:12 +0000
Bug 914817 - [ja] Update icon for Rakuten searchplugin, switch SearchForm to https r=mkaply MozReview-Commit-ID: FJsgvRgtFuh
243f87297824e1120e510f383c57785910cb1119: Bug 1421793 - nsPACMan needs to be shutdown before dtor; r=schien
Liang-Heng Chen <xeonchen@gmail.com> - Fri, 01 Dec 2017 15:10:36 +0800 - rev 394784
Push 56446 by gachen@mozilla.com at Mon, 04 Dec 2017 06:14:52 +0000
Bug 1421793 - nsPACMan needs to be shutdown before dtor; r=schien MozReview-Commit-ID: D2aKTC8s9MQ
17cfd08531e319b1f3e9167d176d8bed1b1bba72: Bug 1422013 - Invalidate image items for webrender. r=mattwoodrow
Ethan Lin <ethlin@mozilla.com> - Mon, 04 Dec 2017 11:12:25 +0800 - rev 394783
Push 56445 by ethlin@mozilla.com at Mon, 04 Dec 2017 06:10:05 +0000
Bug 1422013 - Invalidate image items for webrender. r=mattwoodrow MozReview-Commit-ID: 701MZhcrf4l
81958c54d5571136c5cb5eb0df71bb5c9748a6fa: Bug 1422201: Only handle remote ids passed to IAccessible::accChild on the root accessible. r=MarcoZ
James Teh <jteh@mozilla.com> - Mon, 04 Dec 2017 15:32:56 +1000 - rev 394782
Push 56444 by mzehe@mozilla.com at Mon, 04 Dec 2017 06:09:10 +0000
Bug 1422201: Only handle remote ids passed to IAccessible::accChild on the root accessible. r=MarcoZ Previously, we could return remote accessibles which weren't actually descendants of the accessible on which accChild was called. For example, calling accChild on a local document with the id of a remote document would happily return the remote accessible. This confused clients such as NVDA which use accChild to check whether something is a descendant of a document. MozReview-Commit-ID: 8mJ4m6RC3r2
3176ff271bd7ee01b48faa09d93590e902245975: Bug 1421859 - Remove flat_icon now that it is unused. r=nechen
Ian Moody <moz-ian@perix.co.uk> - Sun, 03 Dec 2017 18:44:53 +0000 - rev 394781
Push 56443 by nechen@mozilla.com at Mon, 04 Dec 2017 05:26:37 +0000
Bug 1421859 - Remove flat_icon now that it is unused. r=nechen MozReview-Commit-ID: 7wJ0JL2fBFC
544693ed0c93719f34c144054f17b5e2a99fb5da: Bug 1421859 - Use ic_status_logo instead of flat_icon in MediaControl and TabReceived notifications. r=nechen
Ian Moody <moz-ian@perix.co.uk> - Sun, 03 Dec 2017 18:43:38 +0000 - rev 394780
Push 56443 by nechen@mozilla.com at Mon, 04 Dec 2017 05:26:37 +0000
Bug 1421859 - Use ic_status_logo instead of flat_icon in MediaControl and TabReceived notifications. r=nechen ic_status_logo is what all the other notifications use, and flat_icon hasn't been updated to the new photon logo. MozReview-Commit-ID: Luo5ibVmXvk
b4af5fae3d9b878b42d245d5c61fa30dc35d5307: Bug 1408227 - part 6: WSRunObject::CheckTrailingNBSP() should take |const EditorRawDOMPoint&| instead of a set of container node and offset in it r=m_kato
Masayuki Nakano <masayuki@d-toybox.com> - Wed, 22 Nov 2017 02:45:08 +0900 - rev 394779
Push 56442 by masayuki@d-toybox.com at Mon, 04 Dec 2017 04:35:03 +0000
Bug 1408227 - part 6: WSRunObject::CheckTrailingNBSP() should take |const EditorRawDOMPoint&| instead of a set of container node and offset in it r=m_kato The name, WSRunObject::CheckTrailingNBSP(), is unclear what it does. It replaces previous character of specified point with ASCII space if it is NBSP and it's not necessary. So, it should be renamed to ReplacePreviousNBSPIfUnncessary(). Additionally, it should take |const EditorRawDOMPoint&| instead of a set of container node and offset in it. MozReview-Commit-ID: 2vq46OiAzo6
ed55a1cbf4ab35f9116f39a3c1ced22470156a91: Bug 1408227 - part 5: Redesign GetCharAfter(), GetCharBefore(), GetWSPointAfter() and GetWSPointBefore() of WSRunObject r=m_kato
Masayuki Nakano <masayuki@d-toybox.com> - Wed, 22 Nov 2017 02:12:27 +0900 - rev 394778
Push 56442 by masayuki@d-toybox.com at Mon, 04 Dec 2017 04:35:03 +0000
Bug 1408227 - part 5: Redesign GetCharAfter(), GetCharBefore(), GetWSPointAfter() and GetWSPointBefore() of WSRunObject r=m_kato WSRunObject::GetChar(After|Before)(nsINode*, int32_t) returns next/previous character's DOM point as WSPoint. If the container node is a text node in mNodeArray, it calls WSRunObject::GetChar(After|Before)(const WSPoint&) which returns next/previous offset if the point isn't end/start of the text node. If the point is at end/start of the text node, it returns start/end of next/previous text node in mNodeArray. If the container node is not a text node in mNodeArray, it calls WSRunObject::GetWSPoint(After|Before)(). It looks for next/previous text node in mNodeArray from the point. Then, it calls WSRunObject::GetChar(After|Before)(const WSPoint&) and returns the result. So, we should rename GetCharAfter() to GetNextCharPoint(), GetCharBefore() to GetPreviousCharPoint(), GetWSPointAfter() to GetNextCharPointInternal() and GetWSPointBefore() to GetPreviousCharPointInternal(). Then, they should take |const EditorRawDOMPoint&| instead of a set of container node and offset in it. So, looks like that "WS"RunObject is not good name for this class, perhaps, AutoTextRunManager or something? But I'm still not sure. MozReview-Commit-ID: 85cX3MdlFwz
baed101e364b48543a24539dc842c759ac6bb6fb: Bug 1408227 - part 4: WSRunObject::DeleteChars() should take two |const EditorRawDOMPoint&| arguments to specify a range to remove r=m_kato
Masayuki Nakano <masayuki@d-toybox.com> - Wed, 22 Nov 2017 01:00:35 +0900 - rev 394777
Push 56442 by masayuki@d-toybox.com at Mon, 04 Dec 2017 04:35:03 +0000
Bug 1408227 - part 4: WSRunObject::DeleteChars() should take two |const EditorRawDOMPoint&| arguments to specify a range to remove r=m_kato WSRunObject::DeleteChars() should take two |const EditorRawDOMPoint&| arguments to represent a range to remove. Additionally, this renames it to DeleteRange() because it also removes any nodes in the range. So, "Chars" isn't good word for this method's name. MozReview-Commit-ID: 5Dmxnia1JPO
33d2e22dc648bbcce4a1e591d2551972754f0a8b: Bug 1408227 - part 3: Redesign WSRunObject::FindRun() with EditorRawDOMPoint r=m_kato
Masayuki Nakano <masayuki@d-toybox.com> - Tue, 21 Nov 2017 19:03:03 +0900 - rev 394776
Push 56442 by masayuki@d-toybox.com at Mon, 04 Dec 2017 04:35:03 +0000
Bug 1408227 - part 3: Redesign WSRunObject::FindRun() with EditorRawDOMPoint r=m_kato WSRunObject::FindRun() finds the nearest run from aPoint to specified direction. So, it uses nsContentUtils::ComparePoints() a lot. Therefore, it should use an overload which takes RawRangeBoundary. Although, it's not optimized for RawRangeBoundary, but if it'd be optimized, this method becomes faster. And this patch renames it to FindNearestRun(). MozReview-Commit-ID: 2NkR5E1st6d
6fc7cf0d7a86cd79f7820cb97695ae2707847349: Bug 1408227 - part 2: WSRunObject::InsertBreak() should take |const EditorRawDOMPoint&| as an argument r=m_kato
Masayuki Nakano <masayuki@d-toybox.com> - Tue, 21 Nov 2017 18:12:12 +0900 - rev 394775
Push 56442 by masayuki@d-toybox.com at Mon, 04 Dec 2017 04:35:03 +0000
Bug 1408227 - part 2: WSRunObject::InsertBreak() should take |const EditorRawDOMPoint&| as an argument r=m_kato WSRunObject::InsertBreak() should take |const EditorRawDOMPoint&| instead of a pair of container node and offset in it. MozReview-Commit-ID: 38OAn4dvR7x
13d9a39c2f97af2c9a6af25133da79255626080b: Bug 1408227 - part 1: TextEditor::CreateBRImpl() should take |const EditorRawDOMPoint&| for insertion point of new <br> element r=m_kato
Masayuki Nakano <masayuki@d-toybox.com> - Tue, 21 Nov 2017 14:38:19 +0900 - rev 394774
Push 56442 by masayuki@d-toybox.com at Mon, 04 Dec 2017 04:35:03 +0000
Bug 1408227 - part 1: TextEditor::CreateBRImpl() should take |const EditorRawDOMPoint&| for insertion point of new <br> element r=m_kato TextEditor::CreateBRImpl() should take |const EditorRawDOMPoint&| as insertion point of new <br> element. Additionally, it doesn't need to have out argument for the point of after <br> element because callers can get it with EditorRawDOMPoint(nsINode*) and the new <br> node, and calling AdvanceOffset(). This cost must be enough cheap. MozReview-Commit-ID: Hxawz3D2dCd
7d20fee48462024b8c44170d59d946560023630c: Bug 1421504 - EditorBase should move children carefully r=m_kato
Masayuki Nakano <masayuki@d-toybox.com> - Wed, 29 Nov 2017 17:57:00 +0900 - rev 394773
Push 56441 by masayuki@d-toybox.com at Mon, 04 Dec 2017 04:33:33 +0000
Bug 1421504 - EditorBase should move children carefully r=m_kato While moving children of a container to another container, mutation observer may move children before moving and/or move node immediately after the insertion point. Therefore, EditorBase should store all children which should be moved with a local variable. Then, move one by one carefully. E.g., if a child before being moved is moved to different container, it shouldn't be moved because JS already handles it as expected for the web app. If next sibling of the insertion point has been moved, EditorBase should stop moving the remaining children because it may move children to odd position and it may cause dataloss. This patch creates EditorBase::MoveChildren() and it moves children carefully with above checks. Additionally, making its callers simpler, this patch also creates EditorBase::MovePreviousSiblings() and MoveAllChildren(). MozReview-Commit-ID: FJDdSqXIlqD
5a71560d58db074a628624d99f12d84e0d2d07b7: Backed out 1 changesets (bug 1422466) for build bustage on a CLOSED TREE
Gurzau Raul <rgurzau@mozilla.com> - Mon, 04 Dec 2017 05:34:10 +0200 - rev 394772
Push 56440 by rgurzau@mozilla.com at Mon, 04 Dec 2017 03:34:52 +0000
Backed out 1 changesets (bug 1422466) for build bustage on a CLOSED TREE Backed out changeset 719224b9a2dd (bug 1422466)
719224b9a2dd25180b4fe58d039dc91d9c579259: Bug 1422466. Drop a copy from TextDrawTarget::FillGlyphs. r=Gankro
Jeff Muizelaar <jmuizelaar@mozilla.com> - Sat, 02 Dec 2017 13:45:57 -0500 - rev 394771
Push 56439 by jmuizelaar@mozilla.com at Mon, 04 Dec 2017 03:15:45 +0000
Bug 1422466. Drop a copy from TextDrawTarget::FillGlyphs. r=Gankro We try hard to ensure that the glyph buffers are of the same type.
(0) -300000 -100000 -30000 -10000 -3000 -1000 -300 -100 -50 -20 +20 +50 +100 +300 +1000 +3000 +10000 +30000 tip