Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(507)

Issue 141283006: Remove stale 'virtual' in DiscardableMemoryAllocator. (Closed)

Created:
6 years, 11 months ago by Philippe
Modified:
6 years, 11 months ago
CC:
chromium-reviews, erikwright+watch_chromium.org, gavinp+memory_chromium.org
Visibility:
Public.

Description

Remove stale 'virtual' in DiscardableMemoryAllocator. DiscardableMemoryAllocator::AshmemRegion used to implement an interface but this is not the case anymore therefore there is no reason to keep ~AshmemRegion() virtual (since AshmemRegion is/should never be subclassed either). TBR=willchan@chromium.org Committed: https://src.chromium.org/viewvc/chrome?view=rev&revision=245520

Patch Set 1 #

Unified diffs Side-by-side diffs Delta from patch set Stats (+1 line, -1 line) Patch
M base/memory/discardable_memory_allocator_android.cc View 1 chunk +1 line, -1 line 0 comments Download

Messages

Total messages: 2 (0 generated)
commit-bot: I haz the power
CQ is trying da patch. Follow status at https://chromium-status.appspot.com/cq/pliard@chromium.org/141283006/1
6 years, 11 months ago (2014-01-17 12:44:01 UTC) #1
commit-bot: I haz the power
6 years, 11 months ago (2014-01-17 14:48:08 UTC) #2
Message was sent while issue was closed.
Change committed as 245520

Powered by Google App Engine
This is Rietveld 408576698