09000b1f817939cf0649348e65b8dd3cdfa629e6: Bug 1436187 - Fix linting errors in shared-head.js. r=bgrins draft
J. Ryan Stinnett <jryans@gmail.com> - Tue, 06 Mar 2018 15:28:55 -0600 - rev 764454
Push 101764 by bmo:jryans@gmail.com at Wed, 07 Mar 2018 20:33:00 +0000
Bug 1436187 - Fix linting errors in shared-head.js. r=bgrins After moving shared-head.js, linting is activated because the new path is not ignored. MozReview-Commit-ID: GCLDfgJ7WCb
4db74ebd9f76100a0887839bb771effa8ab6b097: Bug 1436187 - Move frame-script-utils.js to shared/test. r=bgrins draft
J. Ryan Stinnett <jryans@gmail.com> - Tue, 06 Mar 2018 14:14:55 -0600 - rev 764453
Push 101764 by bmo:jryans@gmail.com at Wed, 07 Mar 2018 20:33:00 +0000
Bug 1436187 - Move frame-script-utils.js to shared/test. r=bgrins Move frame-script-utils.js into shared/test since it is a test only file. In addition, it's removed from jar.mn so it won't be part the file we ship. Test manifests can use the absolute path syntax to make it accessible in new directories. MozReview-Commit-ID: 5sKYvv2rsJs
fd7f1a9541c2fc4d5d802e328ac868b0b98c5a9c: Bug 1436187 - Centralize frame-script-utils loading via shared-head. r=bgrins draft
J. Ryan Stinnett <jryans@gmail.com> - Mon, 05 Mar 2018 16:31:49 -0600 - rev 764452
Push 101764 by bmo:jryans@gmail.com at Wed, 07 Mar 2018 20:33:00 +0000
Bug 1436187 - Centralize frame-script-utils loading via shared-head. r=bgrins Use `loadFrameScriptUtils` from shared-head as a central utility for loading the frame script utils helper. This means less stray references to the utils file's path across our tests. As part of this, I went ahead and converted Canvas Debugger, Shader Editor, and Web Audio Editor to shared-head, as that seemed like the best path to reduce duplication. (I left one extra path reference in profiler-mm-utils.js as-is, since it's a module, so shared-head is not easily usable there.) MozReview-Commit-ID: AKbZt8Jo0GM
b830a608e5404889434f5c7fca872431517ee67b: Bug 1385403: Part 1 - Enable OOP extensions on OS-X for non-release users. r?aswan draft
Kris Maglione <maglione.k@gmail.com> - Wed, 07 Mar 2018 12:22:51 -0800 - rev 764451
Push 101763 by maglione.k@gmail.com at Wed, 07 Mar 2018 20:23:29 +0000
Bug 1385403: Part 1 - Enable OOP extensions on OS-X for non-release users. r?aswan MozReview-Commit-ID: KHLZxzbDFWM
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 764450
Push 101763 by maglione.k@gmail.com at Wed, 07 Mar 2018 20:23:29 +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 764449
Push 101763 by maglione.k@gmail.com at Wed, 07 Mar 2018 20:23:29 +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 764448
Push 101763 by maglione.k@gmail.com at Wed, 07 Mar 2018 20:23:29 +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 764447
Push 101763 by maglione.k@gmail.com at Wed, 07 Mar 2018 20:23:29 +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 764446
Push 101763 by maglione.k@gmail.com at Wed, 07 Mar 2018 20:23:29 +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 764445
Push 101763 by maglione.k@gmail.com at Wed, 07 Mar 2018 20:23:29 +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 764444
Push 101763 by maglione.k@gmail.com at Wed, 07 Mar 2018 20:23:29 +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 764443
Push 101763 by maglione.k@gmail.com at Wed, 07 Mar 2018 20:23:29 +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 764442
Push 101763 by maglione.k@gmail.com at Wed, 07 Mar 2018 20:23:29 +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 764441
Push 101763 by maglione.k@gmail.com at Wed, 07 Mar 2018 20:23:29 +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 764440
Push 101763 by maglione.k@gmail.com at Wed, 07 Mar 2018 20:23:29 +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 764439
Push 101763 by maglione.k@gmail.com at Wed, 07 Mar 2018 20:23:29 +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 764438
Push 101763 by maglione.k@gmail.com at Wed, 07 Mar 2018 20:23:29 +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 764437
Push 101763 by maglione.k@gmail.com at Wed, 07 Mar 2018 20:23:29 +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 764436
Push 101763 by maglione.k@gmail.com at Wed, 07 Mar 2018 20:23:29 +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 764435
Push 101763 by maglione.k@gmail.com at Wed, 07 Mar 2018 20:23:29 +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.
(0) -300000 -100000 -30000 -10000 -3000 -1000 -300 -100 -50 -20 +20 +50 +100 +300 +1000 +3000 +10000 +30000 tip