DevTools: surface workpace 2.0 errors |
||
Issue descriptionCurrently, Workspaces 2.0 don't surface reasons why it doesn't map certain files to each other. This causes a ton on confusion; we should be more transparent on how automapping works.
,
Jan 31 2018
The following revision refers to this bug: https://chromium.googlesource.com/chromium/src.git/+/39e7f07496b73307a619f16a3997fb7935872f19 commit 39e7f07496b73307a619f16a3997fb7935872f19 Author: Andrey Lushnikov <lushnikov@chromium.org> Date: Wed Jan 31 02:41:29 2018 DevTools: move TestMapping to use Persistence.addBdinging/removeBinding This patch: - migrates TestMapping to use generic Persistence methods addBinding/removeBinding instead of creating a custom mapping algorithm. - removes enabling/disabling of automapping; instead, the automapping gets disposed. Since the removal of DefaultMapping, Persistence doesn't need to support different mapping algorithm implementations. BUG= 805740 R=dgozman Change-Id: I5be46e4a3991011ae2bc7496619e4a1c9dcfb700 Reviewed-on: https://chromium-review.googlesource.com/892481 Commit-Queue: Andrey Lushnikov <lushnikov@chromium.org> Reviewed-by: Dmitry Gozman <dgozman@chromium.org> Cr-Commit-Position: refs/heads/master@{#533172} [modify] https://crrev.com/39e7f07496b73307a619f16a3997fb7935872f19/third_party/WebKit/Source/devtools/front_end/bindings_test_runner/PersistenceTestRunner.js [modify] https://crrev.com/39e7f07496b73307a619f16a3997fb7935872f19/third_party/WebKit/Source/devtools/front_end/persistence/Automapping.js [modify] https://crrev.com/39e7f07496b73307a619f16a3997fb7935872f19/third_party/WebKit/Source/devtools/front_end/persistence/Persistence.js
,
Feb 13 2018
The following revision refers to this bug: https://chromium.googlesource.com/chromium/src.git/+/a78f25a0b67f5ce93e331c69e7c0f9700636a8cb commit a78f25a0b67f5ce93e331c69e7c0f9700636a8cb Author: Andrey Lushnikov <lushnikov@chromium.org> Date: Tue Feb 13 07:48:53 2018 DevTools: prepare Automapping for statuses This patch renames AutomappingBinding into AutomappingStatus. The rename is to stress the conceptual difference between PersistenceBinding and AutomappingStatus: - PersistenceBinding will not exist unless there's a successful binding between UISourceCodes. Persistence will sync bound UISourceCodes. - AutomappingStatus reports on the Automapping for the network UISourceCode. It exists regardless of the mapping success and will carry debugging information. R=dgozman BUG= 805740 Change-Id: I345ba245099244b5c215aca022edd86416f400e2 Reviewed-on: https://chromium-review.googlesource.com/898165 Reviewed-by: Dmitry Gozman <dgozman@chromium.org> Commit-Queue: Andrey Lushnikov <lushnikov@chromium.org> Cr-Commit-Position: refs/heads/master@{#536282} [modify] https://crrev.com/a78f25a0b67f5ce93e331c69e7c0f9700636a8cb/third_party/WebKit/Source/devtools/front_end/bindings_test_runner/AutomappingTestRunner.js [modify] https://crrev.com/a78f25a0b67f5ce93e331c69e7c0f9700636a8cb/third_party/WebKit/Source/devtools/front_end/bindings_test_runner/PersistenceTestRunner.js [modify] https://crrev.com/a78f25a0b67f5ce93e331c69e7c0f9700636a8cb/third_party/WebKit/Source/devtools/front_end/persistence/Automapping.js [modify] https://crrev.com/a78f25a0b67f5ce93e331c69e7c0f9700636a8cb/third_party/WebKit/Source/devtools/front_end/persistence/Persistence.js
,
Dec 19
DevTools bug triage: closing low-pri issues with low demand. |
||
►
Sign in to add a comment |
||
Comment 1 by bugdroid1@chromium.org
, Jan 25 2018