New issue
Advanced search Search tips
Note: Color blocks (like █ or █) mean that a user may not be available. Tooltip shows the reason.

Issue 596232 link

Starred by 1 user
Status: Fixed
Owner:
█ maxbogue@chromium.org
Last visit > 30 days ago
Closed: Mar 2016
Cc:
s...@chromium.org
pav...@chromium.org
mastiz@chromium.org
█ tschumann@chromium.org
Components:
EstimatedDays: ----
NextAction: ----
OS: All
Pri: 2
Type: Bug



Sign in to add a comment

[USS] Update responses should be batched in the worker

Project Member Reported by maxbogue@chromium.org, Mar 19 2016 
We want the entire update to come through as one call to the processor so it can deal with the initial merge case in a sane way.
Project Member

Comment 1 by bugdroid1@chromium.org, Mar 24 2016 

The following revision refers to this bug:
  https://chromium.googlesource.com/chromium/src.git/+/8cd3afc795cf42b5bbc1edec1e0b93d234558c05

commit 8cd3afc795cf42b5bbc1edec1e0b93d234558c05
Author: maxbogue <maxbogue@chromium.org>
Date: Thu Mar 24 00:45:47 2016

[Sync] Accumulate GU response chunks in the Worker.

This change renames "pending" updates in the worker to "encrypted"
updates and repurposes "pending" to refer to the updates being
accumulated for sending to the processor.

BUG= 596232 

Review URL: https://codereview.chromium.org/1814823006

Cr-Commit-Position: refs/heads/master@{#382994}

[modify] https://crrev.com/8cd3afc795cf42b5bbc1edec1e0b93d234558c05/sync/engine/model_type_worker.cc
[modify] https://crrev.com/8cd3afc795cf42b5bbc1edec1e0b93d234558c05/sync/engine/model_type_worker.h
[modify] https://crrev.com/8cd3afc795cf42b5bbc1edec1e0b93d234558c05/sync/engine/model_type_worker_unittest.cc
[modify] https://crrev.com/8cd3afc795cf42b5bbc1edec1e0b93d234558c05/sync/engine/worker_entity_tracker.cc
[modify] https://crrev.com/8cd3afc795cf42b5bbc1edec1e0b93d234558c05/sync/engine/worker_entity_tracker.h

Comment 2 by maxbogue@chromium.org, Mar 24 2016

Status: Fixed (was: Assigned)
►

Sign in to add a comment