1471215 - Remove now unnecessary classInfo definition in PlacesTreeView

Status: RESOLVED FIXED

(Firefox :: Bookmarks & History, enhancement, P1)

Description

[Mark Banner (:standard8)]

It seems bug 761494 added classInfo to PlacesTreeView for the purposes of avoiding issues with add-ons.

As far as I can tell, this is no longer needed - we do the right accesses in the right place. All the tests seem to be passing (though I'll push to try once it reopens).

As it is near the start of the cycle, I think we should try and remove it now, and see if we hit any regressions from wider testing.

Comment 1

[Phabricator Automation]

Attachment: Bug 1471215 - Remove now unnecessary classInfo definition in PlacesTreeView.

MozReview-Commit-ID: 2BJGohgrA2l

Comment 2

[Phabricator Automation]

Comment on attachment 8987821 [details]
Bug 1471215 - Remove now unnecessary classInfo definition in PlacesTreeView.

Marco Bonardo [::mak] has approved the revision.

https://phabricator.services.mozilla.com/D1828

Comment 3

[Mark Banner (:standard8)]

Try push: https://treeherder.mozilla.org/#/jobs?repo=try&revision=cd7a4664ff1f80d9d2bf7186796e9d3b47dd3cf2

Comment 4

[Pulsebot]

Pushed by [e-mail]:
https://hg.mozilla.org/integration/autoland/rev/52125f300616
Remove now unnecessary classInfo definition in PlacesTreeView. r=mak

Comment 5

[Dorel Luca [:dluca]]

https://hg.mozilla.org/mozilla-central/rev/52125f300616