869a14571e7e58445b0a33723887cf09de5cbc4a: Merge m-c to autoland. r=backout-merge a=merge-backout
Sebastian Hengst <archaeopteryx@coole-files.de> - Thu, 13 Oct 2016 18:50:18 +0200 - rev 317926
Push 82793 by cbook@mozilla.com at Fri, 14 Oct 2016 10:26:33 +0000
Merge m-c to autoland. r=backout-merge a=merge-backout
f7b6a16eb1f610c2a0291ebcc8474c72b4e993ca: Bug 1280573 - Add testing/mozbase to flake8 linter: r=ahal
Francesco Pischedda <francesco.pischedda@gmail.com> - Fri, 30 Sep 2016 16:08:37 +0200 - rev 317925
Push 82793 by cbook@mozilla.com at Fri, 14 Oct 2016 10:26:33 +0000
Bug 1280573 - Add testing/mozbase to flake8 linter: r=ahal added testing/mozbase to tools/lint/flake8.lint fixed a first batch of PEP8 errors/warnings at first the commad autopep8 -i --max-line-length 99 -r -j 8 . has been used to fix simpler problems, run from testing/mozbase some of the issues can not easily fixed : - undefined 'names' in code for example isLinux - isLinux and isBsd "fixed" with # noqa - undefined 'message' resolved with return fmt.format(... - undefined 'structured' resolved replacing those with mozlog - long comments - some remaining - addressed with # noqa - package level import everything - addressed with # flake8: noqa restored testing/mozbase/mozdevice/mozdevice/Zeroconf.py fixed issues reported on mozreview fixed ')' in testing/mozbase/mozprocess/mozprocess/qijo.py imports finally fixed multiline string at testing/mozbase/manifestparser/tests/test_manifestparser.py:114 ^^^ and again, but now with ./mach python-test --path-only testing/mozbase/manifestparser/tests/test_manifestparser.py passing fixed testing/mozbase/manifestparser/tests/test_convert_directory.py assert fixed this error: 10:15:21 INFO - return lambda line: stack_fixer_module.fixSymbols(line) 10:15:21 INFO - TypeError: fixSymbols() takes exactly 2 arguments (1 given) fixed two spaces lint error even of # noqa comments restored assignement to lambda with # noqa to silence the lint error global noqa for testing/mozbase/manifestparser/tests/test_filters.py stupid is/is not error... MozReview-Commit-ID: 1FpJF54GqIi
797b8b501271080944a1ea7970484872db49390d: Bug 1308878 - Implement minimal Redux store for Net panel r=Honza
Ricky Chien <ricky060709@gmail.com> - Wed, 12 Oct 2016 18:56:08 +0800 - rev 317924
Push 82793 by cbook@mozilla.com at Fri, 14 Oct 2016 10:26:33 +0000
Bug 1308878 - Implement minimal Redux store for Net panel r=Honza MozReview-Commit-ID: GgB5jzNpz9z
b881e74167da67c0e933bb67f12a249f6cea6ba3: Bug 1307030: remove android partner builds r=jmaher
Dustin J. Mitchell <dustin@mozilla.com> - Thu, 13 Oct 2016 14:21:26 +0000 - rev 317923
Push 82793 by cbook@mozilla.com at Fri, 14 Oct 2016 10:26:33 +0000
Bug 1307030: remove android partner builds r=jmaher This leaves the EXTRA_CHECKOUT_REPOSITORIES support in place, but unused. This will likely be removed when the last vestiges of tc-vcs are removed. The mozharness scripts for partner builds remain. MozReview-Commit-ID: CMu5s3nKhTB
d064196c2214c6e2d5e52d4cf9486077533b7f5c: Bug 1088305 - Removed unused constant Services; r=jdescottes
Sebastian Zartner <sebastianzartner@gmail.com> - Mon, 10 Oct 2016 19:13:28 +0200 - rev 317922
Push 82793 by cbook@mozilla.com at Fri, 14 Oct 2016 10:26:33 +0000
Bug 1088305 - Removed unused constant Services; r=jdescottes MozReview-Commit-ID: 53EJz5j0tzb
0eac6bd98bd0bedff228108ec924c87b38762c7a: Bug 1088305 - Adjusted browser theme test to check Firebug theme; r=jdescottes
Sebastian Zartner <sebastianzartner@gmail.com> - Sat, 08 Oct 2016 01:54:17 +0200 - rev 317921
Push 82793 by cbook@mozilla.com at Fri, 14 Oct 2016 10:26:33 +0000
Bug 1088305 - Adjusted browser theme test to check Firebug theme; r=jdescottes MozReview-Commit-ID: 4mCaF46aLx8
720ec39fe42b2a2b834dca6513f4281ab66d1fc2: Bug 1088305 - Allowed to customize the text color of the font preview tooltip; r=jdescottes
Sebastian Zartner <sebastianzartner@gmail.com> - Thu, 06 Oct 2016 22:43:57 +0200 - rev 317920
Push 82793 by cbook@mozilla.com at Fri, 14 Oct 2016 10:26:33 +0000
Bug 1088305 - Allowed to customize the text color of the font preview tooltip; r=jdescottes MozReview-Commit-ID: 4dESdQNgwRA
752382b58cc46c9158af0f5325b388b32fd8d35d: Bug 1304689 - Ensure frame reconstructions don't clobber a 'stronger' scroll origin with a 'weaker' one. r=tnikkel
Kartikaya Gupta <kgupta@mozilla.com> - Tue, 11 Oct 2016 09:36:22 -0400 - rev 317919
Push 82793 by cbook@mozilla.com at Fri, 14 Oct 2016 10:26:33 +0000
Bug 1304689 - Ensure frame reconstructions don't clobber a 'stronger' scroll origin with a 'weaker' one. r=tnikkel If, within a single refresh driver tick, the scroll position is updated by JS explicitly, and then subsequently also updated by a frame reconstruction, the scroll origin from the former (nsGkAtoms::other) can get clobbered by the latter (to nsGkAtoms::restore). The restore scroll origin is "weaker" in that it can be ignored by the APZ code in some circumstances. This is undesirable because it means the JS scroll update also gets ignored. This patch ensures that when setting the scroll origin we don't do this clobbering of stronger origins with weaker origins. MozReview-Commit-ID: DA4EHp1Debu
c07923f57466df5e436ec55c0112d89d59ac3e39: Bug 1309589 - Cleanup old loop.* preferences in profiles after Firefox Hello removal. r=mikedeboer
Mark Banner <standard8@mozilla.com> - Wed, 12 Oct 2016 16:08:55 +0100 - rev 317918
Push 82793 by cbook@mozilla.com at Fri, 14 Oct 2016 10:26:33 +0000
Bug 1309589 - Cleanup old loop.* preferences in profiles after Firefox Hello removal. r=mikedeboer MozReview-Commit-ID: KNOJzUL7hRv
971b4fc7d1f4a23a10f083b6d35a947ab6a8b929: Bug 1309794 - Make RefCnt types non-copyable. r=froydnj
Xidorn Quan <me@upsuper.org> - Thu, 13 Oct 2016 16:52:54 +1100 - rev 317917
Push 82793 by cbook@mozilla.com at Fri, 14 Oct 2016 10:26:33 +0000
Bug 1309794 - Make RefCnt types non-copyable. r=froydnj MozReview-Commit-ID: Lx344xrXDZT
4e2c6be6037962a0d5b9ad868e47278d71d5a535: Bug 1306551 - don't update playback position for video-only seek when seek is completed. r=kaku
JW Wang <jwwang@mozilla.com> - Thu, 06 Oct 2016 17:06:58 +0800 - rev 317916
Push 82793 by cbook@mozilla.com at Fri, 14 Oct 2016 10:26:33 +0000
Bug 1306551 - don't update playback position for video-only seek when seek is completed. r=kaku MozReview-Commit-ID: 7vByX0pPNo
3c6457fdf27c0f4d092049fafea3b21a41ebbc77: Merge mozilla-central to autoland
Carsten "Tomcat" Book <cbook@mozilla.com> - Thu, 13 Oct 2016 12:00:23 +0200 - rev 317915
Push 82793 by cbook@mozilla.com at Fri, 14 Oct 2016 10:26:33 +0000
Merge mozilla-central to autoland
70d19ed29c980ba3d6825e5286bda08fa7dc676f: Bug 1306639 - Searching in locationbar by typing something and pressing enter is not accounted in telemetry. r=adw
Marco Bonardo <mbonardo@mozilla.com> - Thu, 06 Oct 2016 17:40:13 +0200 - rev 317914
Push 82793 by cbook@mozilla.com at Fri, 14 Oct 2016 10:26:33 +0000
Bug 1306639 - Searching in locationbar by typing something and pressing enter is not accounted in telemetry. r=adw MozReview-Commit-ID: 9r8IyyyxruC
64cd7d87e78cf2e4b919aad10fb9f4962834b004: Bug 1309593 - remove b2g, mulet, graphene, horizon references from mozharness. r=catlee
Joel Maher - Wed, 12 Oct 2016 19:20:02 +0000 - rev 317913
Push 82793 by cbook@mozilla.com at Fri, 14 Oct 2016 10:26:33 +0000
Bug 1309593 - remove b2g, mulet, graphene, horizon references from mozharness. r=catlee MozReview-Commit-ID: JdrltSor9dL
d3cf438132337bcc7f615859863271a95f746a28: Bug 1304620 part.5 ContentCacheInParent should store the latest composition start offset with mCompositionStartInChild r=m_kato
Masayuki Nakano <masayuki@d-toybox.com> - Wed, 12 Oct 2016 21:52:01 +0900 - rev 317912
Push 82793 by cbook@mozilla.com at Fri, 14 Oct 2016 10:26:33 +0000
Bug 1304620 part.5 ContentCacheInParent should store the latest composition start offset with mCompositionStartInChild r=m_kato When ContentCacheInParent receives eCompositionStart, it temporarily sets mCompositionStart to selection start offset. However, if there is a composition in the remote process, the selection start is caret position in the composition string. Therefore, it's not useful information. Instead, the composition start offset should be used because around there are a lot of information. For that, ContentCacheInParent should always store compostion start offset in the remote process with mCompositionStartInChild even if mWidgetHasComposition is false. And when it receives eCompositionStart, mCompositionStart should be set to mCompositionStartInChild. MozReview-Commit-ID: DksPNEsi6Ec
aa2c12432274f8926ffbc3b35502eedf176034b1: Bug 1304620 part.4 ContentCacheInParent::mCompositionStart should be set to better value for mWidgetHasComposition state r=m_kato
Masayuki Nakano <masayuki@d-toybox.com> - Wed, 12 Oct 2016 22:05:09 +0900 - rev 317911
Push 82793 by cbook@mozilla.com at Fri, 14 Oct 2016 10:26:33 +0000
Bug 1304620 part.4 ContentCacheInParent::mCompositionStart should be set to better value for mWidgetHasComposition state r=m_kato ContentCacheInParent::mCompositionStart was set to ContentCacheInChild::mCompositionStart without any check. However, that's clearly wrong approach. For example, when the remote process handles some composition events after eCompositionCommit(AsIs) in the parent process, mCompositionStart is valid offset even after mWidgetHasComposition is set to false. Similarly, even after parent process sends eCompositionStart, the remote process may send invalid offset for mCompositionStart due to no composition in it. For solving this issue, ContentCacheInParent should check mWidgetHasComposition. If it's true and coming offset is valid, let's use it (even if mPendingCompositionCount is 2 or bigger since widget shouldn't use WidgetQueryContentEvent when there are some pending events). If the coming offset is invalid but mWidgetHasComposition is false, let's use selection start instead because HandleQueryContentEvent() can work around selection start offset only. Otherwise, i.e., mWidgetHasComposition is false, we should always set mCompositionStart to invalid offset. MozReview-Commit-ID: IONU0Cbhpil
64f030acc66bae6f210b5d8a2d82e38993bde2cf: Bug 1304620 part.3 The start offset of TextComposition instance in the parent process shouldn't be updated with older composition in the remote process r=m_kato
Masayuki Nakano <masayuki@d-toybox.com> - Wed, 12 Oct 2016 22:03:16 +0900 - rev 317910
Push 82793 by cbook@mozilla.com at Fri, 14 Oct 2016 10:26:33 +0000
Bug 1304620 part.3 The start offset of TextComposition instance in the parent process shouldn't be updated with older composition in the remote process r=m_kato When ContentCacheInParent receives content information from the remote process, it notifies TextComposition of the latest composition start offset in the remote process. However, the information may be older composition's, i.e., the composition was already committed in the process but is still being handled by the remote process. TextComposition shouldn't work with such obsolete information. Note that TextComposition instance is created and destroyed when WidgetCompostionEvent is handled by IMEStateManager. Then, TextComposition instance guarantees that all following composition events for a composition are sent to same EventTarget (including TabParent). So, TextComposition is always synced with a composition in widget. MozReview-Commit-ID: 78NuvpE2rPx
97cb51652c868d8698c966ad0483f1421be1d449: Bug 1304620 part.2 ContentCacheInParent should manage if there is pending composition in the remote process r=m_kato
Masayuki Nakano <masayuki@d-toybox.com> - Wed, 12 Oct 2016 17:09:02 +0900 - rev 317909
Push 82793 by cbook@mozilla.com at Fri, 14 Oct 2016 10:26:33 +0000
Bug 1304620 part.2 ContentCacheInParent should manage if there is pending composition in the remote process r=m_kato If the remote process is busy or user restarts composition too quickly, there could be 2 or more compositions in ContentCache. For managing such case, ContentCacheInParent should manage the pending composition count which is increased at dispatching eCompositionStart event to the remote process and decreased at receiving eCompositionCommit(AsIs) event from the remote process. MozReview-Commit-ID: KbTsK20NEZD
8bea9647dc86865e874f346a3fccc14565ce3f65: Bug 1304620 part.1 Rename ContentCacheInParent::mIsComposing to mWidgetHasComposition r=m_kato
Masayuki Nakano <masayuki@d-toybox.com> - Wed, 12 Oct 2016 16:42:28 +0900 - rev 317908
Push 82793 by cbook@mozilla.com at Fri, 14 Oct 2016 10:26:33 +0000
Bug 1304620 part.1 Rename ContentCacheInParent::mIsComposing to mWidgetHasComposition r=m_kato For making the meaning of ContentCacheInParent::mIsComposing clearer, let's rename it to mWidgetHasComposition. It becomes true when the parent process sends eCompositionStart to the remote process and false when the parent process sends eCompositionCommit(AsIs). So, it represents if the widget (i.e., the native IME handler in the chrome process) has composition. MozReview-Commit-ID: 5k05IXMgJxw
84286e454b4e67a193470eb68fb2a504835999fd: Backed out changeset fe84473a739c (bug 567954) for failing mda tests due to own test
Carsten "Tomcat" Book <cbook@mozilla.com> - Thu, 13 Oct 2016 11:32:45 +0200 - rev 317907
Push 82793 by cbook@mozilla.com at Fri, 14 Oct 2016 10:26:33 +0000
Backed out changeset fe84473a739c (bug 567954) for failing mda tests due to own test
(0) -300000 -100000 -30000 -10000 -3000 -1000 -300 -100 -50 -20 +20 +50 +100 +300 +1000 +3000 +10000 +30000 +100000 tip