Skip to content

Commit

Permalink
Backed out 9 changesets (bug 1911021) for causing failures at Executi…
Browse files Browse the repository at this point in the history
…onTracer.cpp. CLOSED TREE

Backed out changeset 3df922f26c68 (bug 1911021)
Backed out changeset 5ae948b4c4cb (bug 1911021)
Backed out changeset b3b69ca7a7e2 (bug 1911021)
Backed out changeset addef9853a2b (bug 1911021)
Backed out changeset aed1adb7245b (bug 1911021)
Backed out changeset 285922033e2b (bug 1911021)
Backed out changeset 217454e2cedc (bug 1911021)
Backed out changeset c7365fc6d4ab (bug 1911021)
Backed out changeset 6d1accf8bf38 (bug 1911021)

UltraBlame original commit: f8229eb11c8571108c176fdc2308a6566b1912e0
  • Loading branch information
marco-c committed Oct 15, 2024
1 parent e91156b commit 827432f
Show file tree
Hide file tree
Showing 43 changed files with 4,000 additions and 5,256 deletions.
73 changes: 39 additions & 34 deletions devtools/client/performance-new/shared/utils.js
Original file line number Diff line number Diff line change
Expand Up @@ -1788,6 +1788,45 @@ windows
name
:
"
JSTracer
"
value
:
"
jstracer
"
title
:
"
Trace
JS
engine
"
experimental
:
true
disabledReason
:
"
JS
Tracer
is
currently
disabled
due
to
crashes
.
See
Bug
1565788
.
"
}
{
name
:
"
IPC
Messages
"
Expand Down Expand Up @@ -2369,40 +2408,6 @@ sample
.
"
}
{
name
:
"
JS
Execution
Tracing
"
value
:
"
tracing
"
title
:
"
Disable
periodic
stack
sampling
and
capture
information
about
every
JS
function
executed
.
"
experimental
:
true
}
]
;
module
Expand Down
7 changes: 7 additions & 0 deletions devtools/server/actors/tracer.js
Original file line number Diff line number Diff line change
Expand Up @@ -770,6 +770,13 @@ tracingListener
.
stop
(
lazy
.
JSTracer
.
maybeGetNativeTrace
(
)
)
;
this
Expand Down
182 changes: 182 additions & 0 deletions devtools/server/tracer/tracer.sys.mjs
Original file line number Diff line number Diff line change
Expand Up @@ -1182,6 +1182,24 @@ throws
*
param
{
Boolean
}
options
.
useNativeTracing
*
Optional
setting
to
enable
the
native
tracing
implementation
.
*
param
{
String
}
options
Expand Down Expand Up @@ -1817,6 +1835,16 @@ traceFunctionReturn
;
this
.
useNativeTracing
=
!
!
options
.
useNativeTracing
;
this
.
maxDepth
=
options
Expand Down Expand Up @@ -2388,6 +2416,24 @@ isTracing
=
true
;
if
(
this
.
useNativeTracing
)
{
this
.
dbg
.
nativeTracing
=
true
;
}
else
{
this
.
dbg
Expand All @@ -2398,6 +2444,7 @@ this
.
onEnterFrame
;
}
if
(
this
Expand Down Expand Up @@ -2669,6 +2716,16 @@ g
;
this
.
dbg
.
nativeTracing
=
this
.
useNativeTracing
;
this
.
debuggerNotificationObserver
.
connect
Expand Down Expand Up @@ -3704,6 +3761,49 @@ null
*
*
*
If
native
tracing
is
enabled
get
the
trace
from
the
native
tracer
*
/
maybeGetNativeTrace
(
)
{
if
(
this
.
useNativeTracing
)
{
return
this
.
dbg
.
collectNativeTrace
(
)
;
}
return
null
;
}
/
*
*
*
Stop
observing
execution
Expand Down Expand Up @@ -3768,6 +3868,25 @@ dbg
return
;
}
if
(
!
this
.
useNativeTracing
)
{
this
.
dbg
.
nativeTracing
=
false
;
}
else
{
this
.
dbg
Expand All @@ -3776,6 +3895,7 @@ onEnterFrame
=
undefined
;
}
this
.
dbg
Expand Down Expand Up @@ -6974,6 +7094,67 @@ tracing
*
*
*
If
native
tracing
is
enabled
get
the
trace
from
the
native
tracer
*
/
function
maybeGetNativeTrace
(
)
{
if
(
activeTracer
)
{
return
activeTracer
.
maybeGetNativeTrace
(
)
;
}
console
.
warn
(
"
Can
'
t
get
a
native
trace
as
we
were
not
tracing
.
"
)
;
return
null
;
}
/
*
*
*
Listen
for
tracing
Expand Down Expand Up @@ -7629,6 +7810,7 @@ JSTracer
{
startTracing
stopTracing
maybeGetNativeTrace
addTracingListener
removeTracingListener
NEXT_INTERACTION_MESSAGE
Expand Down
Loading

0 comments on commit 827432f

Please sign in to comment.