Closed Bug 1812367 Opened 2 years ago Closed 2 years ago

Rename devtools/shim to devtools/startup - SeaMonkey parts

Categories

(SeaMonkey :: General, defect)

defect

Tracking

(seamonkey2.53+ fixed)

RESOLVED FIXED
Tracking Status
seamonkey2.53 + fixed

People

(Reporter: iannbugzilla, Assigned: iannbugzilla)

References

Details

(Whiteboard: SM2.53.16)

Attachments

(1 file)

Some changes were missed for SeaMonkey in the cc patches attached to bug 1444926

+++ This bug was initially created as a clone of Bug #1444926 +++

The name devtools/shim was chosen for DevTools gofaster, but since the addon plans are on hold, we could rename this folder to make it less confusing.

Since the folder contains all the startup code that can run before devtools are started or initialized, I think devtools/startup would be a good fit.

We should also cleanup all references to the devtools addon, and remove them.

[Approval Request Comment]
Regression caused by (bug #): 1444296
User impact if declined: context menus don't work
Testing completed (on m-c, etc.): 2.53.16
Risk to taking this patch (and alternatives if risky): low
String changes made by this patch: none

Attachment #9314076 - Flags: review?(frgrahl)
Attachment #9314076 - Flags: approval-comm-release?
Attachment #9314076 - Flags: approval-comm-esr60?
See Also: → 1451847
Whiteboard: SM2.53.16

Comment on attachment 9314076 [details] [diff] [review]
1812367-devtools-25316.patch

LGTM r/a+

The filter change was part of Bug 1451847 already but can be pushed for 2.53 in the patch. Will remove it for central checkin.

Attachment #9314076 - Flags: review?(frgrahl)
Attachment #9314076 - Flags: review+
Attachment #9314076 - Flags: approval-comm-release?
Attachment #9314076 - Flags: approval-comm-release+
Attachment #9314076 - Flags: approval-comm-esr60?
Attachment #9314076 - Flags: approval-comm-esr60+

Pushed by frgrahl@gmx.net:
https://hg.mozilla.org/comm-central/rev/0b1ebb67ffb3
Rename devtools/shim to devtools/startup - SeaMonkey parts. r=frg

Status: ASSIGNED → RESOLVED
Closed: 2 years ago
Resolution: --- → FIXED
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Creator:
Created:
Updated:
Size: