Closed Bug 764546 Opened 12 years ago Closed 12 years ago

Temporary fix for: Hotmail no longer auto-updates (due to "globalStorage" removal)

Categories

(Core :: DOM: Core & HTML, defect)

13 Branch
defect
Not set
major

Tracking

()

VERIFIED FIXED
mozilla13
Tracking Status
firefox13 + verified
firefox14 - unaffected
firefox15 - unaffected

People

(Reporter: mayhemer, Assigned: mayhemer)

References

Details

(Keywords: regression)

Attachments

(3 files, 3 obsolete files)

Attached patch v1 (obsolete) — Splinter Review
+++ This bug was initially created as a clone of Bug #736731 +++

User Agent: Mozilla/5.0 (Windows NT 6.1; rv:14.0) Gecko/20120316 Firefox/14.0a1
Build ID: 20120316031151

Steps to reproduce:

Logged into my Hotmail account. Hotmail no longer automatically updates the list of emails and count in title bar. Does so in IE.



Expected results:

The web page should have checked for updates.
Attachment #632843 - Flags: review?(jst)
Attachment #632843 - Flags: approval-mozilla-release?
Comment on attachment 632843 [details] [diff] [review]
v1

- In dom/interfaces/base/nsIDOMWindow.idl:

@@ -500,16 +500,21 @@ interface nsIDOMWindow : nsISupports
   readonly attribute long long mozAnimationStartTime;
 
   /**
    * @see <http://dev.w3.org/2006/webapi/FileAPI/#creating-revoking>
    */
   readonly attribute nsIDOMMozURLProperty URL;
 
   /**
+   * Global storage, accessible by domain.
+   */
+  readonly attribute nsIDOMStorageList globalStorage;

Given that we're taking this on the release branch we'll need to create a new interface, i.e. something like nsIDOMWindow_globalStorage (in the same idl file is fine), inheriting only from nsISupports, and make nsGlobalWindow implement that interface (i.e. inherit from it, and add it to the QI map in nsGlobalWindow.cpp), and then add that interface to DOM_CLASSINFO_WINDOW_MAP_ENTRIES in nsDOMClassInfo.cpp. That way we'll be binary compatible with what we've already released.

- In dom/interfaces/storage/nsPIDOMStorage.h

We should at the very least update the IID for this interface. If we do that, I'd be comfortable shipping this, even if we're technically changing an interface here as well. The alternative is to create a new interface here, but that means we'd need to change the value of the enums to something it's never been before, and I think that scares me more than shipping a changed IID to this interface.

r=jst with that.
Attachment #632843 - Flags: review?(jst) → review+
Attachment #632843 - Flags: approval-mozilla-release? → approval-mozilla-release+
Attached patch INTERDIFF v1 -> v1.1 (obsolete) — Splinter Review
- according the review comments
- not sure whether to change iid of nsIDOMWindow now...
Attachment #632971 - Flags: review?(jst)
Attached patch v1.1 (obsolete) — Splinter Review
Comment on attachment 632971 [details] [diff] [review]
INTERDIFF v1 -> v1.1

We should leave the nsIDOMWindow IID *unchanged*, so please revert that, and this is good to go!
Attachment #632971 - Flags: review?(jst) → review+
Status: ASSIGNED → RESOLVED
Closed: 12 years ago
Resolution: --- → FIXED
Target Milestone: --- → mozilla13
Looks all green on tbpl. Marking fixed for 13. Thanks Honza for the very quick turnaround!
Verified fixed in Firefox 13.0.1.
Status: RESOLVED → VERIFIED
Whiteboard: [qa+:ashughes]
Editing summary to avoid confusion.
Summary: Hotmail no longer auto-updates (due to "globalStorage" removal) → Temporary fix for: Hotmail no longer auto-updates (due to "globalStorage" removal)
Can we do the same to FF14? It sounds like MS may not have a fix for bug 736731 prior.
Attachment #635333 - Flags: review?(Ms2ger)
This is actually identical to v1.1 for the release channel, so no need for re-review.  Only change that needed to merge was the introduced patch from bug 762409.
Attachment #632843 - Attachment is obsolete: true
Attachment #632971 - Attachment is obsolete: true
Attachment #635334 - Flags: review+
Comment on attachment 635333 [details] [diff] [review]
revert patches from bug 732708 from beta

Review of attachment 635333 [details] [diff] [review]:
-----------------------------------------------------------------

Haven't looked carefully, but nothing obviously wrong.
Attachment #635333 - Flags: review?(Ms2ger) → review+
Attachment #635333 - Flags: approval-mozilla-beta?
Attachment #635334 - Flags: approval-mozilla-beta?
Comment on attachment 635333 [details] [diff] [review]
revert patches from bug 732708 from beta

This patch effectively backs changesets: 17696f172c01, a188746249cd, 0d871550085e out of m-b.
Attachment #635333 - Flags: approval-mozilla-beta? → approval-mozilla-beta+
Comment on attachment 635334 [details] [diff] [review]
v1.1 for the beta channel

Approving for landing, will add qawanted to this bug to check mozilla-beta tinderbox builds after this lands and confirm that 14 matches up with expecations before next Tuesday's go to build.
Attachment #635334 - Flags: approval-mozilla-beta? → approval-mozilla-beta+
errrr, string changes?
Isn't this fixed by Microsoft or not? (I'm confused)
see bug 736731 comment 37
Yes, I can confirm that we pushed a fix and validated on 13 and 14. I was assuming 14 did not have the patch from Mozilla that would mask the need for the change. 

Do we need to check on 15?
Inbox is auto-updating fine, but Junk folder isn't. Tested on FF 14b10 and Nightly 16.0a1 (2012-07-02) on Win 7/64.
Raul, can you please confirm Paul's results?  

globalStorage feature *is* present on Firefox 13.0.1 only.  globalStorage *is not* present on any newer version of Firefox.
I tested this by sending email from two different accounts to my hotmail.com account. One of the emails would always go into my inbox and the other would always go into the junk folder.

In addition I tested the MSN Messenger functionality online.

On Fx13.0 the inbox and junk folder auto updated and MNN Messenger worked.

On Fx13.0.1 the inbox and junk folder auto updated and MSN Messenger worked.

On Fx14.0b11 (latest) the inbox and junk folder auto updated and MSN Messenger worked.

On Fx15.0a2 (latest) neither the inbox nor the junk folder auto updated, but MSN Messenger worked.
(In reply to juan becerra [:juanb] from comment #23)
> On Fx14.0b11 (latest) the inbox and junk folder auto updated and MSN
> Messenger worked.
> 
> On Fx15.0a2 (latest) neither the inbox nor the junk folder auto updated, but
> MSN Messenger worked.

Honza - this is unexpected behavior, no? Given the positive testing with 14.0, we'll untrack there. But I'm moving the tracking flag to 15 to make sure we understand what's going on prior to release.
Yes, that is strange.  I'll create a hotmail account and try to reproduce.  It could be a completely different bug, though.
Could be an issue with our fix against 15. I'll investigate on our side.
Hi folks,

Just verified on Aurora 15.0a2 (2012-07-11) and no repro.
The message list in the Inbox correctly auto-updates.

Should I try with different build?
(In reply to Raul D @ Microsoft from comment #27)
> Hi folks,
> 
> Just verified on Aurora 15.0a2 (2012-07-11) and no repro.
> The message list in the Inbox correctly auto-updates.

Cool.  Thanks!  Why do you think you observed a failure before?  Could that be some random issue?

> 
> Should I try with different build?

If you have time now, try the nightly (Fx16) at http://ftp.mozilla.org/pub/mozilla.org/firefox/nightly/latest-mozilla-central/ just to be sure, but I expect it to work, there are no major differences from Fx15 according DOM storage.
(In reply to Honza Bambas (:mayhemer) from comment #28)
> (In reply to Raul D @ Microsoft from comment #27)
> > Hi folks,
> > 
> > Just verified on Aurora 15.0a2 (2012-07-11) and no repro.
> > The message list in the Inbox correctly auto-updates.
> 
> Cool.  Thanks!  Why do you think you observed a failure before?  Could that
> be some random issue?
> 
> > 
> > Should I try with different build?
> 
> If you have time now, try the nightly (Fx16) at
> http://ftp.mozilla.org/pub/mozilla.org/firefox/nightly/latest-mozilla-
> central/ just to be sure, but I expect it to work, there are no major
> differences from Fx15 according DOM storage.

Per comments 23, 25, I understood you still saw an issue on FF 15. I can't repro.
I verified no repro on Nightly FF 16 either.

Cheers
Per meeting discussion today, removing qawanted. Please re-add if there's something more QA can do here.
Keywords: qawanted
Whiteboard: [qa+:ashughes]
[Triage Comment]
Untracking for 15 as per comment 29.
Component: DOM → DOM: Core & HTML
You need to log in before you can comment on or make changes to this bug.