Bug 1547084 Part 1 - Remove recordReplayDirective interface and uses, r=mccr8.

--HG--
extra : rebase_source : 9fcddba95a4759fe6eb1222199ad8eeaaba2e146
This commit is contained in:
Brian Hackett 2019-05-12 13:09:55 -10:00
Родитель bb7f35b18a
Коммит a231842bb7
7 изменённых файлов: 2 добавлений и 26 удалений

Просмотреть файл

@ -9,15 +9,12 @@
// Test some issues when stepping around after hitting a breakpoint while recording. // Test some issues when stepping around after hitting a breakpoint while recording.
add_task(async function() { add_task(async function() {
const dbg = await attachRecordingDebugger("doc_rr_continuous.html"); const dbg = await attachRecordingDebugger("doc_rr_continuous.html");
const {threadClient, tab, toolbox, target} = dbg; const {threadClient, tab, toolbox} = dbg;
await threadClient.interrupt(); await threadClient.interrupt();
const bp1 = await setBreakpoint(threadClient, "doc_rr_continuous.html", 19); const bp1 = await setBreakpoint(threadClient, "doc_rr_continuous.html", 19);
await resumeToLine(threadClient, 19); await resumeToLine(threadClient, 19);
await reverseStepOverToLine(threadClient, 18); await reverseStepOverToLine(threadClient, 18);
await checkEvaluateInTopFrame(target,
"SpecialPowers.Cu.recordReplayDirective(/* AlwaysTakeTemporarySnapshots */ 3)",
undefined);
await stepInToLine(threadClient, 22); await stepInToLine(threadClient, 22);
const bp2 = await setBreakpoint(threadClient, "doc_rr_continuous.html", 24); const bp2 = await setBreakpoint(threadClient, "doc_rr_continuous.html", 24);
await resumeToLine(threadClient, 24); await resumeToLine(threadClient, 24);

Просмотреть файл

@ -32,7 +32,5 @@ function testStepping2() {
c--; c--;
} }
window.setTimeout(f, 1); window.setTimeout(f, 1);
// Simulate a longer recording by marking major checkpoints whenever possible.
SpecialPowers.Cu.recordReplayDirective(/* AlwaysMarkMajorCheckpoints */ 4);
</script> </script>
</html> </html>

Просмотреть файл

@ -22,7 +22,5 @@ function f() {
window.setTimeout(recordingFinished); window.setTimeout(recordingFinished);
} }
window.setTimeout(f, 1); window.setTimeout(f, 1);
// Simulate a longer recording by marking major checkpoints whenever possible.
SpecialPowers.Cu.recordReplayDirective(/* AlwaysMarkMajorCheckpoints */ 4);
</script> </script>
</html> </html>

Просмотреть файл

@ -669,9 +669,6 @@ interface nsIXPCComponents_Utils : nsISupports
*/ */
AUTF8String readUTF8URI(in nsIURI url); AUTF8String readUTF8URI(in nsIURI url);
/* Give a directive to the record/replay system. */
void recordReplayDirective(in long directive);
/* Create a spellchecker object. */ /* Create a spellchecker object. */
nsIEditorSpellCheck createSpellChecker(); nsIEditorSpellCheck createSpellChecker();

Просмотреть файл

@ -2409,12 +2409,6 @@ nsXPCComponents_Utils::Now(double* aRetval) {
return NS_OK; return NS_OK;
} }
NS_IMETHODIMP
nsXPCComponents_Utils::RecordReplayDirective(int aDirective) {
recordreplay::RecordReplayDirective(aDirective);
return NS_OK;
}
NS_IMETHODIMP NS_IMETHODIMP
nsXPCComponents_Utils::CreateSpellChecker(nsIEditorSpellCheck** aSpellChecker) { nsXPCComponents_Utils::CreateSpellChecker(nsIEditorSpellCheck** aSpellChecker) {
NS_ENSURE_ARG_POINTER(aSpellChecker); NS_ENSURE_ARG_POINTER(aSpellChecker);

Просмотреть файл

@ -80,9 +80,7 @@ namespace recordreplay {
(aData, aSize)) Macro(InternalRegisterThing, \ (aData, aSize)) Macro(InternalRegisterThing, \
(void* aThing), (aThing)) \ (void* aThing), (aThing)) \
Macro(InternalUnregisterThing, (void* aThing), \ Macro(InternalUnregisterThing, (void* aThing), \
(aThing)) Macro(InternalRecordReplayDirective, \ (aThing)) \
(long aDirective), \
(aDirective)) \
Macro(BeginContentParse, \ Macro(BeginContentParse, \
(const void* aToken, const char* aURL, \ (const void* aToken, const char* aURL, \
const char* aContentType), \ const char* aContentType), \

Просмотреть файл

@ -258,10 +258,6 @@ static inline void RegisterThing(void* aThing);
static inline void UnregisterThing(void* aThing); static inline void UnregisterThing(void* aThing);
static inline size_t ThingIndex(void* aThing); static inline size_t ThingIndex(void* aThing);
// Give a directive to the record/replay system. For possible values for
// aDirective, see ProcessRecordReplay.h. This is used for testing purposes.
static inline void RecordReplayDirective(long aDirective);
// Helper for record/replay asserts, try to determine a name for a C++ object // Helper for record/replay asserts, try to determine a name for a C++ object
// with virtual methods based on its vtable. // with virtual methods based on its vtable.
static inline const char* VirtualThingName(void* aThing); static inline const char* VirtualThingName(void* aThing);
@ -440,8 +436,6 @@ MOZ_MAKE_RECORD_REPLAY_WRAPPER_VOID(UnregisterThing, (void* aThing), (aThing))
MOZ_MAKE_RECORD_REPLAY_WRAPPER(ThingIndex, size_t, 0, (void* aThing), (aThing)) MOZ_MAKE_RECORD_REPLAY_WRAPPER(ThingIndex, size_t, 0, (void* aThing), (aThing))
MOZ_MAKE_RECORD_REPLAY_WRAPPER(VirtualThingName, const char*, nullptr, MOZ_MAKE_RECORD_REPLAY_WRAPPER(VirtualThingName, const char*, nullptr,
(void* aThing), (aThing)) (void* aThing), (aThing))
MOZ_MAKE_RECORD_REPLAY_WRAPPER_VOID(RecordReplayDirective, (long aDirective),
(aDirective))
#undef MOZ_MAKE_RECORD_REPLAY_WRAPPER_VOID #undef MOZ_MAKE_RECORD_REPLAY_WRAPPER_VOID
#undef MOZ_MAKERECORDREPLAYWRAPPER #undef MOZ_MAKERECORDREPLAYWRAPPER