dc3f655d626efcbff67d819f0195d56e06f8cd43: Bug 1561179 - P3: Merge EnsureRDDReady into LaunchRDDProcess. r=mattwoodrow
Dan Glastonbury <dan.glastonbury@gmail.com> - Mon, 04 Nov 2019 03:41:54 +0000 - rev 500314
Push 36761 by dvarga@mozilla.com at Mon, 04 Nov 2019 09:41:18 +0000
Bug 1561179 - P3: Merge EnsureRDDReady into LaunchRDDProcess. r=mattwoodrow LaunchRDDProcess() and CreateContentBridge() create a sync creation. Merge the functions into one function. Keep the IPDL messaging async to avoid adding a exception for the message being sync. Differential Revision: https://phabricator.services.mozilla.com/D50400
8b9accb338043f9e7cdeb90f77b76927cd19a143: Bug 1561179 - P2: Rename VideoBridge methods and members. r=mattwoodrow
Dan Glastonbury <dan.glastonbury@gmail.com> - Mon, 04 Nov 2019 03:41:47 +0000 - rev 500313
Push 36761 by dvarga@mozilla.com at Mon, 04 Nov 2019 09:41:18 +0000
Bug 1561179 - P2: Rename VideoBridge methods and members. r=mattwoodrow To bring the naming more inline with ImageBridge. Differential Revision: https://phabricator.services.mozilla.com/D50399
9942990fbff5baf5c76aa811fa43896b5c678d39: Bug 1561179 - P1: Enable the creation of multiple VideoBridgeParent actors. r=mattwoodrow
Dan Glastonbury <dan.glastonbury@gmail.com> - Mon, 04 Nov 2019 03:41:45 +0000 - rev 500312
Push 36761 by dvarga@mozilla.com at Mon, 04 Nov 2019 09:41:18 +0000
Bug 1561179 - P1: Enable the creation of multiple VideoBridgeParent actors. r=mattwoodrow Differential Revision: https://phabricator.services.mozilla.com/D50398
3a6ca30f318eaa59284333b04c33244be2f750fe: Bug 1593094. Add glyph flashing pref. r=kvark
Jeff Muizelaar <jrmuizel@gmail.com> - Fri, 01 Nov 2019 15:41:49 +0000 - rev 500311
Push 36761 by dvarga@mozilla.com at Mon, 04 Nov 2019 09:41:18 +0000
Bug 1593094. Add glyph flashing pref. r=kvark This makes it easier to debug glyph rasterization issues Differential Revision: https://phabricator.services.mozilla.com/D51310
582ee0d10d0c876207ef584a631e785993653e46: Bug 1370132 - Refactor how we send OnStatus and OnProgress events r=mayhemer
Kershaw Chang <kershaw@mozilla.com> - Sun, 03 Nov 2019 23:28:33 +0000 - rev 500310
Push 36761 by dvarga@mozilla.com at Mon, 04 Nov 2019 09:41:18 +0000
Bug 1370132 - Refactor how we send OnStatus and OnProgress events r=mayhemer 1. Send OnStatus and OnProgress IPC events over main thread. 2. Merge HttpChannelChild::OnProgress and HttpChannelChild::DoOnProgress into one function, since these two basically do the same thing. Differential Revision: https://phabricator.services.mozilla.com/D51407
1e151275792d8193004412bb7e97ae6f36f1728f: Bug 1575159 - Implement partial invalidation on Windows r=gw
sotaro <sotaro.ikeda.g@gmail.com> - Sun, 03 Nov 2019 10:57:03 +0000 - rev 500309
Push 36761 by dvarga@mozilla.com at Mon, 04 Nov 2019 09:41:18 +0000
Bug 1575159 - Implement partial invalidation on Windows r=gw Partial present for WR is off by default. Differential Revision: https://phabricator.services.mozilla.com/D51358
a4c51f3f0bb1dd51ded827d3049d633f56662a7f: Bug 1590898 - Forward stream filter creation requests across PDocumentChannel. r=kmag
Matt Woodrow <mwoodrow@mozilla.com> - Sun, 03 Nov 2019 22:15:48 +0000 - rev 500308
Push 36761 by dvarga@mozilla.com at Mon, 04 Nov 2019 09:41:18 +0000
Bug 1590898 - Forward stream filter creation requests across PDocumentChannel. r=kmag Differential Revision: https://phabricator.services.mozilla.com/D50898
ff0b109afd62c3c2dd396463bb3632228e959ff3: Bug 1548274 - Disabled browser_cancel_content_js.js on Windows10 x64 debug. r=gbrown
Razvan Maries <rmaries@mozilla.com> - Sun, 03 Nov 2019 20:48:08 +0000 - rev 500307
Push 36761 by dvarga@mozilla.com at Mon, 04 Nov 2019 09:41:18 +0000
Bug 1548274 - Disabled browser_cancel_content_js.js on Windows10 x64 debug. r=gbrown Differential Revision: https://phabricator.services.mozilla.com/D51496
611bf4a5e7a22d20f99afc5d56f6b4b8a683cd26: Bug 1575657 - Always create record/replay checkpoints when painting, r=mstange.
Brian Hackett <bhackett1024@gmail.com> - Sun, 03 Nov 2019 12:22:20 +0000 - rev 500306
Push 36760 by rgurzau@mozilla.com at Sun, 03 Nov 2019 21:38:57 +0000
Bug 1575657 - Always create record/replay checkpoints when painting, r=mstange. Differential Revision: https://phabricator.services.mozilla.com/D46244
24345627e6de238e5a666ff5f9a408e8edbdf371: bug 1591927: remote: implement Security.setIgnoreCertificateErrors; r=remote-protocol-reviewers,maja_zf
Andreas Tolfsen <ato@sny.no> - Sat, 02 Nov 2019 18:08:56 +0000 - rev 500305
Push 36760 by rgurzau@mozilla.com at Sun, 03 Nov 2019 21:38:57 +0000
bug 1591927: remote: implement Security.setIgnoreCertificateErrors; r=remote-protocol-reviewers,maja_zf This implements an all-or-nothing insecure sweeping override that bypasses security exceptions when loading documents with invalid or otherwise bad TLS certificates. Differential Revision: https://phabricator.services.mozilla.com/D50838
89521f19577223ea653809b25d04e33bb336cc53: Bug 1590644 - Set font family on the content select menupopup, not menuitem. r=mconley
Emilio Cobos Alvarez <emilio@mozilla.com> - Sun, 03 Nov 2019 10:04:34 +0000 - rev 500304
Push 36760 by rgurzau@mozilla.com at Sun, 03 Nov 2019 21:38:57 +0000
Bug 1590644 - Set font family on the content select menupopup, not menuitem. r=mconley So that fonts chosen by the author inherit into the menuitems properly, instead of being overriden. Differential Revision: https://phabricator.services.mozilla.com/D51086
6cc550e9edcde95eaba889aa340da4f9781dfb09: Bug 1588745 - part 7: Remove `AutoScriptBlocker` from `TextControlState::SetValue()` r=Ehsan
Masayuki Nakano <masayuki@d-toybox.com> - Fri, 01 Nov 2019 21:10:17 +0000 - rev 500303
Push 36760 by rgurzau@mozilla.com at Sun, 03 Nov 2019 21:38:57 +0000
Bug 1588745 - part 7: Remove `AutoScriptBlocker` from `TextControlState::SetValue()` r=Ehsan For allowing `TextEditor` to dispatch DOM events synchronously, we should remove `AutoScriptBlocker` in `TextControlState::SetValue()` right now. According to the comment around the `AutoScriptBlocker`, `PrepareEditor()` may be called while setting value. Therefore, this patch makes `AutoTextControlHandlingState` call it if `PrepareEditor()` is called while handling `SetValue()` and when the top most `SetValue()` ends its job. Depends on D51396 Differential Revision: https://phabricator.services.mozilla.com/D51397
7608c252368ea6389d33d820fc2fbb9207dd2130: Bug 1588745 - part 6: Post processing of setting value with TextEditor should be done before dispatching "input" event r=Ehsan
Masayuki Nakano <masayuki@d-toybox.com> - Sun, 03 Nov 2019 02:50:00 +0000 - rev 500302
Push 36760 by rgurzau@mozilla.com at Sun, 03 Nov 2019 21:38:57 +0000
Bug 1588745 - part 6: Post processing of setting value with TextEditor should be done before dispatching "input" event r=Ehsan Currently, "input" event is fired when the `AutoScriptBlocker` in `SetValue()` is deleted. So, for keeping same behavior, the post processing after calling `TextEditor` methods should be done before editor dispatches "input" event. Fortunately, `TextInputListener::OnEditActionHandled()` is a good chance to do that. Therefore, this patch makes it notify `TextControlState` and `AutoTextControlHandlingState`. Note that ideally, each method of `TextEditor` should return `NS_ERROR_OUT_OF_MEMORY` coming from `AutoTextControlHandlingState::OnEditActionHandled()`. However, it requires a lot of changes in editor classes, and the case is really rare since editor does not use fallible allocation. Therefore, it must be okay to crash in editor if `OnEditActionHandled()` returns `NS_ERROR_OUT_OF_MEMORY`. Depends on D51395 Differential Revision: https://phabricator.services.mozilla.com/D51396
7754b00fcfc82f9c98eafd780b90de4b4fc7cc1c: Bug 1588745 - part 5: Split `TextControlState::SetValue()` r=Ehsan
Masayuki Nakano <masayuki@d-toybox.com> - Sat, 02 Nov 2019 23:23:54 +0000 - rev 500301
Push 36760 by rgurzau@mozilla.com at Sun, 03 Nov 2019 21:38:57 +0000
Bug 1588745 - part 5: Split `TextControlState::SetValue()` r=Ehsan `TextControlState::SetValue()` does 4 things. 1. Committing composition if there is and if possible. 2. Setting value with `TextEditor` if text editor and frame are available. 3. Setting value without `TextEditor` otherwise. 4. Notifying value changed. We can split #2 and #3 from it now because `AutoTextControlHandlingState` manages nested actions. Therefore, this patch creates `SetValueWithTextEditor()` and `SetValueWithoutTextEditor()` which take `AutoTextControlHandlingState`. Depends on D51394 Differential Revision: https://phabricator.services.mozilla.com/D51395
041047456efaad17ede7547324c20c579b96bbd6: Bug 1588745 - part 4: Make `TextControlState` reuse its instance by itself r=Ehsan
Masayuki Nakano <masayuki@d-toybox.com> - Fri, 01 Nov 2019 20:51:48 +0000 - rev 500300
Push 36760 by rgurzau@mozilla.com at Sun, 03 Nov 2019 21:38:57 +0000
Bug 1588745 - part 4: Make `TextControlState` reuse its instance by itself r=Ehsan Currently, only `HTMLInputElement` reuses `TextControlState` instance since `HTMLTextAreaElement` had the instance as a member rather than allocate it. Now, all instances are allocated in the heap independently for guaranteeing their lifetime. So, the reuse mechanism should be managed by `TextControlState` itself. Depends on D51393 Differential Revision: https://phabricator.services.mozilla.com/D51394
5915226eefd509a7916d5534eef0e2942d14596e: Bug 1588745 - part 3: Make `TextControlState` not deleted actually while it handles something r=Ehsan
Masayuki Nakano <masayuki@d-toybox.com> - Fri, 01 Nov 2019 20:49:12 +0000 - rev 500299
Push 36760 by rgurzau@mozilla.com at Sun, 03 Nov 2019 21:38:57 +0000
Bug 1588745 - part 3: Make `TextControlState` not deleted actually while it handles something r=Ehsan Currently, nobody guarantees that `TextControlState` won't be deleted while it handles something with `MOZ_CAN_RUN_SCRIPT` methods. This patch hides its destructor (and constructor) for making only `TextControlState` itself can delete its instances. Then, if instance owner wants to delete it while handling action(s), the oldest `AutoHandlingState` will delete the `TextControlState`. Depends on D51392 Differential Revision: https://phabricator.services.mozilla.com/D51393
191c2592d268a7bd4e510d02d71b87e3764800ea: Bug 1588745 - part 2: Move some `TextControlState` members to stack only class r=Ehsan
Masayuki Nakano <masayuki@d-toybox.com> - Fri, 01 Nov 2019 20:47:35 +0000 - rev 500298
Push 36760 by rgurzau@mozilla.com at Sun, 03 Nov 2019 21:38:57 +0000
Bug 1588745 - part 2: Move some `TextControlState` members to stack only class r=Ehsan Some members of `TextControlState` are initialized and restored in same block scopes. Therefore, with creating new stack only class and storing latest one with a new member variable, we can store all of them in the stack. Depends on D51391 Differential Revision: https://phabricator.services.mozilla.com/D51392
88c5fa54553b1c8d4b1cdd919821b0ee8ea582b7: Bug 1588745 - part 1: Rename `nsTextEditorState` to `mozilla::TextControlState` r=Ehsan
Masayuki Nakano <masayuki@d-toybox.com> - Fri, 01 Nov 2019 20:43:11 +0000 - rev 500297
Push 36760 by rgurzau@mozilla.com at Sun, 03 Nov 2019 21:38:57 +0000
Bug 1588745 - part 1: Rename `nsTextEditorState` to `mozilla::TextControlState` r=Ehsan It should be in `mozilla` namespace and it manages not only `TextEditor`, manages selection, selection controller and callback from editor. so that I think it stores state of "text control widget". Therefore, I name it to `TextControlState`. And cleaning up the cpp file. Differential Revision: https://phabricator.services.mozilla.com/D51391
d0f88eadd0f583df0e805409c8158bfa6f826d92: Bug 1592417 - Reduce work done when picture cache tiles are inside the display port but not currently visible. r=nical
Glenn Watson <git@intuitionlibrary.com> - Sun, 03 Nov 2019 03:38:57 +0000 - rev 500296
Push 36760 by rgurzau@mozilla.com at Sun, 03 Nov 2019 21:38:57 +0000
Bug 1592417 - Reduce work done when picture cache tiles are inside the display port but not currently visible. r=nical Previously, WR needed to update and track dependencies for all allocated picture cache tiles in the virtual display port. This means doing extra CPU work (dependency updates) and in some cases, extra GPU work (larger off-screen child surfaces) than are strictly required. With this patch, each tile determines if it is currently visible in pre_update. If the tile isn't visible, we skip doing dependency updates until it is on screen again. More importantly, this is used to reduce the world culling rect for primitive preparation, which also means large child surfaces only require allocations large enough to enclose the visible tiles, rather than the display port. Differential Revision: https://phabricator.services.mozilla.com/D51006
0de7d040e6d10f2a9222c6f34d1d6af54b5a2d21: Backed out 3 changesets (bug 1582751, bug 1575785, bug 1592616) for causing perma failures in browser/base/content/test/siteProtections/browser_protections_UI.js CLOSED TREE
shindli <shindli@mozilla.com> - Sun, 03 Nov 2019 08:43:22 +0200 - rev 500295
Push 36760 by rgurzau@mozilla.com at Sun, 03 Nov 2019 21:38:57 +0000
Backed out 3 changesets (bug 1582751, bug 1575785, bug 1592616) for causing perma failures in browser/base/content/test/siteProtections/browser_protections_UI.js CLOSED TREE Backed out changeset d08754485c25 (bug 1575785) Backed out changeset be51af34cb7a (bug 1582751) Backed out changeset c753c25b8f77 (bug 1592616)
(0) -300000 -100000 -30000 -10000 -3000 -1000 -300 -100 -50 -20 +20 +50 +100 +300 +1000 +3000 +10000 +30000 tip