Closed Bug 1406193 Opened 3 years ago Closed 3 years ago

Remove last remaining SocialAPI bits

Categories

(Firefox Graveyard :: SocialAPI, enhancement)

50 Branch
enhancement
Not set
normal

Tracking

(firefox57 wontfix, firefox58 fixed)

RESOLVED FIXED
Firefox 58
Tracking Status
firefox57 --- wontfix
firefox58 --- fixed

People

(Reporter: mixedpuppy, Assigned: thegalaxiasgroup, Mentored)

References

Details

(Keywords: good-first-bug, Whiteboard: [tor 13612])

Attachments

(1 file, 4 obsolete files)

Some social.* prefs were missed during removal.
Affected files: http://searchfox.org/mozilla-central/search?q=%5Cbsocial%5C.&case=true&regexp=true&path=
Mentor: dao+bmo
Keywords: good-first-bug
Summary: remove lingering prefs → Remove last remaining SocialAPI bits
If the only task is to simply delete this lines i'm on it and going to put a patch on review soon.
Attachment #8916422 - Attachment is obsolete: true
Comment on attachment 8916423 [details] [diff] [review]
Remove last remaining SocialAPI bits ()

>-    // Messages sent from browser-social.js
>-    "Social:",
>-    "PageMetadata:",

We'll still need "PageMetadata:".

Looks good otherwise!
Assignee: nobody → thegalaxiasgroup
Is it okay like this?
Status: NEW → ASSIGNED
(In reply to Simon Van Accoleyen from comment #7)
> Is it okay like this?

Your new patch seems to be based on the first one. Instead, we'll need one unified patch.
Attachment #8916423 - Attachment is obsolete: true
Attachment #8916714 - Attachment is obsolete: true
Now i hope everything is ok! Sorry to be so long, this is my first ever bug (and i still don't know how to use mercurial correctly...)
Comment on attachment 8917020 [details] [diff] [review]
Third attempt to correctly remove all last bits of SocialAPI

This is just one patch file appended to the other.
We'll need these changes in a single patch produced by mercurial.

(In reply to Simon Van Accoleyen from comment #10)
> Now i hope everything is ok! Sorry to be so long, this is my first ever bug
> (and i still don't know how to use mercurial correctly...)

No problem, take the time you need.
(In reply to Dão Gottwald [::dao] from comment #11)

> This is just one patch file appended to the other.
> We'll need these changes in a single patch produced by mercurial.


What's wrong with this patch? It's a single file for all the changes, and it was made by mercurial...
Status: ASSIGNED → NEW
(In reply to Simon Van Accoleyen from comment #12)
> (In reply to Dão Gottwald [::dao] from comment #11)
> 
> > This is just one patch file appended to the other.
> > We'll need these changes in a single patch produced by mercurial.
> 
> 
> What's wrong with this patch? It's a single file for all the changes, and it
> was made by mercurial...

It contains two patch headers; look for "# HG changeset patch"...
Attached patch Latest PatchSplinter Review
I hope that this will be fine! Thanks for the help guys.
Attachment #8917020 - Attachment is obsolete: true
Flags: needinfo?(dao+bmo)
Comment on attachment 8920809 [details] [diff] [review]
Latest Patch

Looks good. Thanks!
Flags: needinfo?(dao+bmo)
Attachment #8920809 - Flags: review+
Pushed by dgottwald@mozilla.com:
https://hg.mozilla.org/integration/mozilla-inbound/rev/125a67a1750f
Remove last remaining SocialAPI bits. r=dao
Status: NEW → ASSIGNED
https://hg.mozilla.org/mozilla-central/rev/125a67a1750f
Status: ASSIGNED → RESOLVED
Closed: 3 years ago
Resolution: --- → FIXED
Target Milestone: --- → Firefox 58
Whiteboard: [tor 13612]
Product: Firefox → Firefox Graveyard
You need to log in before you can comment on or make changes to this bug.