Closed Bug 882455 Opened 11 years ago Closed 11 years ago

handleEvent no longer implemented in Gecko 18.

Categories

(Marketplace Graveyard :: Validation, defect)

Product:
Marketplace Graveyard
Component:
Validation
Platform:
x86
macOS
Type:
defect
Priority:
Not set
Severity:
normal

Tracking

(Not tracked)

Status:
RESOLVED FIXED
Milestone:
2013-06-13

People

(Reporter: lstowasser, Assigned: basta)

Details

When uploading a packaged app in a ZIP to the App Validator I'm getting a ton of errors saying:

`handleEvent` no longer implemented in Gecko 18.
Error: As of Gecko 18, objects with `handleEvent` methods may no longer be assigned to `on*` properties. Doing so will be equivalent to assigning `null` to the property.

Is this a recent change? It seems strange that developers can't use `onclick` or the like for packaged apps.
Fixed and pushed yesterday.
Assignee: nobody → mattbasta
Status: NEW → RESOLVED
Closed: 11 years ago
Resolution: --- → FIXED
Target Milestone: --- → 2013-06-13
A more detailed explanation:

The error being shown is a remnant of the amo-validator. The code wasn't originally deleted because it contains the logic to flag CSP violations. For instance, the following code will generate a warning:

document.body.onload = "foo()";

The default CSP for privileged packaged apps will (should) cause that code to fail.
You need to log in before you can comment on or make changes to this bug.