Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(75)

Issue 16896007: Timeline: add support for deferred canvas rasterization (Closed)

Created:
7 years, 6 months ago by caseq
Modified:
7 years, 2 months ago
Reviewers:
pfeldman, yurys
CC:
blink-reviews, jamesr, caseq+blink_chromium.org, loislo+blink_chromium.org, eustas+blink_chromium.org, alph+blink_chromium.org, eae+blinkwatch, yurys+blink_chromium.org, lushnikov+blink_chromium.org, abarth-chromium, vsevik+blink_chromium.org, pfeldman+blink_chromium.org, Rik, apavlov+blink_chromium.org, danakj, Stephen Chennney, aandrey+blink_chromium.org, jeez, pdr., nduca
Visibility:
Public.

Description

Timeline: add support for deferred canvas rasterization - add PlatformInsutrmentation::{will,did}RasterizeCanvas() - report the above events from Canvas2DLayerBridge::prepareMailbox() - add TimelineAgent::{will,did}UpdateLayer, plumb it through WebDevToolsAgent; - emit Rasterize event when canvas is being rasterized during update of a layer that belongs to the inspected page. Related chromium change: https://codereview.chromium.org/16848010 BUG=244505

Patch Set 1 #

Unified diffs Side-by-side diffs Delta from patch set Stats (+94 lines, -7 lines) Patch
M Source/WebKit/chromium/src/WebDevToolsAgentImpl.h View 1 chunk +2 lines, -0 lines 0 comments Download
M Source/WebKit/chromium/src/WebDevToolsAgentImpl.cpp View 1 chunk +12 lines, -0 lines 0 comments Download
M Source/core/inspector/InspectorController.h View 1 chunk +2 lines, -0 lines 0 comments Download
M Source/core/inspector/InspectorController.cpp View 1 chunk +12 lines, -0 lines 0 comments Download
M Source/core/inspector/InspectorTimelineAgent.h View 3 chunks +7 lines, -0 lines 0 comments Download
M Source/core/inspector/InspectorTimelineAgent.cpp View 4 chunks +38 lines, -7 lines 0 comments Download
M Source/core/platform/PlatformInstrumentation.h View 3 chunks +16 lines, -0 lines 0 comments Download
M Source/core/platform/graphics/chromium/Canvas2DLayerBridge.cpp View 3 chunks +3 lines, -0 lines 0 comments Download
M public/webpage/WebDevToolsAgent.h View 1 chunk +2 lines, -0 lines 0 comments Download

Messages

Total messages: 4 (0 generated)
caseq
7 years, 6 months ago (2013-06-13 13:50:56 UTC) #1
pfeldman
lgtm. A test?
7 years, 6 months ago (2013-06-13 17:30:26 UTC) #2
pfeldman
What is the status of this issue?
7 years, 3 months ago (2013-08-30 09:11:13 UTC) #3
pfeldman
7 years, 2 months ago (2013-10-24 13:51:38 UTC) #4
This is not moving, removing myself for reviewers.

Powered by Google App Engine
This is Rietveld 408576698