Closed Bug 860159 Opened 12 years ago Closed 3 years ago

Check for any missing IID changes with changed interfaces for Fx22(Aurora)

Categories

(Core :: General, defect)

x86
macOS
defect
Not set
normal

Tracking

()

RESOLVED WORKSFORME
Tracking Status
firefox22 + fixed

People

(Reporter: bajaj, Unassigned)

References

Details

Attachments

(1 file)

We need to find missing IID changes for : a) Before the merge happened when Fx 22 was still on nightly(m-c) b) Check for any missing IID changes when Fx22 was on aurora Scott, last time I tried steps in https://bugzilla.mozilla.org/show_bug.cgi?id=851302#c1 was running into issues. Can you please help identify the interfaces here and we can do the needed co-ordination to create and follow-up on Bugs that need's IID revved.
Scott, a quick check if the automation is ready yet else we may need your help again to find the interfaces and I or akeybl can file bugs do the out reach to make sure the needed IID's are revved.
Assignee: nobody → sjohnson
(In reply to bhavana bajaj [:bajaj] from comment #1) > Scott, a quick check if the automation is ready yet else we may need your > help again to find the interfaces and I or akeybl can file bugs do the out > reach to make sure the needed IID's are revved. Hi Bhavana: No, sorry, the automated tool is pretty much in the same state as for the last release. I really thought I would have time to work on it this cycle, but, alas, I have not.
in terms of timeline,our merge day for next release is May13 and we should be getting any issues out of this resolved by then
(In reply to bhavana bajaj [:bajaj] from comment #3) > in terms of timeline,our merge day for next release is May13 and we should > be getting any issues out of this resolved by then Bhavana: I'm not sure I understood that last comment. I doubt that the automated tool will be ready by that date. I can, however, assist you in finding the IIDs that may have changed in a given release.
(In reply to Scott Johnson (:jwir3) from comment #2) > (In reply to bhavana bajaj [:bajaj] from comment #1) > > Scott, a quick check if the automation is ready yet else we may need your > > help again to find the interfaces and I or akeybl can file bugs do the out > > reach to make sure the needed IID's are revved. > > Hi Bhavana: > > No, sorry, the automated tool is pretty much in the same state as for the > last release. I really thought I would have time to work on it this cycle, > but, alas, I have not. Thanks for the update Scott..hrm, looks like we'll have to do the semi-automated mechanism here to make sure nothing accidentally slips through and come-out as top-crashers :-/ Thanks in advance for your help :)
I spoke with Bhavana last week about checking IIDs for binary compatibility for Firefox 22 (currently aurora, soon to be beta). The changes I found made to .idl files are attached in a diff file. I believe the following interfaces will need to have IIDs revved: nsIB2GKeyboard nsIWebSocketChannel nsINavHistoryResultNode IUIABridge IUIAElement Note that, at least the first of these is B2G-only, so probably isn't a priority. The others I don't know about. I am still finishing up quirks with an automated way of checking this, but it's better than it was. If you would like me to demonstrate it to you (it comes up with a few false positives, but, to my knowledge, it errs on the side of detecting an IID change that probably isn't necessary - i.e. the "safe" side), I would be happy to do so.
* nsIB2GKeyboard caused by bug 844716 ** don't plan to file - no binary compatibility issues here * nsIWebSocketChannel caused by bug 849364 ** Filed bug 870579 * nsINavHistoryResultNode caused by bug 561450 ** Filed bug 870581 * IUIABridge & IUIAElement caused by bug 843456
Depends on: 870579, 870581, 870583
Blocks: 870585
(In reply to Alex Keybl [:akeybl] from comment #8) > * nsIB2GKeyboard caused by bug 844716 > ** don't plan to file - no binary compatibility issues here I would probably still file a bug. At some point, we are going to have binary compatibility issues with B2G, and it's probably better to get in the habit of revving IIDs every time the interface changes, anyway.
No longer depends on: 870581

The bug assignee didn't login in Bugzilla in the last 7 months, so the assignee is being reset.

Assignee: jaywir3 → nobody

It looks like this was an old meta bug that has served its purpose.

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

Attachment

General

Created:
Updated:
Size: