600afe2d45881ef9ce2abcec01e9457c816772d2: Bug 1529518: Polish extension item. r=ladybenko,Ola
Daisuke Akatsuka <dakatsuka@mozilla.com> - Tue, 19 Mar 2019 11:17:04 +0000 - rev 464999
Push 112486 by opoprus@mozilla.com at Tue, 19 Mar 2019 16:41:04 +0000
Bug 1529518: Polish extension item. r=ladybenko,Ola Depends on D21887 Differential Revision: https://phabricator.services.mozilla.com/D21888
9f76239ec701bda25ba5502bbf39fc4439ac44ed: Bug 1529518: Polish tab item. r=ladybenko,Ola
Daisuke Akatsuka <dakatsuka@mozilla.com> - Tue, 19 Mar 2019 11:15:06 +0000 - rev 464998
Push 112486 by opoprus@mozilla.com at Tue, 19 Mar 2019 16:41:04 +0000
Bug 1529518: Polish tab item. r=ladybenko,Ola Differential Revision: https://phabricator.services.mozilla.com/D21887
88b47716a875abe4592c275234d1440cf4fbb99a: Bug 1444133 - Have an option to have view-source open in a separate (browser) window, not separate tab, r=mconley
Dhyey Thakore <dhyey35@gmail.com> - Tue, 19 Mar 2019 11:17:18 +0000 - rev 464997
Push 112486 by opoprus@mozilla.com at Tue, 19 Mar 2019 16:41:04 +0000
Bug 1444133 - Have an option to have view-source open in a separate (browser) window, not separate tab, r=mconley Differential Revision: https://phabricator.services.mozilla.com/D18071
333566a1cbc842ba8e99b798ba9eea6a2a98d3dc: Bug 1524961 - Update mozlog's license to MPL 2.0. r=raphael
pgadige <poojagadige@gmail.com> - Tue, 19 Mar 2019 10:59:24 +0000 - rev 464996
Push 112486 by opoprus@mozilla.com at Tue, 19 Mar 2019 16:41:04 +0000
Bug 1524961 - Update mozlog's license to MPL 2.0. r=raphael Differential Revision: https://phabricator.services.mozilla.com/D23407
93d061fd45de60347ed96b70b0271e83db425bbd: Bug 1534693 - Bail out of font-family search in GlobalFontFallback once we've found a font with an exact style match. r=jwatt
Jonathan Kew <jkew@mozilla.com> - Tue, 19 Mar 2019 10:49:53 +0000 - rev 464995
Push 112486 by opoprus@mozilla.com at Tue, 19 Mar 2019 16:41:04 +0000
Bug 1534693 - Bail out of font-family search in GlobalFontFallback once we've found a font with an exact style match. r=jwatt Differential Revision: https://phabricator.services.mozilla.com/D23162
9ba365837090fe1712e9daac640e188dbe813574: Bug 1534313 - Make GetDeviceCollection accessible only by the enumerator. r=padenot
Alex Chronopoulos <achronop@gmail.com> - Tue, 19 Mar 2019 10:40:25 +0000 - rev 464994
Push 112486 by opoprus@mozilla.com at Tue, 19 Mar 2019 16:41:04 +0000
Bug 1534313 - Make GetDeviceCollection accessible only by the enumerator. r=padenot Differential Revision: https://phabricator.services.mozilla.com/D22989
f6916040dea3ed12ee0cc631b298804b56f3bf25: Bug 1534313 - Remove direct calls to GetDeviceCollection method. r=padenot
Alex Chronopoulos <achronop@gmail.com> - Tue, 19 Mar 2019 10:40:12 +0000 - rev 464993
Push 112486 by opoprus@mozilla.com at Tue, 19 Mar 2019 16:41:04 +0000
Bug 1534313 - Remove direct calls to GetDeviceCollection method. r=padenot Differential Revision: https://phabricator.services.mozilla.com/D22988
a7fb15aa84926a24016bd48357ed8852f883fe21: Bug 1534313 - Add new method in enumerator to get the device info from device name. r=padenot
Alex Chronopoulos <achronop@gmail.com> - Tue, 19 Mar 2019 10:40:05 +0000 - rev 464992
Push 112486 by opoprus@mozilla.com at Tue, 19 Mar 2019 16:41:04 +0000
Bug 1534313 - Add new method in enumerator to get the device info from device name. r=padenot Differential Revision: https://phabricator.services.mozilla.com/D22987
383d67dc98709c423bfc5c60b82f39e138d512aa: Bug 1534313 - Put the default audio output first in the list of devices. r=padenot
Alex Chronopoulos <achronop@gmail.com> - Tue, 19 Mar 2019 10:39:57 +0000 - rev 464991
Push 112486 by opoprus@mozilla.com at Tue, 19 Mar 2019 16:41:04 +0000
Bug 1534313 - Put the default audio output first in the list of devices. r=padenot Differential Revision: https://phabricator.services.mozilla.com/D22986
7626a0df2e262a4afd6220a0dec6bef8d6790706: Bug 1488673 - part10 : enable wpt tests. r=heycam
Alastor Wu <alwu@mozilla.com> - Tue, 19 Mar 2019 03:34:23 +0000 - rev 464990
Push 112486 by opoprus@mozilla.com at Tue, 19 Mar 2019 16:41:04 +0000
Bug 1488673 - part10 : enable wpt tests. r=heycam Differential Revision: https://phabricator.services.mozilla.com/D22885
2a0158d3e47c84ef702822e512efa3cfd511cf52: Bug 1488673 - part9 : disable vtt wpt rendering tests. r=heycam
Alastor Wu <alwu@mozilla.com> - Tue, 19 Mar 2019 10:41:23 +0000 - rev 464989
Push 112486 by opoprus@mozilla.com at Tue, 19 Mar 2019 16:41:04 +0000
Bug 1488673 - part9 : disable vtt wpt rendering tests. r=heycam These wpt tests are using the wrong ref html which always assume the cue should be on the bottom of the video element. According to the spec, the result of cue's position is calulated by rendering area and it won't always be on the bottom of the video. Differential Revision: https://phabricator.services.mozilla.com/D20029
c2aad16bf5448cf829f381cde142eaa721c401c8: Bug 1488673 - part8 : remove unnecessary anonymous function. r=heycam
Alastor Wu <alwu@mozilla.com> - Tue, 19 Mar 2019 03:31:43 +0000 - rev 464988
Push 112486 by opoprus@mozilla.com at Tue, 19 Mar 2019 16:41:04 +0000
Bug 1488673 - part8 : remove unnecessary anonymous function. r=heycam There is no need to have an anonymous function after we replace all `var` to `let`, now those variables won't pollute outer scopes. Differential Revision: https://phabricator.services.mozilla.com/D20028
5934ef641056cd6a40f4fb487f430de3e1d6ff0e: Bug 1488673 - part7 : huge replacement of 'var' by 'let'. r=heycam
Alastor Wu <alwu@mozilla.com> - Tue, 19 Mar 2019 03:31:20 +0000 - rev 464987
Push 112486 by opoprus@mozilla.com at Tue, 19 Mar 2019 16:41:04 +0000
Bug 1488673 - part7 : huge replacement of 'var' by 'let'. r=heycam In order to keep the local variable only visible inside the function scope, we should only use 'let' for those variables. Differential Revision: https://phabricator.services.mozilla.com/D20027
3f6a9b725f218a15e79d87f4106b6ac20a1182ef: Bug 1488673 - part6 : remove unused functions. r=heycam
Alastor Wu <alwu@mozilla.com> - Tue, 19 Mar 2019 03:30:32 +0000 - rev 464986
Push 112486 by opoprus@mozilla.com at Tue, 19 Mar 2019 16:41:04 +0000
Bug 1488673 - part6 : remove unused functions. r=heycam Remove those funcitons which are used to implement the previous way to move the cue box. Differential Revision: https://phabricator.services.mozilla.com/D19610
a1841a4aa3f1188b9ecd9666dcbdab58e1258778: Bug 1488673 - part5 : find the best position for the cue box in order not to overlap it with other cue boxes. r=heycam
Alastor Wu <alwu@mozilla.com> - Tue, 19 Mar 2019 03:30:09 +0000 - rev 464985
Push 112486 by opoprus@mozilla.com at Tue, 19 Mar 2019 16:41:04 +0000
Bug 1488673 - part5 : find the best position for the cue box in order not to overlap it with other cue boxes. r=heycam According to the spec 7.2.10 [1], the step13 to step21 (snap-to-line is true) and the step3 to step5 (snap-to-line is false), we need to find the best position where the cue box has least overlap with other cue boxes. In addition, in order not to be affected by CSS transformation, use non-tranformed attribute in `BoxPosition`. [1] https://w3c.github.io/webvtt/#ref-for-webvtt-cue-snap-to-lines-flag-12 https://w3c.github.io/webvtt/#ref-for-webvtt-cue-snap-to-lines-flag-13 Differential Revision: https://phabricator.services.mozilla.com/D22578
9e2f80f5ec79e1d33826065ef40fb799da3ad574: Bug 1488673 - part4 : adjust cue's position. r=heycam
Alastor Wu <alwu@mozilla.com> - Tue, 19 Mar 2019 03:29:36 +0000 - rev 464984
Push 112486 by opoprus@mozilla.com at Tue, 19 Mar 2019 16:41:04 +0000
Bug 1488673 - part4 : adjust cue's position. r=heycam According to the spec 7.2.10, step1 to step10 (snap-to-line is true) and step1 to step2 (snap-to-line is flase) [1], we would adjust cue box's position depending on the algorithm. The algorithm would calculate the cue box's top (or left) position depending on the writing direction, and move the box to the specific position. However, if the cue box is overlapping with other cue boxes in the rendering area, we would still need to adjust its postion, which will be implemented in next patch. [1] https://w3c.github.io/webvtt/#ref-for-webvtt-cue-snap-to-lines-flag-12 https://w3c.github.io/webvtt/#ref-for-webvtt-cue-snap-to-lines-flag-13 Differential Revision: https://phabricator.services.mozilla.com/D19609
e89cf185fc33bf7deb0edea5d42ca918301aa797: Bug 1488673 - part3 : only use BoxPosition format for position calculation r=heycam
Alastor Wu <alwu@mozilla.com> - Tue, 19 Mar 2019 03:29:29 +0000 - rev 464983
Push 112486 by opoprus@mozilla.com at Tue, 19 Mar 2019 16:41:04 +0000
Bug 1488673 - part3 : only use BoxPosition format for position calculation r=heycam It's confused that we have both simpleBoxPosition object and BoxPosition object, we should only use one format to perform all box related operations. Therefore, BoxPosition should be able to be initiaized by StyleBox, HTMLElement or BoxPosition. In addition, as `right` and `bottom` can be calculated from other attributes, we remove these two attributes from BoxPosition, and use getter to get the correct value, which can reduce some unnessary modification when we changes the `height` or `width`. In order to implement a more readable getter, so we change `BoxPosition` to class-based. Differential Revision: https://phabricator.services.mozilla.com/D22809
367a2a68fb2dfeef6e945b01263335124f05ca12: Bug 1488673 - part2 : keep font size consistent even if applying CSS transform scale on the parent element r=heycam
Alastor Wu <alwu@mozilla.com> - Tue, 19 Mar 2019 03:29:27 +0000 - rev 464982
Push 112486 by opoprus@mozilla.com at Tue, 19 Mar 2019 16:41:04 +0000
Bug 1488673 - part2 : keep font size consistent even if applying CSS transform scale on the parent element r=heycam When calculating font size, we should use the original bounding box as a reference, or it would cause incorrect font size scale. Differential Revision: https://phabricator.services.mozilla.com/D20026
93cb5631f24b43006a5b464ad047f16caaf8d91e: Bug 1488673 - part1 : use correct size to calculate position. r=heycam
Alastor Wu <alwu@mozilla.com> - Tue, 19 Mar 2019 03:27:04 +0000 - rev 464981
Push 112486 by opoprus@mozilla.com at Tue, 19 Mar 2019 16:41:04 +0000
Bug 1488673 - part1 : use correct size to calculate position. r=heycam According to the spec [1], we have calculated the size in step 7.3 and then we should use that for calculating x-position and y-position, instead of using cue's size. [1] https://w3c.github.io/webvtt/#apply-webvtt-cue-settings Differential Revision: https://phabricator.services.mozilla.com/D22808
1ed43c15c0ea97e68338cc1800a2ba9739bc7c32: Bug 1454935 - Skip test_WorkerDebugger and test_WorkerDebuggerManager on QR platforms;r=asuth
Julian Descottes <jdescottes@mozilla.com> - Wed, 06 Mar 2019 14:08:29 +0000 - rev 464980
Push 112486 by opoprus@mozilla.com at Tue, 19 Mar 2019 16:41:04 +0000
Bug 1454935 - Skip test_WorkerDebugger and test_WorkerDebuggerManager on QR platforms;r=asuth Both tests started spiking after they were enabled on QR platforms. They are timing out after terminating a worker. The WorkerDebuggerManager never receives the expected unregister notification. Differential Revision: https://phabricator.services.mozilla.com/D21059
(0) -300000 -100000 -30000 -10000 -3000 -1000 -300 -100 -50 -20 +20 +50 +100 +300 +1000 +3000 +10000 +30000 tip