02d8210b89048b964a604d60b25b94d95b579cfb: Bug 1501353 - Change the exposed accessible role for a named region from region to landmark, r=Jamie
Marco Zehe <mzehe@mozilla.com> - Wed, 24 Oct 2018 00:34:21 +0000 - rev 491054
Push 247 by fmarier@mozilla.com at Sat, 27 Oct 2018 01:06:44 +0000
Bug 1501353 - Change the exposed accessible role for a named region from region to landmark, r=Jamie Differential Revision: https://phabricator.services.mozilla.com/D9554
c3e9b06cc27d0555af93b5c8feba00614f904052: Merge mozilla-central to autoland
arthur.iakab <aiakab@mozilla.com> - Wed, 24 Oct 2018 07:17:07 +0300 - rev 491053
Push 247 by fmarier@mozilla.com at Sat, 27 Oct 2018 01:06:44 +0000
Merge mozilla-central to autoland
39abbcc4230bd8f4e6a6f54c5b771eff57cbb0df: Bug 1496706 - Windows 8 doesn't return valid IInspectable when clicking toast notification. r=aklotz
Makoto Kato <m_kato@ga2.so-net.ne.jp> - Tue, 23 Oct 2018 20:59:02 +0000 - rev 491052
Push 247 by fmarier@mozilla.com at Sat, 27 Oct 2018 01:06:44 +0000
Bug 1496706 - Windows 8 doesn't return valid IInspectable when clicking toast notification. r=aklotz When clicking toast notification window, OnActivate is called. When using Windows 8, 2nd parameter (IInspectable) might be nullptr. So when this parameter is nullptr, it should recognize as alertclickcallback with mClickable=true. Differential Revision: https://phabricator.services.mozilla.com/D8076
8be2131ed18337d6d5011d51aa9d9f39aefff495: Bug 1501270 - Set the UrlbarInput value to the target URL when selecting a result. r=standard8
Dão Gottwald <dao@mozilla.com> - Wed, 24 Oct 2018 11:41:43 +0200 - rev 491051
Push 247 by fmarier@mozilla.com at Sat, 27 Oct 2018 01:06:44 +0000
Bug 1501270 - Set the UrlbarInput value to the target URL when selecting a result. r=standard8
460f9791ba8a7c40171fe8762bb827c59c83e9e3: Bug 1501250 - Intermittent [worker:error] distutils.errors.DistutilsFileError: cannot copy tree '/builds/worker/artifacts': not a directory, r=pmoore
Dragos Crisan <dcrisan@mozilla.com> - Wed, 24 Oct 2018 10:40:33 +0300 - rev 491050
Push 247 by fmarier@mozilla.com at Sat, 27 Oct 2018 01:06:44 +0000
Bug 1501250 - Intermittent [worker:error] distutils.errors.DistutilsFileError: cannot copy tree '/builds/worker/artifacts': not a directory, r=pmoore
915d77bbcf8b9eafb9a51af01e06c146175f1dfc: Bug 1501057 - Trivial clipPaths should respect visibility=hidden r=dholbert
longsonr <longsonr@gmail.com> - Wed, 24 Oct 2018 06:01:57 +0100 - rev 491049
Push 247 by fmarier@mozilla.com at Sat, 27 Oct 2018 01:06:44 +0000
Bug 1501057 - Trivial clipPaths should respect visibility=hidden r=dholbert
c1f0dc8791b7d774225f3fdb7502bcd7d07fb4be: Merge mozilla-central to inbound
arthur.iakab <aiakab@mozilla.com> - Wed, 24 Oct 2018 07:17:50 +0300 - rev 491048
Push 247 by fmarier@mozilla.com at Sat, 27 Oct 2018 01:06:44 +0000
Merge mozilla-central to inbound
079c7a062b23950449e9ce1ef809ff43026cc259: Merge inbound to mozilla-central a=merge
arthur.iakab <aiakab@mozilla.com> - Wed, 24 Oct 2018 07:15:44 +0300 - rev 491047
Push 247 by fmarier@mozilla.com at Sat, 27 Oct 2018 01:06:44 +0000
Merge inbound to mozilla-central a=merge
e61ca916da324a49717d173224c8f54e30097bbe: Bug 1498151 - part 2: Fix simple warnings r=aklotz,Jamie
Masayuki Nakano <masayuki@d-toybox.com> - Tue, 23 Oct 2018 17:35:35 +0000 - rev 491046
Push 247 by fmarier@mozilla.com at Sat, 27 Oct 2018 01:06:44 +0000
Bug 1498151 - part 2: Fix simple warnings r=aklotz,Jamie In AccessibleWrap.cpp, VARIANT instances are initialized with {VT_I4}. However, clang claims they should be initialized with {{{VT_I4}}}. In CompatibilityUIA, the capturing variable is reference to static variable. So, this patch makes it refer the static variable directly. Differential Revision: https://phabricator.services.mozilla.com/D8511
d0d1ccfded5ddefa99bb206ffac2235b457abffe: Bug 1498151 - part 1: Make IUnknownImpl.h define ATTRIBUTE_UNUSED as __attribute__((unused)) when built with clang r=Jamie
Masayuki Nakano <masayuki@d-toybox.com> - Tue, 23 Oct 2018 07:13:33 +0000 - rev 491045
Push 247 by fmarier@mozilla.com at Sat, 27 Oct 2018 01:06:44 +0000
Bug 1498151 - part 1: Make IUnknownImpl.h define ATTRIBUTE_UNUSED as __attribute__((unused)) when built with clang r=Jamie IMPL_IUNKNOWN_QUERY_HEAD defines HRESULT as hr. However, if its user uses IMPL_IUNKNOWN_QUERY_TAIL_AGGREGATED, it's never used. Therefore, it causes unused variable warning. hr is defined with ATTRIBUTE_UNUSED. However, this is empty when built with clang. IUnknownImpl.h already defines ATTRIBUTE_UNUSED as __attribute__((unused)) when built with gcc. So, same warnings should be prevented when built with clang too. Differential Revision: https://phabricator.services.mozilla.com/D8510
3b8a9abe2766c4bdc68c143416a3aafe85083be4: Backed out changeset bee1c344cb47 (bug 1491909) for failures in browser/components/extensions/test/xpcshell/test_ext_geckoProfiler_control.js
Noemi Erli <nerli@mozilla.com> - Wed, 24 Oct 2018 04:23:13 +0300 - rev 491044
Push 247 by fmarier@mozilla.com at Sat, 27 Oct 2018 01:06:44 +0000
Backed out changeset bee1c344cb47 (bug 1491909) for failures in browser/components/extensions/test/xpcshell/test_ext_geckoProfiler_control.js
8ff4dac9916d9bd908611963de3ec8883befea5f: Bug 1448880 - Part 7: Fix hasAnyLiveHooks to include onStep handlers on suspended frames. r=sfink
Jason Orendorff <jorendorff@mozilla.com> - Wed, 24 Oct 2018 01:08:19 +0000 - rev 491043
Push 247 by fmarier@mozilla.com at Sat, 27 Oct 2018 01:06:44 +0000
Bug 1448880 - Part 7: Fix hasAnyLiveHooks to include onStep handlers on suspended frames. r=sfink Depends on D6988 Differential Revision: https://phabricator.services.mozilla.com/D6990
6c8949f053e0e9c6216329d9b3f081fc0b6b2936: Bug 1448880 - Part 6: Re-enable stepping when an async or generator frame with an .onStep hook is resumed. r=jimb
Jason Orendorff <jorendorff@mozilla.com> - Tue, 23 Oct 2018 23:24:11 +0000 - rev 491042
Push 247 by fmarier@mozilla.com at Sat, 27 Oct 2018 01:06:44 +0000
Bug 1448880 - Part 6: Re-enable stepping when an async or generator frame with an .onStep hook is resumed. r=jimb That is, don't put it off until Debugger::getFrame() is called. The effect is subtle, as indicated by the test changes: the onEnterFrame hooks in those tests were causing getFrame to be called very early during generator resumption, which made the tests pass. With this patch, we no longer adjust the step mode count when suspending or resuming. This change is necessary to make the frame->isDebuggee() call in Debugger::onResumeFrame the right criterion for calling slowPathOnResumeFrame. It's true if the step mode count on the script is nonzero. (This approach also simplifies error handling, as resuming a Debugger.Frame is now idempotent: we don't have to worry about adjusting the step mode count too much or not enough on error.) Depends on D6986 Differential Revision: https://phabricator.services.mozilla.com/D6988
27b0121d9e314b26e2c11857b18409f5dabbd75b: Bug 1448880 - Part 5: Tell removeFromFrameMapsAndClearBreakpointsIn() if we are suspending. r=jimb
Jason Orendorff <jorendorff@mozilla.com> - Tue, 23 Oct 2018 23:36:35 +0000 - rev 491041
Push 247 by fmarier@mozilla.com at Sat, 27 Oct 2018 01:06:44 +0000
Bug 1448880 - Part 5: Tell removeFromFrameMapsAndClearBreakpointsIn() if we are suspending. r=jimb Since the argument is not used yet, this too is a pure refactoring, with no change in behavior yet. Depends on D6985 Differential Revision: https://phabricator.services.mozilla.com/D6986
61031045a58cad9c96f2051a858031bf37443769: Bug 1448880 - Part 4: Split Debugger::onResumeFrame from onEnterFrame. r=jimb
Jason Orendorff <jorendorff@mozilla.com> - Tue, 23 Oct 2018 23:23:15 +0000 - rev 491040
Push 247 by fmarier@mozilla.com at Sat, 27 Oct 2018 01:06:44 +0000
Bug 1448880 - Part 4: Split Debugger::onResumeFrame from onEnterFrame. r=jimb Pure refactoring, no change in behavior. This is in anticipation of doing additional work in onResumeFrame. Depends on D6984 Differential Revision: https://phabricator.services.mozilla.com/D6985
9c03b503909a6d03150869a782d5e148870c20f5: Bug 1448880 - Part 3: AbstractFramePtr::isGeneratorFrame(). r=jimb
Jason Orendorff <jorendorff@mozilla.com> - Tue, 23 Oct 2018 23:23:07 +0000 - rev 491039
Push 247 by fmarier@mozilla.com at Sat, 27 Oct 2018 01:06:44 +0000
Bug 1448880 - Part 3: AbstractFramePtr::isGeneratorFrame(). r=jimb This proves handy in several places, later in the stack. Depends on D6983 Differential Revision: https://phabricator.services.mozilla.com/D6984
64810935a7516678a1e655b8d1034ac8700b53d7: Bug 1448880 - Part 2: Persist Debugger.Frame objects for generators across yield/await. r=jimb.
Jason Orendorff <jorendorff@mozilla.com> - Tue, 23 Oct 2018 23:22:00 +0000 - rev 491038
Push 247 by fmarier@mozilla.com at Sat, 27 Oct 2018 01:06:44 +0000
Bug 1448880 - Part 2: Persist Debugger.Frame objects for generators across yield/await. r=jimb. This is the minimal patch, but it leaves two bugs: 1. When a generator or async function is resumed, stepping is reenabled in Debugger::getFrame, which isn't necessarily called. The onStep tests in this patch work because they all use an onEnterFrame hook, which causes getFrame to be called as soon as the generator is resumed. 2. .onStep and .onPop hooks on suspended Frames do not survive GC if there are no other references to the Frame or the Debugger object. The behavior is safe, but the hooks can just mysteriously stop firing when GC happens. The next three patches in this stack lay the groundwork for fixing these bugs, without changing behavior; part 6 fixes the first bug; and part 7 fixes the second. Depends on D6982 Differential Revision: https://phabricator.services.mozilla.com/D6983
2f3da01d8816a0117e39eca43cd4c52c0a381a4d: Bug 1448880 - Part 1: Always close a generator on early forced return. r=jimb
Jason Orendorff <jorendorff@mozilla.com> - Tue, 23 Oct 2018 23:21:01 +0000 - rev 491037
Push 247 by fmarier@mozilla.com at Sat, 27 Oct 2018 01:06:44 +0000
Bug 1448880 - Part 1: Always close a generator on early forced return. r=jimb The previous code failed to close the generator in the case where JSOP_GENERATOR had run but JSOP_INITIAL_YIELD had not, a bit of sloppiness that created yet another special case. Things will get more complicated when we start keeping frames live while suspended; it's better to not have this special case. Differential Revision: https://phabricator.services.mozilla.com/D6982
faaec607a74f01e81bc5a6b54d0c4141ad5cf8f5: Bug 1501073 - Test expectation adjustments. r=kats
Bobby Holley <bobbyholley@gmail.com> - Wed, 24 Oct 2018 00:49:16 +0000 - rev 491036
Push 247 by fmarier@mozilla.com at Sat, 27 Oct 2018 01:06:44 +0000
Bug 1501073 - Test expectation adjustments. r=kats Depends on D9610 Differential Revision: https://phabricator.services.mozilla.com/D9611
61b89032c551c59ab82f64689c1b271c5c0d2263: Bug 1501319 - Update webrender to commit e7d340b0f39bbd0046e983a75245bdde54013cdb. r=kats
WR Updater Bot <graphics-team@mozilla.staktrace.com> - Wed, 24 Oct 2018 00:48:58 +0000 - rev 491035
Push 247 by fmarier@mozilla.com at Sat, 27 Oct 2018 01:06:44 +0000
Bug 1501319 - Update webrender to commit e7d340b0f39bbd0046e983a75245bdde54013cdb. r=kats Differential Revision: https://phabricator.services.mozilla.com/D9610
(0) -300000 -100000 -30000 -10000 -3000 -1000 -300 -100 -50 -20 +20 +50 +100 +300 tip