Skip to content

Commit

Permalink
FBTest: fix script/stepping/4839 test
Browse files Browse the repository at this point in the history
  • Loading branch information
@janodvarko
janodvarko committed Feb 10, 2014
1 parent 0a96a1e commit 7a77df2
Showing 1 changed file with 2 additions and 1 deletion.
3 changes: 2 additions & 1 deletion tests/content/script/stepping/4839/issue4839.js
View file
Expand Up @@ -25,7 +25,8 @@ function runTest()

function createBreakpoint(callback, lineNo)
{
FBTest.setBreakpoint(null, "issue4839.html", lineNo, null, callback);
FBTest.setBreakpoint(null, basePath + "script/stepping/4839/issue4839.html",
lineNo, null, callback);
}

function refreshPage(callback, win)
Expand Down

2 comments on commit 7a77df2

@simonlindholm
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Just curious, why is this change necessary?

@janodvarko
Copy link
Member Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

The URL passed in setBreakpoint must be absolute (not just a file name) since it's consequently used by BreakpointStore that operates with absolute URLs. Perhaps we could simplify and compute the absolute URL in case a file name is used...
Honza

Please sign in to comment.