2339a45e7261396ad5614a592c45dd65e6a3b127: Bug 1440703 - Add automated test to check that the Bookmarks Toolbar and Sidebar can be enabled from the Bookmarks Menu. r=Standard8
Paul Silaghi <psilaghi@mozilla.com> - Wed, 07 Mar 2018 10:47:43 +0200 - rev 406969
Push 100559 by ryanvm@gmail.com at Wed, 07 Mar 2018 20:25:43 +0000
Bug 1440703 - Add automated test to check that the Bookmarks Toolbar and Sidebar can be enabled from the Bookmarks Menu. r=Standard8
857cd5aebd4bb3b166bf37b56fb00d8f98dc6b64: Bug 1440567 - Add non-null checks around assignment of warning report to preflight HTTP channel in nsCORSListenerProxy::StartCORSPreflight. r=dveditz
Honza Bambas <honzab.moz@firemni.cz> - Wed, 07 Mar 2018 08:03:00 -0500 - rev 406968
Push 100558 by ryanvm@gmail.com at Wed, 07 Mar 2018 20:25:13 +0000
Bug 1440567 - Add non-null checks around assignment of warning report to preflight HTTP channel in nsCORSListenerProxy::StartCORSPreflight. r=dveditz
77acafe9c8136da959a24c54b68d3975a2f36c7a: Bug 1442502 - Require event loop roundtrip for WebGLSync. - r=kvark
Jeff Gilbert <jgilbert@mozilla.com> - Tue, 20 Feb 2018 17:34:25 -0800 - rev 406967
Push 100557 by jgilbert@mozilla.com at Wed, 07 Mar 2018 20:16:28 +0000
Bug 1442502 - Require event loop roundtrip for WebGLSync. - r=kvark MozReview-Commit-ID: 6h6j2LvJdXm
7532ccb5c0b39d6abbf7f67caee3f72c7f5addbe: Bug 1442931: Part 2 - Move internal WebIDL interfaces to separate directory. r=mystor
Kris Maglione <maglione.k@gmail.com> - Sun, 04 Mar 2018 16:37:10 -0800 - rev 406966
Push 100556 by maglione.k@gmail.com at Wed, 07 Mar 2018 19:52:28 +0000
Bug 1442931: Part 2 - Move internal WebIDL interfaces to separate directory. r=mystor This change moves several chrome-only WebIDL files to a separate chrome-only directory. There are several other chrome-only interfaces which are heavily tied to DOM code. Since those should probably still require DOM peer review for changes, I left them where they were. MozReview-Commit-ID: K3NsNtfntV6
403b15374e2f245728d7646f60aeb27b578d4a34: Bug 1442931: Part 1 - Forbid web-visible interfaces outside of WebIDL root. r=mystor
Kris Maglione <maglione.k@gmail.com> - Mon, 05 Mar 2018 14:21:38 -0800 - rev 406965
Push 100556 by maglione.k@gmail.com at Wed, 07 Mar 2018 19:52:28 +0000
Bug 1442931: Part 1 - Forbid web-visible interfaces outside of WebIDL root. r=mystor Web-visible WebIDL interfaces require DOM peer review with every change, which is enforced by a commit hook. ChromeOnly interfaces are not exposed to the web, and therefore don't require the same strictures. The current commit hook enforces the review requirement for changes to any (non-Servo) WebIDL file, and is not smart enough to determine if the change is web-visible. In order to loosen that restriction, we need the build system to enforce the requirement that only WebIDL files in certain locations may contain web-visible interfaces, so that the commit hook can restrict itself to checking only those directories. This change restricts the location of web-visible WebIDL interfaces to the dom/webidl/ and dom/bindings/ roots (along with the corresponding objdir root for generated interfaces). A follow-up will change the commit hook to only enforce review requirements on these directories. MozReview-Commit-ID: CiDxXxN4oO4
2c5754dcffa9465565e5ccfee3464a5c491e6986: Bug 1432036 - Fix eslint error in flexbox.js. r=me
Gabriel Luong <gabriel.luong@gmail.com> - Wed, 07 Mar 2018 14:50:33 -0500 - rev 406964
Push 100555 by gabriel.luong@gmail.com at Wed, 07 Mar 2018 19:50:58 +0000
Bug 1432036 - Fix eslint error in flexbox.js. r=me
7939ec78670514e4fb273cc77af89f28eadbcbfa: Bug 1432036 - Remove overlapping flex lines with the flex container in flexbox highlighter. r=gl
Shuoyi Ma <mashuoyi111@sina.com> - Wed, 07 Mar 2018 14:45:18 -0500 - rev 406963
Push 100554 by gabriel.luong@gmail.com at Wed, 07 Mar 2018 19:45:59 +0000
Bug 1432036 - Remove overlapping flex lines with the flex container in flexbox highlighter. r=gl
32c5d61d51ed33a32cd8e7407e60def0108fac8d: Bug 1442304 - make already_AddRefed returnable in registers in non-DEBUG Unix builds; r=glandium,tjr
Nathan Froyd <froydnj@mozilla.com> - Wed, 07 Mar 2018 14:27:28 -0500 - rev 406962
Push 100553 by nfroyd@mozilla.com at Wed, 07 Mar 2018 19:27:56 +0000
Bug 1442304 - make already_AddRefed returnable in registers in non-DEBUG Unix builds; r=glandium,tjr This change saves ~150k (!) of binary size on x86-64 Linux.
fcabcde9cdae95c70c448f56f6bc8d6aaf245da8: Bug 1443316 - part 3 - make cache entry request checking more readable; r=ckerschb
Nathan Froyd <froydnj@mozilla.com> - Wed, 07 Mar 2018 13:39:54 -0500 - rev 406961
Push 100552 by nfroyd@mozilla.com at Wed, 07 Mar 2018 18:40:27 +0000
Bug 1443316 - part 3 - make cache entry request checking more readable; r=ckerschb What we're really doing in CacheEntry::CheckRequest is checking: a) Whether the method is contained in our allowed methods; and b) Whether all of the headers are contained in our allowed headers. nsTArray lets us check directly for containing elements, so let's use that facility rather than rolling our own.
5890a42e14d3aede0d0776e0acde512556681066: Bug 1443316 - part 2 - don't repeatedly construct comparators when checking cache entries; r=ckerschb
Nathan Froyd <froydnj@mozilla.com> - Wed, 07 Mar 2018 13:39:54 -0500 - rev 406960
Push 100552 by nfroyd@mozilla.com at Wed, 07 Mar 2018 18:40:27 +0000
Bug 1443316 - part 2 - don't repeatedly construct comparators when checking cache entries; r=ckerschb nsCaseInsensitiveCStringComparator ought to be cheap to construct, but the object actually has a vtable to install and whatnot. So it's beneficial to pull the construction of it outside of the headers loop.
2df077fac15a16af2befa9a7dd74a9d068d38514: Bug 1443316 - part 1 - micro-optimize purging expired preflight cache entries; r=ckerschb
Nathan Froyd <froydnj@mozilla.com> - Wed, 07 Mar 2018 13:39:54 -0500 - rev 406959
Push 100552 by nfroyd@mozilla.com at Wed, 07 Mar 2018 18:40:27 +0000
Bug 1443316 - part 1 - micro-optimize purging expired preflight cache entries; r=ckerschb The entries in mMethods and mHeaders aren't sorted in any special way, so we can remove expired entries using UnorderedRemoveElementAt, which is faster than RemoveElementAt.
32da8b4abfd7adb3f60b2f138be37e4d43637283: Bug 1443812 - Update HarfBuzz to 1.7.6. r=jfkthame
Ryan VanderMeulen <ryanvm@gmail.com> - Wed, 07 Mar 2018 13:29:23 -0500 - rev 406958
Push 100551 by ryanvm@gmail.com at Wed, 07 Mar 2018 18:29:44 +0000
Bug 1443812 - Update HarfBuzz to 1.7.6. r=jfkthame
35129c889938eeefd8ccc81cf78a223e045e7f00: Bug 1424505: Ensure at least one consistent reference to the COM MTA at all times on Windows 7; r=jimm
Aaron Klotz <aklotz@mozilla.com> - Tue, 06 Mar 2018 17:06:54 -0700 - rev 406957
Push 100550 by aklotz@mozilla.com at Wed, 07 Mar 2018 18:16:01 +0000
Bug 1424505: Ensure at least one consistent reference to the COM MTA at all times on Windows 7; r=jimm
b87c9f3e9d5d314b553cd411a1554fa7cd11528f: Bug 1443613 - don't allocate temporary strings for DNSRequestChild; r=mayhemer
Nathan Froyd <froydnj@mozilla.com> - Wed, 07 Mar 2018 12:22:03 -0500 - rev 406956
Push 100549 by nfroyd@mozilla.com at Wed, 07 Mar 2018 17:22:42 +0000
Bug 1443613 - don't allocate temporary strings for DNSRequestChild; r=mayhemer DNSRequestChild's constructor takes `const nsCString&` parameters. If you have a `const nsACString&` argument to pass in to the constructor, you're forced to construct a temporary nsCString object to satisfy the prototype. But the temporary string will just get copied inside the constructor and you'll have to destroy the temporary string object you created. Let's skip all this, and just have DNSRequestChild take `const nsACString&` arguments instead, and avoid the temporary object.
1383d36fdabf8b134973bc15b69587d2e74e9d8a: Bug 1440719 - Set MOZ_CRASHREPORTER_SHUTDOWN during tests, other than browser-chrome; r=ted
Geoff Brown <gbrown@mozilla.com> - Wed, 07 Mar 2018 10:18:40 -0700 - rev 406955
Push 100548 by gbrown@mozilla.com at Wed, 07 Mar 2018 17:18:54 +0000
Bug 1440719 - Set MOZ_CRASHREPORTER_SHUTDOWN during tests, other than browser-chrome; r=ted
97d8b867071938df9e8e2126d986ea348f55c649: Bug 1442738. Fix test reference removed engine. r=bustag CLOSED TREE
Michael Kaply <mozilla@kaply.com> - Wed, 07 Mar 2018 11:03:45 -0600 - rev 406954
Push 100547 by mozilla@kaply.com at Wed, 07 Mar 2018 17:04:37 +0000
Bug 1442738. Fix test reference removed engine. r=bustag CLOSED TREE
5c78a8fff848584e3fa97628a6c2c11004ba69b7: Bug 1443587 - part 5 - more efficient string serialization; r=chutten
Nathan Froyd <froydnj@mozilla.com> - Wed, 07 Mar 2018 11:00:44 -0500 - rev 406953
Push 100546 by nfroyd@mozilla.com at Wed, 07 Mar 2018 16:01:50 +0000
Bug 1443587 - part 5 - more efficient string serialization; r=chutten The code for serializing an EventKey in SerializeEventsArray does something peculiar: it creates a temporary array of nsCStrings, copies the appropriate strings from the event info for the key, and then converts those into JavaScript strings. But we shouldn't need to do any copying into the temporary array. We can do everything directly on the strings from the event info themselves. It seemed best to introduce a ToJSString for this purpose, which comes in handy in a couple other places as well.
dbf8c7f084e131f9307998690add39bedea3a2bd: Bug 1443587 - part 4 - more efficient snapshot creation; r=chutten
Nathan Froyd <froydnj@mozilla.com> - Wed, 07 Mar 2018 11:00:44 -0500 - rev 406952
Push 100546 by nfroyd@mozilla.com at Wed, 07 Mar 2018 16:01:50 +0000
Bug 1443587 - part 4 - more efficient snapshot creation; r=chutten Once we've created the array of events for a given process, there's no need to copy it in the array for all process events. We can Move() it instead, which is more efficient.
c7c67940a56c050a8c0dd63fcee690e50e53a0b2: Bug 1443587 - part 3 - don't copy ChildEventData; r=chutten
Nathan Froyd <froydnj@mozilla.com> - Wed, 07 Mar 2018 11:00:44 -0500 - rev 406951
Push 100546 by nfroyd@mozilla.com at Wed, 07 Mar 2018 16:01:50 +0000
Bug 1443587 - part 3 - don't copy ChildEventData; r=chutten We were copying the event, rather than taking a reference to it. Let's do the latter instead of the former.
edf99f34ac2c1c079ab2b69b66377198f22055b1: Bug 1443587 - part 2 - improve construction of DynamicEventInfos; r=chutten
Nathan Froyd <froydnj@mozilla.com> - Wed, 07 Mar 2018 11:00:44 -0500 - rev 406950
Push 100546 by nfroyd@mozilla.com at Wed, 07 Mar 2018 16:01:50 +0000
Bug 1443587 - part 2 - improve construction of DynamicEventInfos; r=chutten When constructing DynamicEventInfo, we can require the extra keys to be passed in by rvalue reference, rather than copying the entire array. When registering dynamic events, we also don't need to construct an entirely new nsCString to pass to the constructor; we can simply pass the nsACString reference we received and go from there.
(0) -300000 -100000 -30000 -10000 -3000 -1000 -300 -100 -50 -20 +20 +50 +100 +300 +1000 +3000 +10000 +30000 tip