Don't deallocate the HashTable backing explicitly when it is managed by GC.

Don't deallocate the HashTable backing explicitly when it is managed by GC.

This leads to double finalization.

NOTRY=true

Committed: https://src.chromium.org/viewvc/blink?view=rev&revision=168467

[Vyacheslav Egorov (Chromium), 2014-03-04 13:52:39]

PTAL

[zerny-chromium, 2014-03-04 14:02:05]

lgtm

[tkent, 2014-03-05 01:12:40]

lgtm

https://codereview.chromium.org/180273022/diff/1/Source/wtf/HashTable.h
File Source/wtf/HashTable.h (right):

https://codereview.chromium.org/180273022/diff/1/Source/wtf/HashTable.h#newco...
Source/wtf/HashTable.h:283: if (!Allocator::isGarbageCollected) {
nit: We prefer early return.

{
    if (Allocator::isGarbageCollected)
        return;
    if (LIKELY(...))

[commit-bot: I haz the power, 2014-03-05 01:14:36]

CQ is trying da patch. Follow status at
https://chromium-status.appspot.com/cq/vegorov@chromium.org/180273022/1

[wibling-chromium, 2014-03-05 08:40:13]

lgtm

[Erik Corry, 2014-03-05 08:42:00]

lgtm

[haraken, 2014-03-05 08:44:22]

LGTM

[Vyacheslav Egorov (Chromium), 2014-03-05 11:38:00]

The CQ bit was unchecked by vegorov@chromium.org

[Vyacheslav Egorov (Chromium), 2014-03-05 11:41:04]

The CQ bit was checked by vegorov@chromium.org

[commit-bot: I haz the power, 2014-03-05 11:41:13]

CQ is trying da patch. Follow status at
https://chromium-status.appspot.com/cq/vegorov@chromium.org/180273022/20001

[commit-bot: I haz the power, 2014-03-05 11:41:28]

Change committed as 168467