Closed
Bug 1214593
Opened 9 years ago
Closed 9 years ago
Remove service worker periodic updater
Categories
(Core :: DOM: Service Workers, defect)
Core
DOM: Service Workers
Tracking
()
RESOLVED
FIXED
mozilla45
People
(Reporter: dimi, Assigned: dimi)
References
Details
Attachments
(1 file, 4 obsolete files)
41.46 KB,
patch
|
dimi
:
review+
|
Details | Diff | Splinter Review |
Bug 1207727 will implement the new update algorithm according to latest spec. We can remove periodic updater after new update algorithm is applied
Assignee | ||
Updated•9 years ago
|
Assignee: nobody → dlee
Status: NEW → ASSIGNED
Comment 1•9 years ago
|
||
Thanks Dimi! I can also easily prepare a code removal patch since I wrote it originally. Please let me know if you ant help! :-)
Assignee | ||
Comment 2•9 years ago
|
||
(In reply to Ehsan Akhgari (don't ask for review please) from comment #1) > Thanks Dimi! I can also easily prepare a code removal patch since I wrote > it originally. Please let me know if you ant help! :-) Hi Ehsan, it would be nice to have that patch :) Thanks for help!
Assignee | ||
Comment 3•9 years ago
|
||
Assignee | ||
Comment 4•9 years ago
|
||
Running try: https://treeherder.mozilla.org/#/jobs?repo=try&revision=227728bbdbd5
Comment 5•9 years ago
|
||
Looks like you already have the patch. :-)
Assignee | ||
Comment 6•9 years ago
|
||
Hi Ehsan, Do you know who can help review this ? Thanks!
Flags: needinfo?(ehsan)
Comment 8•9 years ago
|
||
(Or baku!)
Assignee | ||
Comment 9•9 years ago
|
||
Comment on attachment 8679294 [details] [diff] [review] Patch - remove periodic updater Review of attachment 8679294 [details] [diff] [review]: ----------------------------------------------------------------- HI Eshan, If you have time could you help review this since you wrote it :) If you are too busy i will find ben or baku to help on this, thanks!
Attachment #8679294 -
Flags: review?(ehsan)
Comment 10•9 years ago
|
||
Comment on attachment 8679294 [details] [diff] [review] Patch - remove periodic updater Review of attachment 8679294 [details] [diff] [review]: ----------------------------------------------------------------- Looks great, thank you!
Attachment #8679294 -
Flags: review?(ehsan) → review+
Comment 11•9 years ago
|
||
(In reply to Dimi Lee[:dimi][:dlee] from comment #9) > HI Eshan, s/sh/hs/, btw. :-)
Assignee | ||
Comment 12•9 years ago
|
||
(In reply to Ehsan Akhgari (don't ask for review please) [Away Nov 3-19] from comment #11) > (In reply to Dimi Lee[:dimi][:dlee] from comment #9) > > HI Eshan, > > s/sh/hs/, btw. :-) Ah, sorry, won't happen again! :)
Assignee | ||
Comment 13•9 years ago
|
||
-Rebase to latest code
Attachment #8679294 -
Attachment is obsolete: true
Attachment #8681010 -
Flags: review+
Comment 15•9 years ago
|
||
Hi, this failed to apply: patching file dom/workers/ServiceWorkerManager.cpp Hunk #1 FAILED at 4017 1 out of 1 hunks FAILED -- saving rejects to file dom/workers/ServiceWorkerManager.cpp.rej patch failed, unable to continue (try -v) patch failed, rejects left in working directory errors during apply, please fix and refresh bug_1214593.patch
Flags: needinfo?(dlee)
Keywords: checkin-needed
Assignee | ||
Comment 16•9 years ago
|
||
rebase patch
Attachment #8681010 -
Attachment is obsolete: true
Flags: needinfo?(dlee)
Attachment #8681750 -
Flags: review+
Assignee | ||
Updated•9 years ago
|
Keywords: checkin-needed
Assignee | ||
Comment 17•9 years ago
|
||
Comment on attachment 8681750 [details] [diff] [review] Patch - remove periodic updater v3 forget update patch description
Attachment #8681750 -
Attachment is obsolete: true
Assignee | ||
Updated•9 years ago
|
Keywords: checkin-needed
Assignee | ||
Updated•9 years ago
|
Keywords: checkin-needed
Comment 19•9 years ago
|
||
Hi Dimi, this still failed to apply, different file this time: renamed 1214593 -> bug_1214593.patch applying bug_1214593.patch patching file dom/workers/ServiceWorkerPeriodicUpdater.cpp Hunk #1 FAILED at 0 1 out of 1 hunks FAILED -- saving rejects to file dom/workers/ServiceWorkerPeriodicUpdater.cpp.rej
Flags: needinfo?(dlee)
Keywords: checkin-needed
Assignee | ||
Comment 20•9 years ago
|
||
rebase again
Attachment #8681753 -
Attachment is obsolete: true
Attachment #8681809 -
Flags: review+
Assignee | ||
Comment 21•9 years ago
|
||
(In reply to Carsten Book [:Tomcat] from comment #19) > Hi Dimi, this still failed to apply, different file this time: > > renamed 1214593 -> bug_1214593.patch > applying bug_1214593.patch > patching file dom/workers/ServiceWorkerPeriodicUpdater.cpp > Hunk #1 FAILED at 0 > 1 out of 1 hunks FAILED -- saving rejects to file > dom/workers/ServiceWorkerPeriodicUpdater.cpp.rej Sorry i rebase to latest inbound again,could you help check in ? thanks!
Flags: needinfo?(dlee)
Keywords: checkin-needed
Comment 22•9 years ago
|
||
thanks! this time it worked great. Landed !
Comment 23•9 years ago
|
||
https://hg.mozilla.org/integration/mozilla-inbound/rev/b18e03d64493
Keywords: checkin-needed
Comment 24•9 years ago
|
||
bugherder |
https://hg.mozilla.org/mozilla-central/rev/b18e03d64493
Status: ASSIGNED → RESOLVED
Closed: 9 years ago
status-firefox45:
--- → fixed
Resolution: --- → FIXED
Target Milestone: --- → mozilla45
Comment 25•9 years ago
|
||
bugherder uplift |
https://hg.mozilla.org/releases/mozilla-b2g44_v2_5/rev/b18e03d64493
status-b2g-v2.5:
--- → fixed
You need to log in
before you can comment on or make changes to this bug.
Description
•