[telemetry] Fix InspectorPage.CollectGarbage to actually collect garbage.

[telemetry] Fix InspectorPage.CollectGarbage to actually collect garbage.

There was a typo that made this function a no-op.

Performance sheriffs: this will affect memory benchmarks that rely on CollectGarbage.

BUG=

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

[ulan, 2015-02-17 16:31:04]

ulan@chromium.org changed reviewers:
+ skyostil@chromium.org

[ulan, 2015-02-17 16:31:05]

PTAL

[Sami, 2015-02-17 17:01:40]

Good find, lgtm.

https://codereview.chromium.org/927423002/diff/1/tools/telemetry/telemetry/co...
File tools/telemetry/telemetry/core/backends/chrome_inspector/inspector_page.py
(right):

https://codereview.chromium.org/927423002/diff/1/tools/telemetry/telemetry/co...
tools/telemetry/telemetry/core/backends/chrome_inspector/inspector_page.py:161:
assert('result' in res)
nit: drop the parentheses

[ulan, 2015-02-18 08:42:46]

New patchsets have been uploaded after l-g-t-m from skyostil@chromium.org

[ulan, 2015-02-18 08:43:14]

https://codereview.chromium.org/927423002/diff/1/tools/telemetry/telemetry/co...
File tools/telemetry/telemetry/core/backends/chrome_inspector/inspector_page.py
(right):

https://codereview.chromium.org/927423002/diff/1/tools/telemetry/telemetry/co...
tools/telemetry/telemetry/core/backends/chrome_inspector/inspector_page.py:161:
assert('result' in res)
On 2015/02/17 17:01:39, Sami wrote:
> nit: drop the parentheses

Done.

[ulan, 2015-02-18 08:43:17]

The CQ bit was checked by ulan@chromium.org

[commit-bot: I haz the power, 2015-02-18 08:43:27]

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

[commit-bot: I haz the power, 2015-02-18 08:44:16]

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

[commit-bot: I haz the power, 2015-02-18 08:44:17]

Try jobs failed on following builders:
  android_chromium_gn_compile_dbg on tryserver.chromium.linux (JOB_FAILED,
http://build.chromium.org/p/tryserver.chromium.linux/builders/android_chromiu...)

[ulan, 2015-02-18 13:42:10]

The CQ bit was checked by ulan@chromium.org

[commit-bot: I haz the power, 2015-02-18 13:42:41]

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

[commit-bot: I haz the power, 2015-02-18 14:37:25]

Committed patchset #2 (id:20001)

[commit-bot: I haz the power, 2015-02-18 14:38:01]

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