762bbf5db2b749d174b72ec2eb37bf42718861c5: Bug 1347402 part 3 - get owner document via polymorphism; r?jwwang draft
Kaku Kuo <kaku@mozilla.com> - Wed, 15 Mar 2017 14:56:05 +0800 - rev 500354
Push 49701 by bmo:kaku@mozilla.com at Fri, 17 Mar 2017 03:54:30 +0000
Bug 1347402 part 3 - get owner document via polymorphism; r?jwwang MozReview-Commit-ID: GCr2xQyST4X
b5c8187ed786aff6c72462d665d79488c574b780: Bug 1347402 part 2 - open a GetOwnerDoc() interface at the MediaDecoderOwner; r?jwwang draft
Kaku Kuo <kaku@mozilla.com> - Wed, 15 Mar 2017 11:40:37 +0800 - rev 500353
Push 49701 by bmo:kaku@mozilla.com at Fri, 17 Mar 2017 03:54:30 +0000
Bug 1347402 part 2 - open a GetOwnerDoc() interface at the MediaDecoderOwner; r?jwwang Open a GetOwnerDoc() method to the MediaDecoderOwner interface and then we can get the owner document via a pointer to MediaDecoderOwner in MediaDecoder. MozReview-Commit-ID: JCzQDLx1MsU
6edf5eef27d110c00369e7da2562fd39591d7a99: Bug 1347402 part 1 - call DownloadSuspended() via polymorphism; r?jwwang draft
Kaku Kuo <kaku@mozilla.com> - Wed, 15 Mar 2017 11:20:57 +0800 - rev 500352
Push 49701 by bmo:kaku@mozilla.com at Fri, 17 Mar 2017 03:54:30 +0000
Bug 1347402 part 1 - call DownloadSuspended() via polymorphism; r?jwwang Instead of calling DownloadSuspended() via a pointer to a HTMLMediaElement, we should call DownloadSuspended() via a pointer to a MediaDecoderOwner. MozReview-Commit-ID: BvExQuchsWb
109f59230301fb1ff2715214cad5bb71be16003c: Bug 1347073 Get rid of UIEvent.isChar since it's not initialized properly on most platforms and the other browsers don't support this r?smaug draft
Masayuki Nakano <masayuki@d-toybox.com> - Tue, 14 Mar 2017 18:29:39 +0900 - rev 500351
Push 49700 by masayuki@d-toybox.com at Fri, 17 Mar 2017 03:43:59 +0000
Bug 1347073 Get rid of UIEvent.isChar since it's not initialized properly on most platforms and the other browsers don't support this r?smaug UIEvent.isChar is not supported by the other browsers and the value isn't initialized any platforms except on macOS. So, the value isn't useful and we have no reason to keep it. MozReview-Commit-ID: 4BLpo88gSZj
8e49481a9c9aab28760b27dbf45c213c0875e4a7: Bug 1259355 - so help me god; r?chunmin draft
Chun-Min Chang <chun.m.chang@gmail.com> - Fri, 17 Mar 2017 11:40:32 +0800 - rev 500350
Push 49699 by bmo:cchang@mozilla.com at Fri, 17 Mar 2017 03:42:11 +0000
Bug 1259355 - so help me god; r?chunmin
353b9b287dd1f6160d883dda5852c2f57fe85256: Bug 1347834 - update rust mp4 parser for audio codec specific data. r=kinetik draft
Alfredo.Yang <ayang@mozilla.com> - Fri, 17 Mar 2017 11:13:22 +0800 - rev 500349
Push 49698 by bmo:ayang@mozilla.com at Fri, 17 Mar 2017 03:34:37 +0000
Bug 1347834 - update rust mp4 parser for audio codec specific data. r=kinetik MozReview-Commit-ID: 29yu6VsFIE6
3f4f27c0a2bc9f0d200ee646d68666c0c8adda59: Bug 1347834 - retrieve audio codec specific config from rust parser. r=kinetik draft
Alfredo.Yang <ayang@mozilla.com> - Thu, 16 Mar 2017 16:06:28 +0800 - rev 500348
Push 49698 by bmo:ayang@mozilla.com at Fri, 17 Mar 2017 03:34:37 +0000
Bug 1347834 - retrieve audio codec specific config from rust parser. r=kinetik MozReview-Commit-ID: 2x4sN0e7c5j
593f965bdf322f030de5ab393ea8a94cb7dcbb3a: Bug 1340468 - Notify formautofill add-on of which item is being selected. r=MattN draft
Ray Lin <ralin@mozilla.com> - Fri, 24 Feb 2017 10:15:55 +0800 - rev 500347
Push 49697 by bmo:ralin@mozilla.com at Fri, 17 Mar 2017 03:30:48 +0000
Bug 1340468 - Notify formautofill add-on of which item is being selected. r=MattN MozReview-Commit-ID: 3YfW35Zon1Q
cd6d80e9cac9564d8cc60578f7e1b74bae84eb3b: add mock data to result draft
Ray Lin <ralin@mozilla.com> - Mon, 20 Feb 2017 14:38:58 +0800 - rev 500346
Push 49697 by bmo:ralin@mozilla.com at Fri, 17 Mar 2017 03:30:48 +0000
add mock data to result MozReview-Commit-ID: HtrsgGDytwh
c7276cb639bed2fb2580cfdbb469a4754320b001: Bug 1347809 Set Selection::mCalledByJS to false before moving focus in NotifySelectionListeners() because non-*JS() methods don't set it to false r?smaug draft
Masayuki Nakano <masayuki@d-toybox.com> - Thu, 16 Mar 2017 17:15:20 +0900 - rev 500345
Push 49696 by masayuki@d-toybox.com at Fri, 17 Mar 2017 03:29:47 +0000
Bug 1347809 Set Selection::mCalledByJS to false before moving focus in NotifySelectionListeners() because non-*JS() methods don't set it to false r?smaug Currently, Selection::NotifySelectionListeners() moves focus before setting mCalledByJS to false. Therefore, if moving focus causes some calls of internal Selection methods, it may cause moving focus due to mCalledByJS being still true. So, mCalledByJS should be set to false before moving focus in NotifySelectionListeners(). MozReview-Commit-ID: F879bOmhZlv
925f1825593d65a1e76d428d9d2f2af9360a65ba: Bug 1347031 - Move the MediaCache off of opening its temporary file fd synchronously in the content process. r=jwwang draft
Chris Pearce <cpearce@mozilla.com> - Fri, 17 Mar 2017 09:54:23 +1300 - rev 500344
Push 49695 by bmo:cpearce@mozilla.com at Fri, 17 Mar 2017 03:26:35 +0000
Bug 1347031 - Move the MediaCache off of opening its temporary file fd synchronously in the content process. r=jwwang In bug 1346987 we're attempting to remove uses of the NS_OpenAnonymousTemporaryFile() in the content process as it sends a synchronous IPC to the parent process on the main thread, which can cause UI jank. This patch makes the MediaCache use the async anonymous temporary file creation function added in bug 1346987. The file descriptor is held by the FileBlockCache. This object buffers data passed to it in memory, and defers writing of said data to another thread. I added the async wait for the file descriptor to be inside that async "defer to other thread" step. This means that while the content process is waiting for the file descriptor to come down from the parent process, we'll buffer media data being streamed in memory. Given that our MSE implementation will buffer up to 100MB of media data in memory anyway, it seems that more buffering in the src=url case while we wait for an async IPC to do a round trip to the main process is acceptable. MozReview-Commit-ID: 3OTBTWw5pr0
6ad562a333aaf781bc94dd680f55be1cb452edea: Bug 1347809 Set Selection::mCalledByJS to false before moving focus in NotifySelectionListeners() because non-*JS() methods don't set it to false r?smaug draft
Masayuki Nakano <masayuki@d-toybox.com> - Thu, 16 Mar 2017 17:15:20 +0900 - rev 500343
Push 49694 by masayuki@d-toybox.com at Fri, 17 Mar 2017 03:23:12 +0000
Bug 1347809 Set Selection::mCalledByJS to false before moving focus in NotifySelectionListeners() because non-*JS() methods don't set it to false r?smaug Currently, Selection::NotifySelectionListeners() moves focus before setting mCalledByJS to false. Therefore, that causes internal selection API calls and it may cause moving focus. So, mCalledByJS should be set to false before moving focus in NotifySelectionListeners(). MozReview-Commit-ID: F879bOmhZlv
592ff740f1d7a92b041981240d44f06eba85903f: Bug 1339331 TextEventDispatcher should replace \r in composition string with \n and TextComposition should allow to input \n with composition events r?m_kato draft
Masayuki Nakano <masayuki@d-toybox.com> - Thu, 16 Mar 2017 16:26:43 +0900 - rev 500342
Push 49694 by masayuki@d-toybox.com at Fri, 17 Mar 2017 03:23:12 +0000
Bug 1339331 TextEventDispatcher should replace \r in composition string with \n and TextComposition should allow to input \n with composition events r?m_kato According to ATOK's behavior, IME may send different line breaker from its platform's standard. Therefore, we should treat \r as \n too. Additionally, currently, TextComposition doesn't allow to input \n with composition. However, this was added for preventing to see odd control characters as boxes with code point. Therefore, we should allow \n for IMEs. (It was allowed, this limitation is unexpected when I reviewed the patch to reject control characters in TextComposition.) MozReview-Commit-ID: DzGSMgp89Av
cac3f211d9b89f9cc89fe8cbaf7bf90d66ff7e72: Bug 1347829 - If the timestamp "<00:00.005" missing the last ">" character, we still parse it. r=alwu draft
bechen <bechen@mozilla.com> - Fri, 17 Mar 2017 11:17:52 +0800 - rev 500341
Push 49693 by bechen@mozilla.com at Fri, 17 Mar 2017 03:18:56 +0000
Bug 1347829 - If the timestamp "<00:00.005" missing the last ">" character, we still parse it. r=alwu MozReview-Commit-ID: KzW1uTSyDE
542d68c1d304cc9283d9ab2d7f35f3f0563c743e: Bug 1344604 - part2: Return empty DocumentFragment if the ConvertCueToDOMTree() return null. r=alwu draft
bechen <bechen@mozilla.com> - Thu, 16 Mar 2017 18:12:21 +0800 - rev 500340
Push 49693 by bechen@mozilla.com at Fri, 17 Mar 2017 03:18:56 +0000
Bug 1344604 - part2: Return empty DocumentFragment if the ConvertCueToDOMTree() return null. r=alwu It is a regression of bug1307710, if something wrong during the ConvertCueToDOMTree() in vtt.jsm, we will get null ptr. MozReview-Commit-ID: LSQrJIhBzRU
7c9ec4c2d903e426998d1c65fc83eed47fb12f84: Bug 1344604 - Return null if there is an incomplete tag in nextToken(). r=alwu draft
bechen <bechen@mozilla.com> - Thu, 16 Mar 2017 17:40:21 +0800 - rev 500339
Push 49693 by bechen@mozilla.com at Fri, 17 Mar 2017 03:18:56 +0000
Bug 1344604 - Return null if there is an incomplete tag in nextToken(). r=alwu MozReview-Commit-ID: 5mvFhaAx2s5
c2da17935c3519474e762722f325aed9dea8ba8d: Bug 1347402 part 5 - create MediaElementGMPCrashHelper in HTMLMediaElement.cpp; r?jwwang, cpearce draft
Kaku Kuo <kaku@mozilla.com> - Thu, 16 Mar 2017 11:16:15 +0800 - rev 500338
Push 49692 by bmo:kaku@mozilla.com at Fri, 17 Mar 2017 03:16:50 +0000
Bug 1347402 part 5 - create MediaElementGMPCrashHelper in HTMLMediaElement.cpp; r?jwwang, cpearce Move the creation of MediaElementGMPCrashHelper out from MediaDecoder.cpp which reduces the dependency of MediaDecoder to HTMLMediaElement. MozReview-Commit-ID: E60aMfcFr7V
077a28833893548ff8fbd69a3bc439d8de822e96: Bug 1347402 part 4 - move ConstructMediaTracks/RemoveMediaTracks to HTMLMediaElemnt; r?jwwang draft
Kaku Kuo <kaku@mozilla.com> - Wed, 15 Mar 2017 17:33:21 +0800 - rev 500337
Push 49692 by bmo:kaku@mozilla.com at Fri, 17 Mar 2017 03:16:50 +0000
Bug 1347402 part 4 - move ConstructMediaTracks/RemoveMediaTracks to HTMLMediaElemnt; r?jwwang ConstructMediaTracks and RemoveMediaTracks are actually HTMLMediaElement's responsibilities. MozReview-Commit-ID: 8lOdzD4pN7N
f687e8abd41108f8db7d6d60f139ebebb4147e43: Bug 1347402 part 3 - get owner document via polymorphism; r?jwwang draft
Kaku Kuo <kaku@mozilla.com> - Wed, 15 Mar 2017 14:56:05 +0800 - rev 500336
Push 49692 by bmo:kaku@mozilla.com at Fri, 17 Mar 2017 03:16:50 +0000
Bug 1347402 part 3 - get owner document via polymorphism; r?jwwang MozReview-Commit-ID: GCr2xQyST4X
6c120a119ccda07b8ee4085884cb174920829cc8: Bug 1347402 part 2 - open a GetOwnerDoc() interface at the MediaDecoderOwner; r?jwwang draft
Kaku Kuo <kaku@mozilla.com> - Wed, 15 Mar 2017 11:40:37 +0800 - rev 500335
Push 49692 by bmo:kaku@mozilla.com at Fri, 17 Mar 2017 03:16:50 +0000
Bug 1347402 part 2 - open a GetOwnerDoc() interface at the MediaDecoderOwner; r?jwwang Open a GetOwnerDoc() interface at the MediaDecoderOwner and then we can get the owner document via a pointer to MediaDecoderOwner in MediaDecoder. MozReview-Commit-ID: JCzQDLx1MsU
(0) -300000 -100000 -30000 -10000 -3000 -1000 -300 -100 -50 -20 +20 +50 +100 +300 +1000 +3000 +10000 +30000 +100000 +300000 tip