Commit be6e4c32 authored by hclam@chromium.org's avatar hclam@chromium.org

Revert of Resubmit: Add real file for AdNetworks DB (https://codereview.chromium.org/274563003/)

Reason for revert:
Linux Asan is failing with this patch: http://build.chromium.org/p/chromium.memory/builders/Linux%20ASan%20LSan%20Tests%20%282%29/builds/2483/steps/unit_tests/logs/stdio

Original issue's description:
> Resubmit: Add real file for AdNetworks DB
>
> This is a resubmit of https://codereview.chromium.org/268063004/.
>
> The previous version was reverted because it caused a memory leak. This was a
> problem with RefCounting from ui::ResourceBundle::LoadDataResourceBytes(), which
> has been fixed.
>
> Details:
> Even though the method returns a non-refcounted ptr to a RefCountedStaticMemory
> (which according to the class comment, "the ref counting does not matter" [1]),
> the ref-counting _does_ matter. The fix for this went in as part of a refactor
> in https://codereview.chromium.org/263953003/.
>
> [1] http://src.chromium.org/viewvc/chrome/trunk/src/base/memory/ref_counted_memory.h?revision=267321 line 44
>
> BUG=357204
>
> TBR=jhawkins@chromium.org (previously approved)
> TBR=felt@chromium.org (very minor changes)
>
> Committed: https://src.chromium.org/viewvc/chrome?view=rev&revision=269494

TBR=jhawkins@chromium.org,felt@chromium.org,rdevlin.cronin@chromium.org
NOTREECHECKS=true
NOTRY=true
BUG=357204

Committed: https://src.chromium.org/viewvc/chrome?view=rev&revision=269512

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

git-svn-id: svn://svn.chromium.org/chrome/trunk/src@269535 0039d316-1c4b-4281-b951-d872f2087c98
parent 3cd6c5f5
Markdown is supported
0%
or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment