9cd5f82e915c28e57a8c35a3fe1dd9d3a75c7f71: Bug 1554777 - Re-enable test. r=dholbert
Emilio Cobos Álvarez <emilio@crisal.io> - Wed, 07 Aug 2019 23:14:03 +0000 - rev 488475
Push 92770 by ealvarez@mozilla.com at Fri, 16 Aug 2019 09:50:03 +0000
Bug 1554777 - Re-enable test. r=dholbert I think they should pass after the last few patches. Differential Revision: https://phabricator.services.mozilla.com/D41007
87ed773c0a80425133bae39146f06de4a7425974: Bug 1554777 - Use SIZE_AVAILABLE rather than loaded to figure out whether to try to draw a border image. r=tnikkel
Emilio Cobos Álvarez <emilio@crisal.io> - Thu, 15 Aug 2019 19:03:06 +0000 - rev 488474
Push 92769 by ealvarez@mozilla.com at Fri, 16 Aug 2019 09:49:34 +0000
Bug 1554777 - Use SIZE_AVAILABLE rather than loaded to figure out whether to try to draw a border image. r=tnikkel The current code needs to handle incomplete draws already due to async decoding, and this prevents the race condition where we paint between the size-available and load notifications, as the CSS image loader only invalidates for the first. Once we try to decode the image at least once, then we'd repaint properly from ImageLoader::OnFrameComplete. Differential Revision: https://phabricator.services.mozilla.com/D41006
da9ae31b5000eccbc38438680952ff08d06addea: Bug 1554777 - Call SchedulePaint() rather than MarkNeedsDisplayItemRebuild() when we get the size available notification for a style image. r=mattwoodrow,tnikkel
Emilio Cobos Álvarez <emilio@crisal.io> - Thu, 15 Aug 2019 19:03:18 +0000 - rev 488473
Push 92769 by ealvarez@mozilla.com at Fri, 16 Aug 2019 09:49:34 +0000
Bug 1554777 - Call SchedulePaint() rather than MarkNeedsDisplayItemRebuild() when we get the size available notification for a style image. r=mattwoodrow,tnikkel So the issue here is that this test-case has a zero-sized border, but still with a border-image (which we should paint). So the first time we get here, the image is not loaded, and thus we don't get here: https://searchfox.org/mozilla-central/rev/e0b0c38ee83f99d3cf868bad525ace4a395039f1/layout/painting/nsDisplayList.h#4254 Which means that we end up with zero bounds and thus we don't even get to the border painting code. However, when the image loads, we get to MarkNeedsDisplayItemRebuild(), but that doesn't schedule any paint, only marks the frames as modified in order for display items to be rebuilt _eventually_. Thus eventually, where we force a repaint by other means, we paint correctly. This only works in more general cases because we get to the nsImageRenderer code which does vastly different stuff. InvalidateFrame() seems to do the right thing and schedule a paint, so use it. It's not clear to me which one of nsIFrame::InvalidateFrame() or nsIFrame::SchedulePaint() we should use... If I understand correctly, InvalidateFrame() will only do something iff there are display items for the frame, so that should make the IsVisible() check redundant. Note however that I think there's still a race condition, if we get to paint in between the SIZE_AVAILABLE notification (the one where we actually invalidate the display items), and the LOAD notification (the one the border-image code checks). I'll send a separate patch for that, I think SIZE_AVAILABLE should be a strong-enough hint and that allows us to remove nsStyleImage::IsLoaded()... The RequestReflow stuff also looks highly suspicious... shape-outside sync-decodes, and it seems we could end up invalidating reflow from the reflow code... Differential Revision: https://phabricator.services.mozilla.com/D41005
870a350fb1360725bb7206cf8c392ef3a8d400e5: Bug 1571898 - Fix the color for the hover state of the buttons in the protections panel. r=nhnt11
Tim Huang <tihuang@mozilla.com> - Fri, 16 Aug 2019 09:43:04 +0000 - rev 488472
Push 92768 by tihuang@mozilla.com at Fri, 16 Aug 2019 09:44:18 +0000
Bug 1571898 - Fix the color for the hover state of the buttons in the protections panel. r=nhnt11 This patch adds some missing rules for showing the proper hover state in Linux. Depends on D42282 Differential Revision: https://phabricator.services.mozilla.com/D42283
21bd83f9fa50dd22622b5fa1447fb52b4e281eaf: Bug 1567417 - Make the buttons in the footer section of the protections panel to ignore the appearance setting of the toolbarbutton. r=nhnt11
Tim Huang <tihuang@mozilla.com> - Fri, 16 Aug 2019 09:32:03 +0000 - rev 488471
Push 92768 by tihuang@mozilla.com at Fri, 16 Aug 2019 09:44:18 +0000
Bug 1567417 - Make the buttons in the footer section of the protections panel to ignore the appearance setting of the toolbarbutton. r=nhnt11 We need to overwrite the '-moz-appearance' CSS rule of the buttons in the footer section of the protections panel in order to properly style the buttons. Differential Revision: https://phabricator.services.mozilla.com/D42282
09ce6041df3bc1451e914a84f9b8e742c2f294de: Bug 1574179 - Simplify <img align> rules in quirks.css. r=jfkthame
Emilio Cobos Álvarez <emilio@crisal.io> - Thu, 15 Aug 2019 19:26:02 +0000 - rev 488470
Push 92767 by ealvarez@mozilla.com at Fri, 16 Aug 2019 09:38:03 +0000
Bug 1574179 - Simplify <img align> rules in quirks.css. r=jfkthame It seems to me that it should be equivalent, and we don't need to do directionality-dependent stuff. align=right and align=left map to the physical values anyway: https://searchfox.org/mozilla-central/rev/3366c3d24f1c3818df37ec0818833bf085e41a53/dom/html/nsGenericHTMLElement.cpp#1165 This also matches Chromium: https://cs.chromium.org/chromium/src/third_party/blink/renderer/core/html/resources/quirks.css?l=26&rcl=e019448580f331c4c7c756271843d1bcf27e0071 (Modulo the case-insensitivity) Differential Revision: https://phabricator.services.mozilla.com/D42142
f96d420517972051aeb08355243e6d7b081c4b94: Bug 1572689 - Properly restore size of the Search panel; r=nchevobbe
Jan Odvarko <odvarko@gmail.com> - Fri, 16 Aug 2019 06:18:16 +0000 - rev 488469
Push 92766 by jodvarko@mozilla.com at Fri, 16 Aug 2019 09:15:16 +0000
Bug 1572689 - Properly restore size of the Search panel; r=nchevobbe Differential Revision: https://phabricator.services.mozilla.com/D41936
c670f40348ee7794f031ef7f84019592d56154cd: Bug 1570556 - Remove call to pango_fc_font_map_shutdown, which was moved out of public headers in Pango 1.44. r=dbaron
Jonathan Kew <jkew@mozilla.com> - Thu, 15 Aug 2019 20:10:41 +0000 - rev 488468
Push 92765 by jkew@mozilla.com at Fri, 16 Aug 2019 09:09:36 +0000
Bug 1570556 - Remove call to pango_fc_font_map_shutdown, which was moved out of public headers in Pango 1.44. r=dbaron Differential Revision: https://phabricator.services.mozilla.com/D42127
47f90b1619aebaa588c039fcde7b6ca46debeb25: Bug 1568514 - Add telemetry histograms to report hyphenation memory footprint and resource load times. r=heycam
Jonathan Kew <jkew@mozilla.com> - Fri, 16 Aug 2019 07:24:46 +0000 - rev 488467
Push 92764 by jkew@mozilla.com at Fri, 16 Aug 2019 09:06:39 +0000
Bug 1568514 - Add telemetry histograms to report hyphenation memory footprint and resource load times. r=heycam Differential Revision: https://phabricator.services.mozilla.com/D39323
9f97c09349098de632bc6ac28c1446c2c06c7077: Bug 1572692 - Properly crop results in the Search panel; r=nchevobbe
Jan Odvarko <odvarko@gmail.com> - Fri, 16 Aug 2019 09:04:11 +0000 - rev 488466
Push 92763 by jodvarko@mozilla.com at Fri, 16 Aug 2019 09:04:44 +0000
Bug 1572692 - Properly crop results in the Search panel; r=nchevobbe Differential Revision: https://phabricator.services.mozilla.com/D41940
83efa3e53a72a21fe1048f20a3a5054681fc2f69: Bug 1571710 - Adds test to download link. r=johannh
dleblanccyr <danielleleb12@gmail.com> - Thu, 15 Aug 2019 15:20:51 +0000 - rev 488465
Push 92762 by archaeopteryx@coole-files.de at Fri, 16 Aug 2019 08:30:33 +0000
Bug 1571710 - Adds test to download link. r=johannh Differential Revision: https://phabricator.services.mozilla.com/D41979
36d75347441bbeaf2f9f3e321f6318d7643ccec2: Bug 1571710 - Adds link to download certificate. r=johannh,fluent-reviewers,flod
dleblanccyr <danielleleb12@gmail.com> - Thu, 15 Aug 2019 15:20:44 +0000 - rev 488464
Push 92762 by archaeopteryx@coole-files.de at Fri, 16 Aug 2019 08:30:33 +0000
Bug 1571710 - Adds link to download certificate. r=johannh,fluent-reviewers,flod Differential Revision: https://phabricator.services.mozilla.com/D41078
983ff93a11ad7c08fd65a507dedf01526e4ae682: Bug 1505412 - Skip CSP-Nonce check for CSP-Ro+Redirected Channels r=ckerschb
Sebastian Streich <sstreich@mozilla.com> - Thu, 15 Aug 2019 15:51:33 +0000 - rev 488463
Push 92761 by archaeopteryx@coole-files.de at Fri, 16 Aug 2019 08:13:25 +0000
Bug 1505412 - Skip CSP-Nonce check for CSP-Ro+Redirected Channels r=ckerschb Differential Revision: https://phabricator.services.mozilla.com/D36916
a550b308d88ae8e6bbdd1bab0123028ca98512d8: Bug 1573833 - Add missing & and parentheses r=pehrsons
Sylvestre Ledru <sledru@mozilla.com> - Fri, 16 Aug 2019 08:11:39 +0000 - rev 488462
Push 92760 by sledru@mozilla.com at Fri, 16 Aug 2019 08:12:54 +0000
Bug 1573833 - Add missing & and parentheses r=pehrsons Differential Revision: https://phabricator.services.mozilla.com/D41957
0a7463d82e5f5f3cbca589baab9abc07881f5142: Bug 1566810 - Part 2: Add the permission granted indicator to the identity panel. r=johannh
Tim Huang <tihuang@mozilla.com> - Fri, 16 Aug 2019 07:42:39 +0000 - rev 488461
Push 92759 by tihuang@mozilla.com at Fri, 16 Aug 2019 07:46:54 +0000
Bug 1566810 - Part 2: Add the permission granted indicator to the identity panel. r=johannh This patch adds the permission granted indicator to the identity panel. The granted indicator will be shown if there is any granted permission. And it will got update if permission changes. We also update a test for this change. Differential Revision: https://phabricator.services.mozilla.com/D42121
93ae831e2fb9a6ceac0f0495a280d6538fc62f21: Bug 1566810 - Part 1: Add the granted permission indicator on the lock icon. r=johannh
Tim Huang <tihuang@mozilla.com> - Thu, 15 Aug 2019 14:21:27 +0000 - rev 488460
Push 92759 by tihuang@mozilla.com at Fri, 16 Aug 2019 07:46:54 +0000
Bug 1566810 - Part 1: Add the granted permission indicator on the lock icon. r=johannh This patch adds a permission indictor on the lock icon and it is only shown if there is any permission has been granted. Differential Revision: https://phabricator.services.mozilla.com/D42119
22637ac7ff7f665ae02be0619c1a51fd47ef2826: Bug 1574090 - Make fission.autostart available via StaticPrefs. r=njn
Henri Sivonen <hsivonen@hsivonen.fi> - Fri, 16 Aug 2019 07:35:29 +0000 - rev 488459
Push 92758 by hsivonen@mozilla.com at Fri, 16 Aug 2019 07:36:08 +0000
Bug 1574090 - Make fission.autostart available via StaticPrefs. r=njn Differential Revision: https://phabricator.services.mozilla.com/D42091
ab84d8979801b3ba477f9cc530c63917fafffeaa: Bug 1572633 - Document some additional details about pref files. r=KrisWright
Nicholas Nethercote <nnethercote@mozilla.com> - Fri, 16 Aug 2019 07:27:22 +0000 - rev 488458
Push 92757 by nnethercote@mozilla.com at Fri, 16 Aug 2019 07:29:25 +0000
Bug 1572633 - Document some additional details about pref files. r=KrisWright Differential Revision: https://phabricator.services.mozilla.com/D41331
f9534a732e48d4b292da4dee06dbeff6586c698e: Bug 1572633 - Remove apz.overscroll.spring_{stiffness,friction}. r=botond
Nicholas Nethercote <nnethercote@mozilla.com> - Fri, 16 Aug 2019 07:28:08 +0000 - rev 488457
Push 92757 by nnethercote@mozilla.com at Fri, 16 Aug 2019 07:29:25 +0000
Bug 1572633 - Remove apz.overscroll.spring_{stiffness,friction}. r=botond They are unused. Differential Revision: https://phabricator.services.mozilla.com/D41330
c9ece2f3ad559ed5a861b8b7c829bb6277c12fe7: Bug 1574107 - Remove nsIFrame::IsBlockInside since it's dead code. r=emilio
Mats Palmgren <mats@mozilla.com> - Fri, 16 Aug 2019 06:29:59 +0000 - rev 488456
Push 92756 by mpalmgren@mozilla.com at Fri, 16 Aug 2019 06:46:15 +0000
Bug 1574107 - Remove nsIFrame::IsBlockInside since it's dead code. r=emilio Differential Revision: https://phabricator.services.mozilla.com/D42088
(0) -300000 -100000 -30000 -10000 -3000 -1000 -300 -100 -50 -20 +20 +50 +100 +300 +1000 +3000 tip