Remove obsolete TakeHeapSnapshot method from API

Remove obsolete TakeHeapSnapshot method from API

BUG=chromium:465651
LOG=Y

Committed: https://crrev.com/f92c8a36709efb1514846b3bb672c40e4fb4c672
Cr-Commit-Position: refs/heads/master@{#27208}

[yurys, 2015-03-12 06:36:57]

yurys@chromium.org changed reviewers:
+ alph@chromium.org, loislo@chromium.org, yangguo@chromium.org

[Yang, 2015-03-12 08:04:13]

On 2015/03/12 06:36:58, yurys wrote:

lgtm.

[yurys, 2015-03-13 06:09:46]

The CQ bit was checked by yurys@chromium.org

[commit-bot: I haz the power, 2015-03-13 06:09:53]

CQ is trying da patch. Follow status at
 https://chromium-cq-status.appspot.com/patch-status/997583004/1

[commit-bot: I haz the power, 2015-03-13 06:44:03]

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

[commit-bot: I haz the power, 2015-03-13 06:44:04]

Try jobs failed on following builders:
  v8_linux_chromium_gn_rel on tryserver.v8 (JOB_FAILED,
http://build.chromium.org/p/tryserver.v8/builders/v8_linux_chromium_gn_rel/bu...)

[yurys, 2015-03-13 10:39:55]

The CQ bit was checked by yurys@chromium.org

[commit-bot: I haz the power, 2015-03-13 10:40:02]

CQ is trying da patch. Follow status at
 https://chromium-cq-status.appspot.com/patch-status/997583004/1

[commit-bot: I haz the power, 2015-03-13 11:35:28]

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

[commit-bot: I haz the power, 2015-03-13 11:35:29]

Try jobs failed on following builders:
  v8_linux_chromium_gn_rel on tryserver.v8 (JOB_FAILED,
http://build.chromium.org/p/tryserver.v8/builders/v8_linux_chromium_gn_rel/bu...)

[Yang, 2015-03-13 11:41:37]

On 2015/03/13 11:35:29, I haz the power (commit-bot) wrote:
> Try jobs failed on following builders:
>   v8_linux_chromium_gn_rel on tryserver.v8 (JOB_FAILED,
>
http://build.chromium.org/p/tryserver.v8/builders/v8_linux_chromium_gn_rel/bu...)

I think you either have to fix the call site in Blink or wait for the bot to
update.

[yurys, 2015-03-13 13:45:29]

On 2015/03/13 11:41:37, Yang wrote:
> On 2015/03/13 11:35:29, I haz the power (commit-bot) wrote:
> > Try jobs failed on following builders:
> >   v8_linux_chromium_gn_rel on tryserver.v8 (JOB_FAILED,
> >
>
http://build.chromium.org/p/tryserver.v8/builders/v8_linux_chromium_gn_rel/bu...)
> 
> I think you either have to fix the call site in Blink or wait for the bot to
> update.

Blink just hasn't been rolled into Chromium for too long. So the change is in
Blink but not in Chromium ToT, I was abusing CQ to check this a day after the
change landed in Blink.

[yurys, 2015-03-13 13:50:26]

On 2015/03/13 13:45:29, yurys wrote:
> On 2015/03/13 11:41:37, Yang wrote:
> > On 2015/03/13 11:35:29, I haz the power (commit-bot) wrote:
> > > Try jobs failed on following builders:
> > >   v8_linux_chromium_gn_rel on tryserver.v8 (JOB_FAILED,
> > >
> >
>
http://build.chromium.org/p/tryserver.v8/builders/v8_linux_chromium_gn_rel/bu...)
> > 
> > I think you either have to fix the call site in Blink or wait for the bot to
> > update.
> 
> Blink just hasn't been rolled into Chromium for too long. So the change is in
> Blink but not in Chromium ToT, I was abusing CQ to check this a day after the
> change landed in Blink.

Oh, there is one more usage of the method in Blink unit tests. Will update them
first.

[yurys, 2015-03-14 15:10:57]

The CQ bit was checked by yurys@chromium.org

[commit-bot: I haz the power, 2015-03-14 15:11:01]

CQ is trying da patch. Follow status at
 https://chromium-cq-status.appspot.com/patch-status/997583004/1

[commit-bot: I haz the power, 2015-03-14 16:14:03]

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

[commit-bot: I haz the power, 2015-03-14 16:14:04]

Try jobs failed on following builders:
  v8_linux_chromium_gn_rel on tryserver.v8 (JOB_FAILED,
http://build.chromium.org/p/tryserver.v8/builders/v8_linux_chromium_gn_rel/bu...)

[yurys, 2015-03-15 06:44:17]

The CQ bit was checked by yurys@chromium.org

[commit-bot: I haz the power, 2015-03-15 06:44:21]

CQ is trying da patch. Follow status at
 https://chromium-cq-status.appspot.com/patch-status/997583004/1

[commit-bot: I haz the power, 2015-03-15 07:32:32]

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

[commit-bot: I haz the power, 2015-03-15 07:32:33]

Try jobs failed on following builders:
  v8_linux_chromium_gn_rel on tryserver.v8 (JOB_FAILED,
http://build.chromium.org/p/tryserver.v8/builders/v8_linux_chromium_gn_rel/bu...)

[yurys, 2015-03-16 09:22:35]

The CQ bit was checked by yurys@chromium.org

[commit-bot: I haz the power, 2015-03-16 09:22:44]

CQ is trying da patch. Follow status at
 https://chromium-cq-status.appspot.com/patch-status/997583004/1

[commit-bot: I haz the power, 2015-03-16 09:49:23]

Committed patchset #1 (id:1)

[commit-bot: I haz the power, 2015-03-16 09:49:35]

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

[ulan, 2015-06-15 11:44:15]

On 2015/03/16 09:49:35, commit-bot: I haz the power wrote:
> Patchset 1 (id:??) landed as
> https://crrev.com/f92c8a36709efb1514846b3bb672c40e4fb4c672
> Cr-Commit-Position: refs/heads/master@{#27208}

When removing deprecated functions in future, please add in the description what
should be used instead.

[yurys, 2015-06-15 11:53:36]

On 2015/06/15 11:44:15, ulan wrote:
> On 2015/03/16 09:49:35, commit-bot: I haz the power wrote:
> > Patchset 1 (id:??) landed as
> > https://crrev.com/f92c8a36709efb1514846b3bb672c40e4fb4c672
> > Cr-Commit-Position: refs/heads/master@{#27208}
> 
> When removing deprecated functions in future, please add in the description
what
> should be used instead.

Sure.