Remove unused code from DuplicateFinder.

Remove unused code from DuplicateFinder.

BUG=v8:4947
Committed: https://crrev.com/7a8c5049c5445aebfd270840dd659c5207089e6d
Cr-Commit-Position: refs/heads/master@{#41453}

[vogelheim, 2016-12-01 10:58:45]

The CQ bit was checked by vogelheim@chromium.org to run a CQ dry run

[commit-bot: I haz the power, 2016-12-01 10:58:55]

Dry run: CQ is trying da patch. Follow status at
 
https://chromium-cq-status.appspot.com/v2/patch-status/codereview.chromium.or...

[commit-bot: I haz the power, 2016-12-01 11:23:58]

The CQ bit was unchecked by commit-bot@chromium.org

[commit-bot: I haz the power, 2016-12-01 11:23:58]

Dry run: This issue passed the CQ dry run.

[vogelheim, 2016-12-01 11:40:56]

Description was changed from

==========
Remove unused code from DuplicateFinder.

BUG=v8:4947
==========

to

==========
Remove unused code from DuplicateFinder.

BUG=v8:4947
==========

[vogelheim, 2016-12-01 11:40:56]

vogelheim@chromium.org changed reviewers:
+ verwaest@chromium.org

[vogelheim, 2016-12-01 11:41:22]

ptal

This is pretty much a deletion-only CL. It seems DuplicateFinder was originally
meant to support more complex functionality (data conversion/normalization +
built-in aggregation of values), but the only uses I could find are using it as
a glorified hash set.

It might make sense to replace it with a hash set in a 2nd step. I currently
don't want to do it in order to not loose the one-byte/two-byte-non-copy
optimization (although I'm not sure it buys a whole lot).

[vogelheim, 2016-12-02 10:49:09]

vogelheim@chromium.org changed reviewers:
+ marja@chromium.org

[marja, 2016-12-02 11:45:59]

lgtm

(As discussed offline, duplicatefinder could reuse the hash already computed for
the aststring, but that's a follow-up...)

[vogelheim, 2016-12-02 12:08:48]

The CQ bit was checked by vogelheim@chromium.org

[commit-bot: I haz the power, 2016-12-02 12:08:51]

CQ is trying da patch. Follow status at
 
https://chromium-cq-status.appspot.com/v2/patch-status/codereview.chromium.or...

[commit-bot: I haz the power, 2016-12-02 12:34:59]

CQ is committing da patch.

Bot data: {"patchset_id": 1, "attempt_start_ts": 1480680528840850, "parent_rev":
"2104e2d8cee9325076fd42a709910ff2c35ed508", "commit_rev":
"ccb4c1a181c8a88af893c1c635bf95b5cb26be1f"}

[commit-bot: I haz the power, 2016-12-02 12:35:04]

Description was changed from

==========
Remove unused code from DuplicateFinder.

BUG=v8:4947
==========

to

==========
Remove unused code from DuplicateFinder.

BUG=v8:4947
==========

[commit-bot: I haz the power, 2016-12-02 12:35:05]

Committed patchset #1 (id:1)

[commit-bot: I haz the power, 2016-12-02 12:35:33]

Description was changed from

==========
Remove unused code from DuplicateFinder.

BUG=v8:4947
==========

to

==========
Remove unused code from DuplicateFinder.

BUG=v8:4947

Committed: https://crrev.com/7a8c5049c5445aebfd270840dd659c5207089e6d
Cr-Commit-Position: refs/heads/master@{#41453}
==========

[commit-bot: I haz the power, 2016-12-02 12:35:33]

Patchset 1 (id:??) landed as
https://crrev.com/7a8c5049c5445aebfd270840dd659c5207089e6d
Cr-Commit-Position: refs/heads/master@{#41453}