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

Unified Diff: Source/devtools/front_end/Tests.js

Issue 396993003: DevTools: get rid of WebInspector.cssModel, use target models instead (Closed) Base URL: svn://svn.chromium.org/blink/trunk
Patch Set: Created 6 years, 5 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-test.js ('k') | Source/devtools/front_end/audits/AuditsPanel.js » ('j') | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: Source/devtools/front_end/Tests.js
diff --git a/Source/devtools/front_end/Tests.js b/Source/devtools/front_end/Tests.js
index 2d6759afd4e3e62bae47a65a7698d36c94731af3..97532c91bb04d0a647120ef4990a706caa66a41b 100644
--- a/Source/devtools/front_end/Tests.js
+++ b/Source/devtools/front_end/Tests.js
@@ -497,7 +497,7 @@ TestSuite.prototype.testNetworkTiming = function()
TestSuite.prototype.testConsoleOnNavigateBack = function()
{
- if (WebInspector.consoleModel.messages.length === 1)
+ if (WebInspector.consoleModel.messages().length === 1)
pfeldman 2014/07/16 19:57:58 I'll have to modify this since consoleModel is not
sergeyv 2014/07/16 20:50:28 Looks like it should fail since we do not have Web
firstConsoleMessageReceived.call(this);
else
WebInspector.consoleModel.addEventListener(WebInspector.ConsoleModel.Events.MessageAdded, firstConsoleMessageReceived, this);
@@ -509,7 +509,7 @@ TestSuite.prototype.testConsoleOnNavigateBack = function()
function didClickLink() {
// Check that there are no new messages(command is not a message).
- this.assertEquals(3, WebInspector.consoleModel.messages.length);
+ this.assertEquals(3, WebInspector.consoleModel.messages().length);
this.evaluateInConsole_("history.back();", didNavigateBack.bind(this));
}
@@ -520,7 +520,7 @@ TestSuite.prototype.testConsoleOnNavigateBack = function()
}
function didCompleteNavigation() {
- this.assertEquals(7, WebInspector.consoleModel.messages.length);
+ this.assertEquals(7, WebInspector.consoleModel.messages().length);
this.releaseControl();
}
@@ -597,7 +597,7 @@ TestSuite.prototype.testTimelineFrames = function()
TestSuite.prototype.enableTouchEmulation = function()
{
- WebInspector.targetManager.activeTarget().domModel.emulateTouchEventObjects(true);
+ WebInspector.targetManager.mainTarget().domModel.emulateTouchEventObjects(true);
};
// Regression test for http://webk.it/97466
@@ -765,7 +765,7 @@ TestSuite.prototype.stopTimeline = function()
TestSuite.prototype.waitForTestResultsInConsole = function()
{
- var messages = WebInspector.consoleModel.messages;
+ var messages = WebInspector.consoleModel.messages();
for (var i = 0; i < messages.length; ++i) {
var text = messages[i].messageText;
if (text === "PASS")
@@ -789,7 +789,7 @@ TestSuite.prototype.waitForTestResultsInConsole = function()
TestSuite.prototype.checkLogAndErrorMessages = function()
{
- var messages = WebInspector.consoleModel.messages;
+ var messages = WebInspector.consoleModel.messages();
var matchesCount = 0;
function validMessage(message)
@@ -878,7 +878,7 @@ TestSuite.prototype.evaluateInConsole_ = function(code, callback)
{
function innerEvaluate()
{
- WebInspector.consoleModel.show();
+ WebInspector.console.show();
var consoleView = WebInspector.ConsolePanel._view();
consoleView._prompt.text = code;
consoleView._promptElement.dispatchEvent(TestSuite.createKeyEvent("Enter"));
« no previous file with comments | « LayoutTests/inspector/tracing-test.js ('k') | Source/devtools/front_end/audits/AuditsPanel.js » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698