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

Unified Diff: LayoutTests/inspector/tracing/timeline-paint-with-style-recalc-invalidations.html

Issue 674963003: Clean up the individual invalidation tests to be self contained. (Closed) Base URL: svn://svn.chromium.org/blink/trunk
Patch Set: Created 6 years, 2 months ago
Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.
Jump to:
View side-by-side diff with in-line comments
Download patch
« no previous file with comments | « LayoutTests/inspector/tracing/timeline-paint-with-layout-invalidations-on-deleted-node.html ('k') | no next file » | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: LayoutTests/inspector/tracing/timeline-paint-with-style-recalc-invalidations.html
diff --git a/LayoutTests/inspector/tracing/timeline-paint-with-style-recalc-invalidations.html b/LayoutTests/inspector/tracing/timeline-paint-with-style-recalc-invalidations.html
index 9405a670ebd180bc3b90f13167ad12b59eb47929..e3ec16bbacc0e152b4e3f3a49f038b83e523e1ca 100644
--- a/LayoutTests/inspector/tracing/timeline-paint-with-style-recalc-invalidations.html
+++ b/LayoutTests/inspector/tracing/timeline-paint-with-style-recalc-invalidations.html
@@ -42,25 +42,29 @@ function test()
InspectorTest.assertEquals(invalidations[1].type, WebInspector.TracingTimelineModel.RecordType.StyleRecalcInvalidationTracking);
InspectorTest.assertEquals(invalidations[1].nodeName, "DIV id='testElement'");
InspectorTest.assertEquals(invalidations[1].reason, "StyleSheetChange");
- InspectorTest.invokeAsyncWithTimeline("updateSubframeAndDisplay", next);
+
+ next();
});
},
function testSubframe(next)
{
- // The first paint corresponds to the local frame and should have no invalidations.
- var firstPaintRecord = InspectorTest.findFirstTimelineRecord(WebInspector.TimelineModel.RecordType.Paint);
- var firstInvalidations = firstPaintRecord._event.invalidationTrackingEvents;
- InspectorTest.assertEquals(firstInvalidations, undefined);
+ InspectorTest.invokeAsyncWithTimeline("updateSubframeAndDisplay", function() {
+ // The first paint corresponds to the local frame and should have no invalidations.
+ var firstPaintRecord = InspectorTest.findFirstTimelineRecord(WebInspector.TimelineModel.RecordType.Paint);
+ var firstInvalidations = firstPaintRecord._event.invalidationTrackingEvents;
+ InspectorTest.assertEquals(firstInvalidations, undefined);
+
+ // The second paint corresponds to the subframe and should have our style invalidations.
+ var secondPaintRecord = InspectorTest.findTimelineRecord(WebInspector.TimelineModel.RecordType.Paint, 1);
+ var secondInvalidations = secondPaintRecord._event.invalidationTrackingEvents;
+ InspectorTest.assertEquals(secondInvalidations.length, 1);
+ InspectorTest.assertEquals(secondInvalidations[0].type, WebInspector.TracingTimelineModel.RecordType.StyleRecalcInvalidationTracking);
+ InspectorTest.assertEquals(secondInvalidations[0].nodeName, "DIV");
+ InspectorTest.assertEquals(secondInvalidations[0].reason, "StyleSheetChange");
- // The second paint corresponds to the subframe and should have our style invalidations.
- var secondPaintRecord = InspectorTest.findTimelineRecord(WebInspector.TimelineModel.RecordType.Paint, 1);
- var secondInvalidations = secondPaintRecord._event.invalidationTrackingEvents;
- InspectorTest.assertEquals(secondInvalidations.length, 1);
- InspectorTest.assertEquals(secondInvalidations[0].type, WebInspector.TracingTimelineModel.RecordType.StyleRecalcInvalidationTracking);
- InspectorTest.assertEquals(secondInvalidations[0].nodeName, "DIV");
- InspectorTest.assertEquals(secondInvalidations[0].reason, "StyleSheetChange");
- next();
+ next();
+ });
}
]);
}
« no previous file with comments | « LayoutTests/inspector/tracing/timeline-paint-with-layout-invalidations-on-deleted-node.html ('k') | no next file » | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698