Issue 658053 link

Starred by 1 user

Issue metadata

Status:	Fixed
Owner:	s...@chromium.org
Closed:	Oct 2016
Cc:	mastiz@chromium.org █ tschumann@chromium.org
Components:	Services>Sync
EstimatedDays:	----
NextAction:	----
OS:	----
Pri:	3
Type:	Bug
Hotlist-CodeHealth

Remove ScopedVector from ModelTypeRegistry

Project Member Reported by s...@chromium.org, Oct 20 2016

Issue description

Probably some more pointer cleanup we can do in there as well.

Project Member

Comment 1 by bugdroid1@chromium.org, Oct 21 2016

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

commit 6754203e3e27c476757265c0bd35a823258aa7bf
Author: skym <skym@chromium.org>
Date: Fri Oct 21 19:05:10 2016

[Sync] Removed ScopdVector and raw news from ModelTypeRegistry.

BUG= 658053 

Review-Url: https://chromiumcodereview.appspot.com/2437873006
Cr-Commit-Position: refs/heads/master@{#426856}

[modify] https://crrev.com/6754203e3e27c476757265c0bd35a823258aa7bf/components/sync/engine_impl/get_updates_delegate.cc
[modify] https://crrev.com/6754203e3e27c476757265c0bd35a823258aa7bf/components/sync/engine_impl/model_type_registry.cc
[modify] https://crrev.com/6754203e3e27c476757265c0bd35a823258aa7bf/components/sync/engine_impl/model_type_registry.h

Comment 2 by s...@chromium.org, Oct 27 2016

Status: Fixed (was: Assigned)

►

Issue 658053 link

Issue metadata

Remove ScopedVector from ModelTypeRegistry

Issue description

Comment 1 by bugdroid1@chromium.org, Oct 21 2016

Comment 1 Deleted

Comment 2 by s...@chromium.org, Oct 27 2016

Comment 2 Deleted

Sign in to add a comment