Move remainders of webshell into /docshell/base

RESOLVED FIXED in mozilla5

Status

()

--
trivial
RESOLVED FIXED
8 years ago
8 years ago

People

(Reporter: Ms2ger, Assigned: Ms2ger)

Tracking

Trunk
mozilla5
Points:
---
Bug Flags:
in-testsuite -

Firefox Tracking Flags

(Not tracked)

Details

Attachments

(1 attachment, 1 obsolete attachment)

(Assignee)

Description

8 years ago
Created attachment 450858 [details] [diff] [review]
Patch v1

See bug 490957 comment 1. I got a little carried away by whitespace cleanup; I hope you don't mind.
Attachment #450858 - Flags: review?(benjamin)

Comment 1

8 years ago
Comment on attachment 450858 [details] [diff] [review]
Patch v1

ok with me, but bz should really confirm
Attachment #450858 - Flags: review?(benjamin) → review?(bzbarsky)
Comment on attachment 450858 [details] [diff] [review]
Patch v1

Sure thing.
Attachment #450858 - Flags: review?(bzbarsky) → review+
(Assignee)

Comment 3

8 years ago
Comment on attachment 450858 [details] [diff] [review]
Patch v1

Thanks!
Attachment #450858 - Flags: approval2.0?

Comment 4

8 years ago
Comment on attachment 450858 [details] [diff] [review]
Patch v1

I don't think we want to take this until after branch.
Attachment #450858 - Flags: approval2.0? → approval2.0-
(Assignee)

Updated

8 years ago
Whiteboard: [needs landing]
(Assignee)

Updated

8 years ago
Depends on: 610267
Comment on attachment 450858 [details] [diff] [review]
Patch v1

>diff --git a/browser/installer/removed-files.in b/browser/installer/removed-files.in
>--- a/browser/installer/removed-files.in
>+++ b/browser/installer/removed-files.in
>@@ -52,16 +52,17 @@ components/nsCloseAllWindows.js
> components/nsCloseAllWindows.js
> components/nsDictionary.js
> components/nsExtensionManager.js
> components/nsInterfaceInfoToIDL.js
> components/nsScriptableIO.js
> components/nsUrlClassifierTable.js
> components/nsXmlRpcClient.js
> components/sidebar.xpt
>+components/webshell_idls.xpt
> components/xmlextras.xpt
> components/xpcom.xpt
> components/xpti.dat
> components/xptitemp.dat
> defaults/pref/all.js
> defaults/pref/bug259708.js
> defaults/pref/bug307259.js
> defaults/pref/security-prefs.js

Adding this to removed-files.in is unnecessary, we don't actually ship this file.
(Assignee)

Updated

8 years ago
Keywords: checkin-needed
Whiteboard: [needs landing]
(Assignee)

Comment 6

8 years ago
Created attachment 521546 [details] [diff] [review]
Patch for checkin
Attachment #450858 - Attachment is obsolete: true
http://hg.mozilla.org/projects/cedar/rev/9267d80c7c63
Flags: in-testsuite-
Whiteboard: fixed-in-cedar

Updated

8 years ago
Keywords: checkin-needed

Comment 8

8 years ago
http://hg.mozilla.org/mozilla-central/rev/9267d80c7c63
Status: ASSIGNED → RESOLVED
Last Resolved: 8 years ago
No longer depends on: 610267
Resolution: --- → FIXED
Whiteboard: fixed-in-cedar
Target Milestone: --- → mozilla2.2
You need to log in before you can comment on or make changes to this bug.