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

Unified Diff: LayoutTests/inspector/sources/debugger/network-uisourcecode-provider.html

Issue 1238103002: [DevTools] Do not report edited resources via Page.getResourceContent. (Closed) Base URL: svn://svn.chromium.org/blink/trunk
Patch Set: Nuked concatenated, fixed comments Created 5 years, 4 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
Index: LayoutTests/inspector/sources/debugger/network-uisourcecode-provider.html
diff --git a/LayoutTests/inspector/sources/debugger/network-uisourcecode-provider.html b/LayoutTests/inspector/sources/debugger/network-uisourcecode-provider.html
index 2245fff335270b5cdcd7ef9f0c7f2d6fae740014..eeb2e87ffef8882d65e87d901a6edae1276c6669 100644
--- a/LayoutTests/inspector/sources/debugger/network-uisourcecode-provider.html
+++ b/LayoutTests/inspector/sources/debugger/network-uisourcecode-provider.html
@@ -87,24 +87,13 @@ function test()
}
},
- function testScriptResource(next)
+ function testScriptResourceAndVMScript(next)
{
createNetworkUISourceCodeProvider();
- InspectorTest.addResult("Creating resource.");
InspectorTest.waitForWorkspaceUISourceCodeAddedEvent(uiSourceCodeAdded);
+ InspectorTest.addResult("Creating script resource.");
createResourceMock(WebInspector.resourceTypes.Script, "<script resource content>");
-
- function uiSourceCodeAdded(uiSourceCode)
- {
- setTimeout(function() { InspectorTest.dumpUISourceCode(uiSourceCode, next); });
- }
- },
-
- function testVMScript(next)
- {
- createNetworkUISourceCodeProvider();
InspectorTest.addResult("Creating script.");
- InspectorTest.waitForWorkspaceUISourceCodeAddedEvent(uiSourceCodeAdded);
createScriptMock("<script content>");
function uiSourceCodeAdded(uiSourceCode)
@@ -113,19 +102,6 @@ function test()
}
},
- function testStylesheetResource(next)
pfeldman 2015/08/12 23:38:42 Why is this removed?
dgozman 2015/08/13 18:27:26 Because adding stylesheet resource does not affect
pfeldman 2015/08/13 19:22:33 Does this mean that we no longer show stylesheets
- {
- createNetworkUISourceCodeProvider();
- InspectorTest.addResult("Creating resource.");
- InspectorTest.waitForWorkspaceUISourceCodeAddedEvent(uiSourceCodeAdded);
- createResourceMock(WebInspector.resourceTypes.Stylesheet, "<stylesheet resource content>");
-
- function uiSourceCodeAdded(uiSourceCode)
- {
- setTimeout(function() { InspectorTest.dumpUISourceCode(uiSourceCode, next); });
- }
- },
-
function testRemoveStyleSheetFromModelWithComplexURL(next)
{
var mockStyleSheetHeader = createMockStyleSheetHeader("http://example.com/foo.css");
@@ -143,6 +119,9 @@ function test()
{
createNetworkUISourceCodeProvider();
InspectorTest.waitForWorkspaceUISourceCodeAddedEvent(uiSourceCodeAdded);
+ InspectorTest.addResult("Creating stylesheet resource.");
+ createResourceMock(WebInspector.resourceTypes.Stylesheet, "<stylesheet resource content>");
+
WebInspector.CSSStyleModel.fromTarget(target)._styleSheetAdded(mockStyleSheetHeader);
function uiSourceCodeAdded(uiSourceCode)

Powered by Google App Engine
This is Rietveld 408576698