0f6244fb0d66654c0d5469c906e9c9343e18c6bf: Bug 1347040 - Prevent -Woverloaded-virtual of AccessibleWrap.h when __clang__ defined r=Jamie
Masayuki Nakano <masayuki@d-toybox.com> - Wed, 05 Sep 2018 04:13:30 +0000 - rev 434724
Push 34576 by ebalazs@mozilla.com at Wed, 05 Sep 2018 09:43:04 +0000
Bug 1347040 - Prevent -Woverloaded-virtual of AccessibleWrap.h when __clang__ defined r=Jamie Some classes of a11y are defined from both IUnknown and nsISupports. Both of them have QueryInterface(). So, one of them is overridden by a sub class hides the other and this is warned by clang. However, AccessibleWrap.h prevents this warning since it's by design when the compiler is GCC. So, it should do same thing when the compiler is clang. Differential Revision: https://phabricator.services.mozilla.com/D4310
434d6bf22d1dd69f1f70dca9bc59c357593d8c1a: Bug 1485177 - Add |siteOrigin| information to nsIPrincipal r=Ehsan
Nika Layzell <nika@thelayzells.com> - Wed, 05 Sep 2018 03:22:16 +0000 - rev 434723
Push 34576 by ebalazs@mozilla.com at Wed, 05 Sep 2018 09:43:04 +0000
Bug 1485177 - Add |siteOrigin| information to nsIPrincipal r=Ehsan Differential Revision: https://phabricator.services.mozilla.com/D4140
d7b738242b7ba029c546ee92d9c11d41f896fd31: Backed out changeset 56aa054d4cc0 (bug 1399367) for browser/extensions/formautofill/test/unit/test_activeStatus.js failures. CLOSED TREE
Csoregi Natalia <ncsoregi@mozilla.com> - Wed, 05 Sep 2018 05:47:21 +0300 - rev 434722
Push 34576 by ebalazs@mozilla.com at Wed, 05 Sep 2018 09:43:04 +0000
Backed out changeset 56aa054d4cc0 (bug 1399367) for browser/extensions/formautofill/test/unit/test_activeStatus.js failures. CLOSED TREE
d9298e2f81956956a1894c58c7a2cb8f0f864bf0: Bug 1478212 - Propagate MediaFeatureChangeReason::SystemMetricsChange into sub frames. r=emilio
Hiroyuki Ikezoe <hikezoe@mozilla.com> - Wed, 05 Sep 2018 01:50:33 +0000 - rev 434721
Push 34576 by ebalazs@mozilla.com at Wed, 05 Sep 2018 09:43:04 +0000
Bug 1478212 - Propagate MediaFeatureChangeReason::SystemMetricsChange into sub frames. r=emilio A test case covers this will be introduced in bug 1486971. Differential Revision: https://phabricator.services.mozilla.com/D4998
56aa054d4cc02a3c1c845d81ba8cc10f43260f05: Bug 1399367 - Remove MasterPassword.{encrypt|decrypt}Sync() methods r=MattN
Timothy Guan-tin Chien <timdream@gmail.com> - Wed, 05 Sep 2018 00:32:57 +0000 - rev 434720
Push 34576 by ebalazs@mozilla.com at Wed, 05 Sep 2018 09:43:04 +0000
Bug 1399367 - Remove MasterPassword.{encrypt|decrypt}Sync() methods r=MattN This also makes various AutofillRecords methods async, with the exception of remove() and removeAll(). Noted that I didn't implement any kind of "lock" for FormAutofillStorage -- please do not call these methods concurrently -- if you must please |await| for the last call to resolve. This most likely would happen in tests, and shouldn't happen in the real world, given that all user actions happen on macrotasks, and probably not at the next tick, unless Quicksilver is a Firefox user. FormAutofillStorage can be improved if there are complex use cases for it. Differential Revision: https://phabricator.services.mozilla.com/D4420
59b228f7faf14eee64ae6c446a1bb6ee0c520646: Bug 1488337 - Rename mozilla::dom::SelectionChangeListener to mozilla::SelectionChangeEventDispatcher r=smaug
Masayuki Nakano <masayuki@d-toybox.com> - Tue, 04 Sep 2018 11:18:03 +0000 - rev 434719
Push 34576 by ebalazs@mozilla.com at Wed, 05 Sep 2018 09:43:04 +0000
Bug 1488337 - Rename mozilla::dom::SelectionChangeListener to mozilla::SelectionChangeEventDispatcher r=smaug SelectionChangeListener is too generic name but it just dispatches selectionchange event when it's necessary. So, it should be renamed to SelectionChangeEventDispatcher. Additionally, it's in mozilla::dom namespace but it does not represent any DOM object. So, it should be in mozilla namespace instead. Differential Revision: https://phabricator.services.mozilla.com/D4913
91a9eae5ae1ea30e45b016d58a15c44b327a5eef: Bug 1488219 - Mark nsDateTimeControlFrame as a leaf frame. r=dholbert
Ting-Yu Lin <aethanyc@gmail.com> - Wed, 05 Sep 2018 00:27:43 +0000 - rev 434718
Push 34576 by ebalazs@mozilla.com at Wed, 05 Sep 2018 09:43:04 +0000
Bug 1488219 - Mark nsDateTimeControlFrame as a leaf frame. r=dholbert nsDateTimeControlFrame should be a leaf like all the other <input> frames like nsTextControlFrame, nsCheckboxRadioFrame, etc. Differential Revision: https://phabricator.services.mozilla.com/D4985
8f50aac218b58dfb67271e1dbed724baa8e1d713: Merge mozilla-central to autoland. a=merge CLOSED TREE
Bogdan Tara <btara@mozilla.com> - Wed, 05 Sep 2018 03:15:48 +0300 - rev 434717
Push 34576 by ebalazs@mozilla.com at Wed, 05 Sep 2018 09:43:04 +0000
Merge mozilla-central to autoland. a=merge CLOSED TREE
da67b43defd6c695aea49ba90978a931c9d58304: Merge mozilla-central to autoland. a=merge CLOSED TREE
Bogdan Tara <btara@mozilla.com> - Wed, 05 Sep 2018 02:43:23 +0300 - rev 434716
Push 34576 by ebalazs@mozilla.com at Wed, 05 Sep 2018 09:43:04 +0000
Merge mozilla-central to autoland. a=merge CLOSED TREE
8a4e8a30cd2ff2ff20c40cb7c051ca424f96e8e6: Backed out changeset 4ad33c6fbfca (bug 832834) for merge conflict. CLOSED TREE
Bogdan Tara <btara@mozilla.com> - Wed, 05 Sep 2018 02:41:48 +0300 - rev 434715
Push 34576 by ebalazs@mozilla.com at Wed, 05 Sep 2018 09:43:04 +0000
Backed out changeset 4ad33c6fbfca (bug 832834) for merge conflict. CLOSED TREE
4ad33c6fbfcad5dd5a72a1cc6a916eb49a93c6e4: bug 832834 - reimplement nsSecureBrowserUIImpl r=franziskus,Felipe
David Keeler <dkeeler@mozilla.com> - Tue, 04 Sep 2018 23:10:37 +0000 - rev 434714
Push 34576 by ebalazs@mozilla.com at Wed, 05 Sep 2018 09:43:04 +0000
bug 832834 - reimplement nsSecureBrowserUIImpl r=franziskus,Felipe It turns out nsSecureBrowserUIImpl is considerably more complicated than it needs to be. This patch reimplements it in terms of OnLocationChange only, which is all it needs to produce the same behavior as before. Differential Revision: https://phabricator.services.mozilla.com/D3548
098f6aaf52410eb6b51b230cfd2e478339d290c3: Bug 1485157 - Check to make sure document.activeElement exists before attempting to blur it;r=Gijs
Brian Grinstead <bgrinstead@mozilla.com> - Tue, 04 Sep 2018 20:46:08 +0000 - rev 434713
Push 34576 by ebalazs@mozilla.com at Wed, 05 Sep 2018 09:43:04 +0000
Bug 1485157 - Check to make sure document.activeElement exists before attempting to blur it;r=Gijs We're ending up in a case here where document.activeElement is null in browser.xhtml but it's a <browser> tag in browser.xul. We'll need more analysis and testing to decide if we want the HTML or XUL activeElement behavior, and then adjust as needed. But in the meantime, this unbreaks a bunch of browser.xhtml tests and is a safe null check in both cases. Differential Revision: https://phabricator.services.mozilla.com/D4705
0db77a36565cd9971eb5794785defe498673c303: Bug 1487931 - Avoid using linker scripts as file lists when doing LTO with BFD ld. r=froydnj
Mike Hommey <mh+mozilla@glandium.org> - Tue, 04 Sep 2018 21:49:16 +0000 - rev 434712
Push 34576 by ebalazs@mozilla.com at Wed, 05 Sep 2018 09:43:04 +0000
Bug 1487931 - Avoid using linker scripts as file lists when doing LTO with BFD ld. r=froydnj At least, when that combination doesn't work. This requires passing the LTO flags to the compiler invocations for the test, which in turn requires actually calling the linker directly on Windows, instead of directly through the compiler, as when actually building. Differential Revision: https://phabricator.services.mozilla.com/D4739
e119dbd3463f2cbe37ef4df3da54e47b5ae5a252: Bug 1488000 - Allow theming sidebar border color. r=jaws
Tim Nguyen <ntim.bugs@gmail.com> - Tue, 04 Sep 2018 21:33:46 +0000 - rev 434711
Push 34576 by ebalazs@mozilla.com at Wed, 05 Sep 2018 09:43:04 +0000
Bug 1488000 - Allow theming sidebar border color. r=jaws Differential Revision: https://phabricator.services.mozilla.com/D4839
146b7cab176227d21eeb5680c68100862171a322: Bug 1486410 - Bug 1466427 - Enable new clang-tidy 7.0 checks. r=andi
Jan Keromnes <janx@linux.com> - Tue, 04 Sep 2018 11:55:19 +0000 - rev 434710
Push 34576 by ebalazs@mozilla.com at Wed, 05 Sep 2018 09:43:04 +0000
Bug 1486410 - Bug 1466427 - Enable new clang-tidy 7.0 checks. r=andi Differential Revision: https://phabricator.services.mozilla.com/D4210
f7bddc0708f1b85bf7f4b0b5f337b150e5fc5056: Bug 1488026 - Always fire onDetached event before onAttached. r=mixedpuppy
Oriol Brufau <oriol-bugzilla@hotmail.com> - Tue, 04 Sep 2018 20:50:12 +0000 - rev 434709
Push 34576 by ebalazs@mozilla.com at Wed, 05 Sep 2018 09:43:04 +0000
Bug 1488026 - Always fire onDetached event before onAttached. r=mixedpuppy Differential Revision: https://phabricator.services.mozilla.com/D4826
417abd88f3076a9ff086e7c6026991ddae7ef4c6: Bug 1476996 - Implement cross process redirection in Http on the parent process r=bagder,nika
Valentin Gosu <valentin.gosu@gmail.com> - Tue, 04 Sep 2018 20:45:22 +0000 - rev 434708
Push 34576 by ebalazs@mozilla.com at Wed, 05 Sep 2018 09:43:04 +0000
Bug 1476996 - Implement cross process redirection in Http on the parent process r=bagder,nika This patch builds the foundation for the ability to relocate HTTP channels from one content process to another in order to ensure that origins are properly isolated. This relocation would normally occur when the response to an HTTP request is a redirect to a different origin. The patch merely adds the mechanism for relocating the channel, rather than the logic of doing so. This will be provided in a follow-up patch by a specialized service. Right now that functionality is mocked in the test. How this works: In nsHttpChannel::OnStartRequest we will query the service that decides whether we need to direct the response to another process. If so, it will return a promise that resolves to a TabParent. When the promise resolves, in HttpChannelParentListener::TriggerCrossProcessRedirect we call NeckoParent::SendCrossProcessRedirect passing along the required information to recreate the channel in the new process. The NeckoChild in the new process will then instantiate a new channel, call ConnectParent() which creates the associated parent channel, and connects it with the existing nsHttpChannel. A listener in the new process is then notified of the existence of the new channel. It is required to call completeRedirectSetup on the channel, passing an nsIStreamListener to the call. We then finish the entire operation with a call to HttpChannelChild::SendCrossProcessRedirectDone which causes us to close the old HttpChannelChild in the previous process and to resume the nsHttpChannel in the main process. Differential Revision: https://phabricator.services.mozilla.com/D2958
8e94abbf3ef96c542a1319f74f69db3c40f0634a: Bug 1484966 - Part 3: Clang format layout/painting/ r=mattwoodrow
Miko Mynttinen <mikokm@gmail.com> - Tue, 04 Sep 2018 20:46:21 +0000 - rev 434707
Push 34576 by ebalazs@mozilla.com at Wed, 05 Sep 2018 09:43:04 +0000
Bug 1484966 - Part 3: Clang format layout/painting/ r=mattwoodrow Depends on D3870 Differential Revision: https://phabricator.services.mozilla.com/D3871
400dede23df2515991e97c31ae7e313dc6f446d5: Bug 1484966 - Part 2: Fix virtual/override specifiers and whitespace r=mattwoodrow
Miko Mynttinen <mikokm@gmail.com> - Tue, 04 Sep 2018 20:45:55 +0000 - rev 434706
Push 34576 by ebalazs@mozilla.com at Wed, 05 Sep 2018 09:43:04 +0000
Bug 1484966 - Part 2: Fix virtual/override specifiers and whitespace r=mattwoodrow Depends on D3869 Differential Revision: https://phabricator.services.mozilla.com/D3870
895320802eb428bd7f7c54a6c55a2d8b97734cc8: Backed out 2 changesets (bug 1484966) for bustage on RefPtr.h:42. CLOSED TREE
Csoregi Natalia <ncsoregi@mozilla.com> - Tue, 04 Sep 2018 23:42:36 +0300 - rev 434705
Push 34576 by ebalazs@mozilla.com at Wed, 05 Sep 2018 09:43:04 +0000
Backed out 2 changesets (bug 1484966) for bustage on RefPtr.h:42. CLOSED TREE Backed out changeset 7b246c8f9de8 (bug 1484966) Backed out changeset 399158a0a9b3 (bug 1484966)
(0) -300000 -100000 -30000 -10000 -3000 -1000 -300 -100 -50 -20 +20 +50 +100 +300 +1000 +3000 +10000 +30000 +100000 tip