Task Manager: Remove dead FPS code.

Task Manager: Remove dead FPS code.

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

[Lei Zhang, 2014-06-06 19:00:02]

The FPS column in the task manager has been reporting 0 for a while. Unless
someone is going to resurrect the code to get the FPS value, this is all useless
code.

jam: contents
palmer: message removal
yoz: chrome extensions bits
sky: chrome sans extensions

[Lei Zhang, 2014-06-06 19:01:46]

+nduca FYI

[palmer, 2014-06-06 19:16:59]

IPC security LGTM.

[sky, 2014-06-06 19:55:44]

LGTM

[jam, 2014-06-06 20:13:06]

lgtm

[Yoyo Zhou, 2014-06-06 20:45:26]

You should also remove it from the API in processes.json. The fps parameter
doesn't appear to be used by any extensions that I could find in the webstore,
so it should be okay (and processes hasn't been released to stable yet, so it's
okay to make changes like this).

[Lei Zhang, 2014-06-06 20:51:14]

On 2014/06/06 20:45:26, Yoyo Zhou wrote:
> You should also remove it from the API in processes.json. The fps parameter
> doesn't appear to be used by any extensions that I could find in the webstore,
> so it should be okay (and processes hasn't been released to stable yet, so
it's
> okay to make changes like this).

Done in patch set 2.

[Yoyo Zhou, 2014-06-06 21:32:55]

LGTM

[Lei Zhang, 2014-06-06 21:39:03]

The CQ bit was checked by thestig@chromium.org

[commit-bot: I haz the power, 2014-06-06 21:41:02]

CQ is trying da patch. Follow status at
 https://chromium-status.appspot.com/cq/thestig@chromium.org/319953002/40001

[commit-bot: I haz the power, 2014-06-07 04:42:24]

FYI, CQ is re-trying this CL (attempt #1).
The failing builders are:
  android_chromium_gn_compile_rel on tryserver.chromium
(http://build.chromium.org/p/tryserver.chromium/builders/android_chromium_gn_c...)
  android_dbg on tryserver.chromium
(http://build.chromium.org/p/tryserver.chromium/builders/android_dbg/builds/19...)
  chromium_presubmit on tryserver.chromium
(http://build.chromium.org/p/tryserver.chromium/builders/chromium_presubmit/bu...)
  linux_chromium_chromeos_clang_dbg on tryserver.chromium
(http://build.chromium.org/p/tryserver.chromium/builders/linux_chromium_chrome...)
  linux_chromium_chromeos_rel on tryserver.chromium
(http://build.chromium.org/p/tryserver.chromium/builders/linux_chromium_chrome...)
  linux_chromium_clang_dbg on tryserver.chromium
(http://build.chromium.org/p/tryserver.chromium/builders/linux_chromium_clang_...)
  linux_chromium_gn_rel on tryserver.chromium
(http://build.chromium.org/p/tryserver.chromium/builders/linux_chromium_gn_rel...)
  linux_chromium_rel on tryserver.chromium
(http://build.chromium.org/p/tryserver.chromium/builders/linux_chromium_rel/bu...)

[commit-bot: I haz the power, 2014-06-07 04:46:09]

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

[commit-bot: I haz the power, 2014-06-07 04:46:10]

Try jobs failed on following builders:
  android_chromium_gn_compile_rel on tryserver.chromium
(http://build.chromium.org/p/tryserver.chromium/builders/android_chromium_gn_c...)
  android_dbg on tryserver.chromium
(http://build.chromium.org/p/tryserver.chromium/builders/android_dbg/builds/19...)
  chromium_presubmit on tryserver.chromium
(http://build.chromium.org/p/tryserver.chromium/builders/chromium_presubmit/bu...)
  linux_chromium_chromeos_clang_dbg on tryserver.chromium
(http://build.chromium.org/p/tryserver.chromium/builders/linux_chromium_chrome...)
  linux_chromium_chromeos_rel on tryserver.chromium
(http://build.chromium.org/p/tryserver.chromium/builders/linux_chromium_chrome...)
  linux_chromium_clang_dbg on tryserver.chromium
(http://build.chromium.org/p/tryserver.chromium/builders/linux_chromium_clang_...)
  linux_chromium_gn_rel on tryserver.chromium
(http://build.chromium.org/p/tryserver.chromium/builders/linux_chromium_gn_rel...)
  linux_chromium_rel on tryserver.chromium
(http://build.chromium.org/p/tryserver.chromium/builders/linux_chromium_rel/bu...)

[Lei Zhang, 2014-06-07 05:30:10]

The CQ bit was checked by thestig@chromium.org

[commit-bot: I haz the power, 2014-06-07 05:30:38]

CQ is trying da patch. Follow status at
 https://chromium-status.appspot.com/cq/thestig@chromium.org/319953002/60001

[commit-bot: I haz the power, 2014-06-07 09:28:22]

Change committed as 275670