Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Moving service worker job related code to its own file. #628

Merged
merged 1 commit into from
Jun 23, 2023

Conversation

aee-google
Copy link
Contributor

b/286597506

Change-Id: I0ca3ac56d3f1e1acdc6fd2cd8faac34a463b8d61

@aee-google aee-google requested a review from sherryzy June 14, 2023 00:17
Copy link
Contributor

@sherryzy sherryzy left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Nice first step! This looks good to me. Thanks for doing the refactor job. Would love @jellefoks to take a look as well.

cobalt/worker/service_worker_jobs.h Show resolved Hide resolved
@sherryzy sherryzy requested a review from jellefoks June 14, 2023 18:35
Copy link
Member

@jellefoks jellefoks left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Looks great. Thanks for taking the first stab at this. I hope that maybe sometime later we won't need a 'friend class' between these two either.

Copy link
Member

@kaidokert kaidokert left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Please make sure the /internal directory/submodule doesn't get in, git submodules cause a lot of havoc.

Other than that looks like a great refactor

internal Outdated Show resolved Hide resolved
b/286597506

Change-Id: I0ca3ac56d3f1e1acdc6fd2cd8faac34a463b8d61
@aee-google aee-google force-pushed the refactor-service-worker-jobs branch from c3d2d56 to ab2a2d4 Compare June 20, 2023 16:12
@aee-google aee-google merged commit 6e280c8 into youtube:main Jun 23, 2023
338 checks passed
@aee-google aee-google deleted the refactor-service-worker-jobs branch June 23, 2023 16:50
@aee-google aee-google added the cp-24.lts.1+ Cherry Pick to the 24.lts.1+ branch label Jun 27, 2023
cobalt-github-releaser-bot pushed a commit that referenced this pull request Jun 27, 2023
b/286597506

Change-Id: I0ca3ac56d3f1e1acdc6fd2cd8faac34a463b8d61
(cherry picked from commit 6e280c8)
aee-google added a commit to aee-google/cobalt that referenced this pull request Jul 5, 2023
b/286597506

Change-Id: I0ca3ac56d3f1e1acdc6fd2cd8faac34a463b8d61
(cherry picked from commit 6e280c8)
aee-google added a commit that referenced this pull request Jul 6, 2023
b/286597506

Change-Id: I0ca3ac56d3f1e1acdc6fd2cd8faac34a463b8d61
(cherry picked from commit 6e280c8)
aee-google added a commit that referenced this pull request Jul 6, 2023
b/286597506

Change-Id: I0ca3ac56d3f1e1acdc6fd2cd8faac34a463b8d61
(cherry picked from commit 6e280c8)
aee-google added a commit that referenced this pull request Jul 6, 2023
b/286597506

Change-Id: I0ca3ac56d3f1e1acdc6fd2cd8faac34a463b8d61
(cherry picked from commit 6e280c8)

Co-authored-by: aee <[email protected]>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
cp-24.lts.1+ Cherry Pick to the 24.lts.1+ branch
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants