Closed Bug 526635 Opened 10 years ago Closed 10 years ago

Interfaces changed on the 1.9.2 branch with no IID rev

Categories

(Core :: General, defect)

Product:
Core
Component:
General
Type:
defect
Priority:
Not set

Tracking

()

Status:
RESOLVED FIXED
Tracking Flags:
Tracking Status
status1.9.2 --- beta2-fixed

People

(Reporter: mossop, Assigned: mossop)

Details

(Keywords: dev-doc-complete)

Attachments

(9 files)

rev for bug 500889 [1.9.2 branch only]
700 bytes, patch
jaas
: review+
Details | Diff | Splinter Review
rev for bug 463088
822 bytes, patch
enndeakin
: review+
Details | Diff | Splinter Review
rev for bug 515354
687 bytes, patch
vlad
: review+
Details | Diff | Splinter Review
rev for bug 489864
1.64 KB, patch
jaas
: review+
Details | Diff | Splinter Review
rev for bug 484033
661 bytes, patch
ted
: review+
Details | Diff | Splinter Review
rev for bug 510110
940 bytes, patch
roc
: review+
Details | Diff | Splinter Review
rev for bug 498130
1.03 KB, patch
mak
: review+
Details | Diff | Splinter Review
rev for bug 503025 [1.9.2 branch only]
629 bytes, patch
jaas
: review+
Details | Diff | Splinter Review
combined branch patch
7.00 KB, patch
mossop
: review+
shaver
: approval1.9.2+
Details | Diff | Splinter Review 
Going to post a bunch of patchs to track the issues found by bug 477166
Flags: blocking1.9.2?
Bug 489864 revved the IID for the wrong interface. This undoes that rev and updates the correct one.
Attachment #410381 - Attachment description: rev for bug 503025 → rev for bug 503025 [1.9.2 branch only]
Attachment #410367 - Attachment description: rev for bug 500889 → rev for bug 500889 [1.9.2 branch only]
Attachment #410367 - Flags: review?(joshmoz)
Attachment #410371 - Flags: review?(vladimir)
Attachment #410373 - Flags: review?(joshmoz)
Attachment #410375 - Flags: review?(ted.mielczarek)
Attachment #410376 - Flags: review?(roc)
Attachment #410378 - Flags: review?(mak77)
Attachment #410381 - Flags: review?(joshmoz)
Attachment #410369 - Flags: review?(enndeakin)
Comment on attachment 410376 [details] [diff] [review]
rev for bug 510110

OK

We probably don't want to change IIDs on some of the other interfaces, now that it's post-beta, but this one should be OK.
Attachment #410376 - Flags: review?(roc) → review+
Attachment #410375 - Flags: review?(ted.mielczarek) → review+
Comment on attachment 410375 [details] [diff] [review]
rev for bug 484033

Crap, did I forget to rev that? Sorry!
Attachment #410381 - Flags: review?(joshmoz) → review+
Attachment #410373 - Flags: review?(joshmoz) → review+
Attachment #410367 - Flags: review?(joshmoz) → review+
Comment on attachment 410369 [details] [diff] [review]
rev for bug 463088

I'm assuming this change will be made on trunk too.
Attachment #410369 - Flags: review?(enndeakin) → review+
Comment on attachment 410378 [details] [diff] [review]
rev for bug 498130

Thank you for catching this, i also assume this will also land on trunk.
Attachment #410378 - Flags: review?(mak77) → review+
Landed the patches that are also applicable to trunk there: http://hg.mozilla.org/mozilla-central/rev/2fccec04a391
This is the combined branch patch that we should land as soon as possible.
Attachment #410835 - Flags: review+
Attachment #410835 - Flags: approval1.9.2?
Comment on attachment 410835 [details] [diff] [review]
combined branch patch

If we must...

Can you give the list of interfaces to Nick and co, so they can inform add-on devs as needed?
Attachment #410835 - Flags: approval1.9.2? → approval1.9.2+
Status: NEW → RESOLVED
Closed: 10 years ago
Resolution: --- → FIXED
Added a temporary section to the Firefox 3.6 for developers page; it'll be removed before Fx 3.6 goes final.

https://developer.mozilla.org/en/Firefox_3.6_for_developers#IID_changes_between_beta_1_and_beta_2
Keywords: dev-doc-needed → dev-doc-complete
You need to log in before you can comment on or make changes to this bug.