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

Unified Diff: LayoutTests/http/tests/inspector/appcache/appcache-test.js

Issue 268293003: DevTools: Get rid of WebInspector.panels (Closed) Base URL: svn://svn.chromium.org/blink/trunk
Patch Set: Created 6 years, 7 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/http/tests/inspector/appcache/appcache-test.js
diff --git a/LayoutTests/http/tests/inspector/appcache/appcache-test.js b/LayoutTests/http/tests/inspector/appcache/appcache-test.js
index 36e5577e79e01b0464b45a60f764df8f1a6f4169..72ad18003d9bcefa5d35fdc48d6723b9befaa7f0 100644
--- a/LayoutTests/http/tests/inspector/appcache/appcache-test.js
+++ b/LayoutTests/http/tests/inspector/appcache/appcache-test.js
@@ -81,7 +81,7 @@ InspectorTest.dumpApplicationCache = function()
InspectorTest.dumpApplicationCacheTree = function()
{
InspectorTest.addResult("Dumping application cache tree:");
- var applicationCacheTreeElement = WebInspector.panels.resources.applicationCacheListTreeElement;
+ var applicationCacheTreeElement = WebInspector.inspectorView.panel("resources").applicationCacheListTreeElement;
pfeldman 2014/05/06 19:57:14 I don't see how this is better. Lets keep it as is
apavlov 2014/05/07 10:26:13 There doesn't seem to be any point in keeping WebI
if (!applicationCacheTreeElement.children.length) {
InspectorTest.addResult(" (empty)");
return;
@@ -127,7 +127,7 @@ InspectorTest.applicationCacheStatusToString = function(status)
InspectorTest.dumpApplicationCacheModel = function()
{
InspectorTest.addResult("Dumping application cache model:");
- var model = WebInspector.panels.resources._applicationCacheModel;
+ var model = WebInspector.inspectorView.panel("resources")._applicationCacheModel;
var frameIds = [];
for (var frameId in model._manifestURLsByFrame)
@@ -154,8 +154,8 @@ InspectorTest.dumpApplicationCacheModel = function()
InspectorTest.waitForFrameManifestURLAndStatus = function(frameId, manifestURL, status, callback)
{
- var frameManifestStatus = WebInspector.panels.resources._applicationCacheModel.frameManifestStatus(frameId);
- var frameManifestURL = WebInspector.panels.resources._applicationCacheModel.frameManifestURL(frameId);
+ var frameManifestStatus = WebInspector.inspectorView.panel("resources")._applicationCacheModel.frameManifestStatus(frameId);
+ var frameManifestURL = WebInspector.inspectorView.panel("resources")._applicationCacheModel.frameManifestURL(frameId);
if (frameManifestStatus === status && frameManifestURL.indexOf(manifestURL) !== -1) {
callback();
return;

Powered by Google App Engine
This is Rietveld 408576698