Closed
Bug 1377362
Opened 7 years ago
Closed 7 years ago
Make recording more generic so that it can go faster
Categories
(Core :: Graphics, enhancement)
Core
Graphics
Tracking
()
RESOLVED
FIXED
mozilla56
Tracking | Status | |
---|---|---|
firefox56 | --- | fixed |
People
(Reporter: jrmuizel, Assigned: jrmuizel)
References
Details
Attachments
(3 files)
62.56 KB,
patch
|
lsalzman
:
review+
|
Details | Diff | Splinter Review |
8.39 KB,
patch
|
lsalzman
:
review+
|
Details | Diff | Splinter Review |
8.28 KB,
patch
|
lsalzman
:
review+
|
Details | Diff | Splinter Review |
No description provided.
Assignee | ||
Comment 1•7 years ago
|
||
We need to use a couple of constructors explicitly instead of implicitly in DrawTargetRecording/DrawTargetWrapAndRecord because the otherwise they end up using the private generic one.
Assignee: nobody → jmuizelaar
Attachment #8882469 -
Flags: review?(lsalzman)
Updated•7 years ago
|
Attachment #8882469 -
Flags: review?(lsalzman) → review+
Assignee | ||
Comment 2•7 years ago
|
||
Attachment #8883445 -
Flags: review?(lsalzman)
Assignee | ||
Comment 3•7 years ago
|
||
Attachment #8883446 -
Flags: review?(lsalzman)
Updated•7 years ago
|
Attachment #8883445 -
Flags: review?(lsalzman) → review+
Updated•7 years ago
|
Attachment #8883446 -
Flags: review?(lsalzman) → review+
Pushed by jmuizelaar@mozilla.com: https://hg.mozilla.org/integration/mozilla-inbound/rev/53a2b6e02029 Make the constructors to the recording events generic. r=lsalzman https://hg.mozilla.org/integration/mozilla-inbound/rev/700d4236893e Make LoadEventFromStream generic. r=lsalzman https://hg.mozilla.org/integration/mozilla-inbound/rev/b811c588cfe2 Make InlineTranslator::TranslateRecording() take data instead of a stream. r=lsalzman
Comment 5•7 years ago
|
||
bugherder |
https://hg.mozilla.org/mozilla-central/rev/53a2b6e02029 https://hg.mozilla.org/mozilla-central/rev/700d4236893e https://hg.mozilla.org/mozilla-central/rev/b811c588cfe2
Status: NEW → RESOLVED
Closed: 7 years ago
status-firefox56:
--- → fixed
Resolution: --- → FIXED
Target Milestone: --- → mozilla56
You need to log in
before you can comment on or make changes to this bug.
Description
•