Open Bug 1444836 Opened 6 years ago Updated 2 years ago

Clarify DAMP head.js responsibilities

Categories

(DevTools :: General, enhancement, P3)

59 Branch
enhancement

Tracking

(Not tracked)

People

(Reporter: jdescottes, Unassigned)

References

Details

From comment on Bug 1441703

> It looks weird to have this module be only wrappers around damp.
> 
>    we could move reloadPage implementation within head.js. Same for
>      garbageCollect, but it is used from damp.js...
>    I'm wondering if we should only use modules and also make damp.js 
>      be a module?
> 
>    but runTest, testSetup, testTeardown and logTestResult are all specific
>    to Damp, specific to the test runner. So it makes sense to delegate to 
>    Damp and keep Damp object to focus on the test runner itself.
> 

The goal here is to make the split between damp.js and head.js clearer. For now, some of the methods that have been left in damp.js are there for technical reasons (reloadPage would not work when loaded as a module?).

The test runner part could however be a different module.
Product: Firefox → DevTools
Severity: normal → S3
You need to log in before you can comment on or make changes to this bug.