Closed
Bug 1028565
Opened 10 years ago
Closed 10 years ago
Flatten away public/src subdirectories under editor/
Categories
(Core :: DOM: Editor, defect)
Core
DOM: Editor
Tracking
()
RESOLVED
FIXED
mozilla33
People
(Reporter: poiru, Assigned: poiru)
Details
Attachments
(5 files)
19.45 KB,
patch
|
ehsan.akhgari
:
review+
|
Details | Diff | Splinter Review |
8.82 KB,
patch
|
ehsan.akhgari
:
review+
|
Details | Diff | Splinter Review |
5.30 KB,
patch
|
ehsan.akhgari
:
review+
|
Details | Diff | Splinter Review |
25.77 KB,
patch
|
ehsan.akhgari
:
review+
|
Details | Diff | Splinter Review |
6.06 KB,
patch
|
ehsan.akhgari
:
review+
|
Details | Diff | Splinter Review |
This is similar to e.g. bug 946065.
Assignee | ||
Comment 1•10 years ago
|
||
This would involve: - Flattening away public/src subdirectories under editor/, editor/composer/, editor/txmgr/, and editor/txtsvc/ - Flattening away base subdirectory under editor/libeditor/ I could also flatten away the idl/ subdirectories if needed. Would you be OK with this?
Flags: needinfo?(ehsan)
Comment 3•10 years ago
|
||
(FWIW flattening the idl dirs also sounds good, but it's cool if you don't want to do that work in this bug.)
Assignee | ||
Comment 4•10 years ago
|
||
Assignee | ||
Comment 5•10 years ago
|
||
Attachment #8444799 -
Flags: review?(ehsan)
Assignee | ||
Comment 6•10 years ago
|
||
Attachment #8444800 -
Flags: review?(ehsan)
Assignee | ||
Comment 7•10 years ago
|
||
Attachment #8444801 -
Flags: review?(ehsan)
Assignee | ||
Comment 8•10 years ago
|
||
Attachment #8444802 -
Flags: review?(ehsan)
Updated•10 years ago
|
Attachment #8444798 -
Flags: review?(ehsan) → review+
Updated•10 years ago
|
Attachment #8444799 -
Flags: review?(ehsan) → review+
Updated•10 years ago
|
Attachment #8444800 -
Flags: review?(ehsan) → review+
Updated•10 years ago
|
Attachment #8444801 -
Flags: review?(ehsan) → review+
Updated•10 years ago
|
Attachment #8444802 -
Flags: review?(ehsan) → review+
Assignee | ||
Comment 9•10 years ago
|
||
https://hg.mozilla.org/integration/mozilla-inbound/rev/e7557f70acfe https://hg.mozilla.org/integration/mozilla-inbound/rev/62ad3254903e https://hg.mozilla.org/integration/mozilla-inbound/rev/3510bcdba3fc https://hg.mozilla.org/integration/mozilla-inbound/rev/5e9777d38052 https://hg.mozilla.org/integration/mozilla-inbound/rev/fdd74c065e5d
Assignee | ||
Comment 10•10 years ago
|
||
... and earlier try push: https://tbpl.mozilla.org/?tree=Try&rev=22bead20b14f
Backed out in https://hg.mozilla.org/integration/mozilla-inbound/rev/71f6b8a9b2e8 along with the changes from bug 1028559 for Android mochitest bustage: https://tbpl.mozilla.org/php/getParsedLog.php?id=42398428&tree=Mozilla-Inbound
Flags: needinfo?(birunthan)
Assignee | ||
Comment 12•10 years ago
|
||
The first 3 parts are green on try (https://tbpl.mozilla.org/?tree=Try&rev=16321d89d46c) so relanding them: https://hg.mozilla.org/integration/mozilla-inbound/rev/015a4d8eb89f https://hg.mozilla.org/integration/mozilla-inbound/rev/1f9ddc6738f6 https://hg.mozilla.org/integration/mozilla-inbound/rev/852cf6f6780e
Flags: needinfo?(birunthan)
Assignee | ||
Updated•10 years ago
|
Keywords: leave-open
Comment 13•10 years ago
|
||
https://hg.mozilla.org/mozilla-central/rev/015a4d8eb89f https://hg.mozilla.org/mozilla-central/rev/1f9ddc6738f6 https://hg.mozilla.org/mozilla-central/rev/852cf6f6780e
Assignee | ||
Comment 14•10 years ago
|
||
https://hg.mozilla.org/integration/mozilla-inbound/rev/faecfa9d0bb6 Try push: https://tbpl.mozilla.org/?tree=Try&rev=84fc6f1ad59a Removing leave-open even though part 4 was never checked in (due to test failures caused by test reordering).
Keywords: leave-open
Comment 15•10 years ago
|
||
https://hg.mozilla.org/mozilla-central/rev/faecfa9d0bb6
Status: ASSIGNED → RESOLVED
Closed: 10 years ago
Resolution: --- → FIXED
Target Milestone: --- → mozilla33
You need to log in
before you can comment on or make changes to this bug.
Description
•