945ae1cf83ec8dd7c811044f8cabf52c0252bfd8: Backed out changeset 4ad8241eb92a (bug 1511398) gv-junit2 crashes on [@ mozilla::a11y::Accessible::IsDefunct() const] . CLOSED TREE
Narcis Beleuzu <nbeleuzu@mozilla.com> - Wed, 05 Dec 2018 22:06:21 +0200 - rev 449459
Push 35164 by shindli@mozilla.com at Thu, 06 Dec 2018 04:02:49 +0000
Backed out changeset 4ad8241eb92a (bug 1511398) gv-junit2 crashes on [@ mozilla::a11y::Accessible::IsDefunct() const] . CLOSED TREE
d7d78e79f0b34d84ba17ebd5311989cee49b5b6c: Bug 1504756 - [marionette] Added opening a new browsing context to Marionette client. r=ato
Henrik Skupin <mail@hskupin.info> - Wed, 05 Dec 2018 19:58:32 +0000 - rev 449458
Push 35164 by shindli@mozilla.com at Thu, 06 Dec 2018 04:02:49 +0000
Bug 1504756 - [marionette] Added opening a new browsing context to Marionette client. r=ato The patch updates the Marionette client and all Marionette unit tests to make use of the new `Create Window` command as much as possible. Depends on D13663 Differential Revision: https://phabricator.services.mozilla.com/D13664
5c495fd7f64d5b344954bbf66f930a9c25bcbbf8: Bug 1504756 - [marionette] Added "WebDriver:NewWindow" command to open a new top-level browsing context. r=ato
Henrik Skupin <mail@hskupin.info> - Wed, 05 Dec 2018 19:58:06 +0000 - rev 449457
Push 35164 by shindli@mozilla.com at Thu, 06 Dec 2018 04:02:49 +0000
Bug 1504756 - [marionette] Added "WebDriver:NewWindow" command to open a new top-level browsing context. r=ato The patch adds the end-point for the recently defined `Create window` command (https://github.com/w3c/webdriver/issues/1138). It allows to open a new top-level browsing context as tab or as window. Depends on D13662 Differential Revision: https://phabricator.services.mozilla.com/D13663
5c2826c58f9ef0bd89b5a5c94100a1c2d5f7e34e: Bug 1504756 - [marionette] Use waitForObserverTopic when waiting for observer notifications. r=ato
Henrik Skupin <mail@hskupin.info> - Wed, 05 Dec 2018 19:57:48 +0000 - rev 449456
Push 35164 by shindli@mozilla.com at Thu, 06 Dec 2018 04:02:49 +0000
Bug 1504756 - [marionette] Use waitForObserverTopic when waiting for observer notifications. r=ato Depends on D13661 Differential Revision: https://phabricator.services.mozilla.com/D13662
f23b667d8bfa4780add93901e66a71603eb2baf9: Bug 1504756 - [marionette] Use waitForMessage() to wait for an expected message manager message. r=ato
Henrik Skupin <mail@hskupin.info> - Wed, 05 Dec 2018 19:57:23 +0000 - rev 449455
Push 35164 by shindli@mozilla.com at Thu, 06 Dec 2018 04:02:49 +0000
Bug 1504756 - [marionette] Use waitForMessage() to wait for an expected message manager message. r=ato Depends on D13660 Differential Revision: https://phabricator.services.mozilla.com/D13661
6068c233f4eff8bb8a4124275032fa7c604f76d8: Bug 1504756 - [marionette] Use waitForEvent() when waiting for events. r=ato
Henrik Skupin <mail@hskupin.info> - Wed, 05 Dec 2018 19:56:56 +0000 - rev 449454
Push 35164 by shindli@mozilla.com at Thu, 06 Dec 2018 04:02:49 +0000
Bug 1504756 - [marionette] Use waitForEvent() when waiting for events. r=ato Depends on D13659 Differential Revision: https://phabricator.services.mozilla.com/D13660
65858c8c0fbd2a8e1d76c92730328d1168bacb89: Bug 1504756 - [marionette] Add executeSoon() to run tasks on the main thread. r=ato
Henrik Skupin <mail@hskupin.info> - Wed, 05 Dec 2018 19:56:32 +0000 - rev 449453
Push 35164 by shindli@mozilla.com at Thu, 06 Dec 2018 04:02:49 +0000
Bug 1504756 - [marionette] Add executeSoon() to run tasks on the main thread. r=ato To be closer to other test harnesses which are using executeSoon() to run a task on the main thread, this patch adds the same method to Marionette's sync module. Differential Revision: https://phabricator.services.mozilla.com/D13659
795d02901ad0a295b7f741ccf8a726f184a2799d: Bug 1512112: Remove redundant includes from source files in layout. r=TYLin
Daniel Holbert <dholbert@cs.stanford.edu> - Wed, 05 Dec 2018 18:55:59 +0000 - rev 449452
Push 35164 by shindli@mozilla.com at Thu, 06 Dec 2018 04:02:49 +0000
Bug 1512112: Remove redundant includes from source files in layout. r=TYLin All of the removed includes are redundant (i.e. they're #included elsewhere in the same file). In most cases, I'm removing the second (redundant) copy of the #include, except when that copy makes more sense (i.e. if it's in better sorted order, or if it's paired alongside a closely-associated header while the earlier copy is not). Here's the script that I used to generate candidates here -- I ran this in every subdirectory of layout, on my linux machine (warning, this writes two files to your /tmp directory): for FILE in *.h *.cpp; do nonunique=$(grep \#include $FILE | grep -v List\.h | cut -f2 -d'"' | cut -f2- -d'/'| cut -f2- -d'/' | sort | wc -l) unique=$( grep \#include $FILE | grep -v List\.h | cut -f2 -d'"' | cut -f2- -d'/'| cut -f2- -d'/' | sort | uniq | wc -l) if [[ "$unique" != "$nonunique" ]]; then echo "$FILE: $nonunique / $unique" grep \#include $FILE | cut -f2 -d'"' | grep -v List\.h | cut -f2- -d'/'| cut -f2- -d'/' | sort > /tmp/nonunique.txt grep \#include $FILE | cut -f2 -d'"' | grep -v List\.h | cut -f2- -d'/'| cut -f2- -d'/' | sort | uniq > /tmp/unique.txt diff /tmp/nonunique.txt /tmp/unique.txt echo fi done Depends on D13773 Differential Revision: https://phabricator.services.mozilla.com/D13774
a259d877cf28367132faa18fd7c94fa72d347b9f: Bug 1512276 - Remove the waitForSecurityChange() defined in browser_trackingUI_animation_2.js r=johannh
Ehsan Akhgari <ehsan@mozilla.com> - Wed, 05 Dec 2018 19:48:01 +0000 - rev 449451
Push 35164 by shindli@mozilla.com at Thu, 06 Dec 2018 04:02:49 +0000
Bug 1512276 - Remove the waitForSecurityChange() defined in browser_trackingUI_animation_2.js r=johannh Differential Revision: https://phabricator.services.mozilla.com/D13850
863de59a48424d5f61c60146162a6e80f9a46653: Bug 1509460 - Disable / re-enable and delete/bring back should not bring the flexbox highlighter back r=gl
Michael Ratcliffe <mratcliffe@mozilla.com> - Wed, 05 Dec 2018 19:54:56 +0000 - rev 449450
Push 35164 by shindli@mozilla.com at Thu, 06 Dec 2018 04:02:49 +0000
Bug 1509460 - Disable / re-enable and delete/bring back should not bring the flexbox highlighter back r=gl Differential Revision: https://phabricator.services.mozilla.com/D12778
d186ca68a92d11b4d45397a04300e30226628e75: Bug 1512054 part 2: Fix alphabetical sorting of SVGObserverUtils.cpp's includes. r=TYLin
Daniel Holbert <dholbert@cs.stanford.edu> - Wed, 05 Dec 2018 15:20:58 +0000 - rev 449449
Push 35164 by shindli@mozilla.com at Thu, 06 Dec 2018 04:02:49 +0000
Bug 1512054 part 2: Fix alphabetical sorting of SVGObserverUtils.cpp's includes. r=TYLin (Also, include the full export path when including 'ImageLoader.h', for consistency with how we include it everywhere else outside of its own directory.) Depends on D13743 Differential Revision: https://phabricator.services.mozilla.com/D13773
0648f6fb0eeffe0975343ffbc879a95c284e1b78: Bug 1512054 part 1: Add some needed "Inlines" includes to .cpp files in layout, to address some -Wundefined-inline build warnings in non-unified builds. r=TYLin
Daniel Holbert <dholbert@cs.stanford.edu> - Wed, 05 Dec 2018 02:03:04 +0000 - rev 449448
Push 35164 by shindli@mozilla.com at Thu, 06 Dec 2018 04:02:49 +0000
Bug 1512054 part 1: Add some needed "Inlines" includes to .cpp files in layout, to address some -Wundefined-inline build warnings in non-unified builds. r=TYLin I'm also adding the non-"inlines" version of the added "inlines" includes, too, since it's nice to have them included as a pair. (It's not strictly necessary, since in these cases we were already indirectly including the non-"inlines" header, but it seems like a good practice.) This patch fixes build warnings in non-unified builds for the following calls: - The call to IsColumnSpan() in layout/generic/ColumnSetWrapperFrame.cpp - The call to Type() in layout/style/MappedDeclarations.cpp - The call to IsInAnonymousSubtree() in layout/svg/SVGObserverUtils.cpp Differential Revision: https://phabricator.services.mozilla.com/D13743
ad7fce1f1772f64763f2a6006d7a6d645606addc: Bug 1510602 - update build documentation to mention artifact builds r=firefox-build-system-reviewers,gps
Kim Moir <kmoir@mozilla.com> - Wed, 05 Dec 2018 19:49:38 +0000 - rev 449447
Push 35164 by shindli@mozilla.com at Thu, 06 Dec 2018 04:02:49 +0000
Bug 1510602 - update build documentation to mention artifact builds r=firefox-build-system-reviewers,gps Differential Revision: https://phabricator.services.mozilla.com/D13435
8049d1a1188073f9cb5bfb489b690ef57ee3e5d9: Bug 1511759 - Null-check focus path cached accessible. r=yzen
Eitan Isaacson <eitan@monotonous.org> - Wed, 05 Dec 2018 19:14:54 +0000 - rev 449446
Push 35164 by shindli@mozilla.com at Thu, 06 Dec 2018 04:02:49 +0000
Bug 1511759 - Null-check focus path cached accessible. r=yzen Differential Revision: https://phabricator.services.mozilla.com/D13729
79e8494d5df5c0e59a373e054ba25f1f25bb40bf: Bug 1511137 - Track more accurately when the main thread originates a resolution change. r=kats
Botond Ballo <botond@mozilla.com> - Wed, 05 Dec 2018 16:29:18 +0000 - rev 449445
Push 35164 by shindli@mozilla.com at Thu, 06 Dec 2018 04:02:49 +0000
Bug 1511137 - Track more accurately when the main thread originates a resolution change. r=kats The tracking is done using nsAtom origins, similarly to how updates to the scroll offset are tracked. Currently, APZ still uses some heuristics to deduce that the main thread originated a resolution change in some cases, but the intention is to try to remove those and rely only on this mechanism in the future. Differential Revision: https://phabricator.services.mozilla.com/D13741
4ad8241eb92a80771580ac497e4e4c2468f861da: Bug 1511398 - Check if the retrieved accessible child is actually valid when filling the bundle's data, r=yzen
Eitan Isaacson <eitan@monotonous.org> - Wed, 05 Dec 2018 19:06:40 +0000 - rev 449444
Push 35164 by shindli@mozilla.com at Thu, 06 Dec 2018 04:02:49 +0000
Bug 1511398 - Check if the retrieved accessible child is actually valid when filling the bundle's data, r=yzen Differential Revision: https://phabricator.services.mozilla.com/D13611
b551becf88cfa065b2394e2262836b3207464898: Bug 1511465: Fix race condition in browser_oversized.js. r=mak
Dave Townsend <dtownsend@oxymoronical.com> - Wed, 05 Dec 2018 19:03:36 +0000 - rev 449443
Push 35164 by shindli@mozilla.com at Thu, 06 Dec 2018 04:02:49 +0000
Bug 1511465: Fix race condition in browser_oversized.js. r=mak Seems that when run as the first test in a test run there is a race for whether the favicon for the initial tab has already been seen or not. Rarely we fail the race and end up seeing a successful favicon load. This makes us ignore any favicons other than the one we're interested in. Differential Revision: https://phabricator.services.mozilla.com/D13838
f2574f5b186e520f5ee9a1626c8bc828f2761bf1: Bug 1504659 Part 2: Add tests of viewport RDM scroll behavior, with and without touch simulation. r=gl
Brad Werth <bwerth@mozilla.com> - Wed, 05 Dec 2018 17:16:36 +0000 - rev 449442
Push 35164 by shindli@mozilla.com at Thu, 06 Dec 2018 04:02:49 +0000
Bug 1504659 Part 2: Add tests of viewport RDM scroll behavior, with and without touch simulation. r=gl Differential Revision: https://phabricator.services.mozilla.com/D13174
45f63618f66e184f8b20da48fe37fb3f546094f2: Bug 1504659 Part 1: Make RefreshVisualViewportSize allow non-APZ zooming, and call it during RefreshViewportSize. r=botond
Brad Werth <bwerth@mozilla.com> - Wed, 05 Dec 2018 17:16:34 +0000 - rev 449441
Push 35164 by shindli@mozilla.com at Thu, 06 Dec 2018 04:02:49 +0000
Bug 1504659 Part 1: Make RefreshVisualViewportSize allow non-APZ zooming, and call it during RefreshViewportSize. r=botond Differential Revision: https://phabricator.services.mozilla.com/D13173
643a4a6bbfe98aacb7f285b8f9e0a001794a9f8e: Merge inbound to mozilla-central. a=merge
shindli <shindli@mozilla.com> - Wed, 05 Dec 2018 23:33:31 +0200 - rev 449440
Push 35163 by shindli@mozilla.com at Wed, 05 Dec 2018 21:36:23 +0000
Merge inbound to mozilla-central. a=merge
(0) -300000 -100000 -30000 -10000 -3000 -1000 -300 -100 -50 -20 +20 +50 +100 +300 +1000 +3000 +10000 +30000 +100000 tip