Added vtune profiler.

Add vtune profiler support on linux.

BUG=
TEST=./tools/perf/run_measurement --browser=release page_cycler tools/perf/page_sets/tough_layout_cases.json --page-filter=amar --pageset-repeat=2 --profiler=vtune

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

[tonyg, 2013-09-04 18:16:03]

lgtm

https://codereview.chromium.org/23536025/diff/1/tools/telemetry/telemetry/cor...
File tools/telemetry/telemetry/core/platform/profiler/vtune_profiler.py (right):

https://codereview.chromium.org/23536025/diff/1/tools/telemetry/telemetry/cor...
tools/telemetry/telemetry/core/platform/profiler/vtune_profiler.py:23:
stdout=self._tmp_output_file, stderr=subprocess.STDOUT)
funny indentation here

https://codereview.chromium.org/23536025/diff/1/tools/telemetry/telemetry/cor...
tools/telemetry/telemetry/core/platform/profiler/vtune_profiler.py:36: #   
Provide a valid PID.
Why do we allow this error?

https://codereview.chromium.org/23536025/diff/1/tools/telemetry/telemetry/cor...
tools/telemetry/telemetry/core/platform/profiler/vtune_profiler.py:40:
self._GetStdOut()))
bad indent

https://codereview.chromium.org/23536025/diff/1/tools/telemetry/telemetry/cor...
tools/telemetry/telemetry/core/platform/profiler/vtune_profiler.py:66: if not
'renderer' in output_file:
Does this condition still work with:

--extra-browser-args=--single-process

[shatch, 2013-09-04 20:30:41]

New snapshot uploaded.

https://codereview.chromium.org/23536025/diff/1/tools/telemetry/telemetry/cor...
File tools/telemetry/telemetry/core/platform/profiler/vtune_profiler.py (right):

https://codereview.chromium.org/23536025/diff/1/tools/telemetry/telemetry/cor...
tools/telemetry/telemetry/core/platform/profiler/vtune_profiler.py:23:
stdout=self._tmp_output_file, stderr=subprocess.STDOUT)
On 2013/09/04 18:16:03, tonyg wrote:
> funny indentation here

Done.

https://codereview.chromium.org/23536025/diff/1/tools/telemetry/telemetry/cor...
tools/telemetry/telemetry/core/platform/profiler/vtune_profiler.py:36: #   
Provide a valid PID.
On 2013/09/04 18:16:03, tonyg wrote:
> Why do we allow this error?

That's the error from the first run renderer getting swapped out.

https://codereview.chromium.org/23536025/diff/1/tools/telemetry/telemetry/cor...
tools/telemetry/telemetry/core/platform/profiler/vtune_profiler.py:40:
self._GetStdOut()))
On 2013/09/04 18:16:03, tonyg wrote:
> bad indent 

Done.

https://codereview.chromium.org/23536025/diff/1/tools/telemetry/telemetry/cor...
tools/telemetry/telemetry/core/platform/profiler/vtune_profiler.py:66: if not
'renderer' in output_file:
On 2013/09/04 18:16:03, tonyg wrote:
> Does this condition still work with:
> 
> --extra-browser-args=--single-process

Actually I think just attaching to every pid works after all (just takes a
couple minutes), should I just do the same as perf in that case and profile
everything? Or still just profile renderer?

[shatch, 2013-09-04 20:49:37]

New snapshot uploaded, ptal.

[tonyg, 2013-09-04 21:03:33]

lgtm

[commit-bot: I haz the power, 2013-09-04 21:06:49]

CQ is trying da patch. Follow status at
https://chromium-status.appspot.com/cq/simonhatch@chromium.org/23536025/10001

[commit-bot: I haz the power, 2013-09-05 00:19:07]

Retried try job too often on win7_aura for step(s) interactive_ui_tests
http://build.chromium.org/p/tryserver.chromium/buildstatus?builder=win7_aura&...

[commit-bot: I haz the power, 2013-09-05 00:21:25]

CQ is trying da patch. Follow status at
https://chromium-status.appspot.com/cq/simonhatch@chromium.org/23536025/10001

[commit-bot: I haz the power, 2013-09-05 03:09:22]

Change committed as 221333