DevTools: properly update paint profiler upon new event being set

DevTools: properly update paint profiler upon new event being set

We used to update paint profiler only upon onShow(), which accidentally
worked because the view kept being re-attached when selection changed.
Now that we keep the view, we need to to kick update explicitly when
a new event is set.

BUG=

Committed: https://src.chromium.org/viewvc/blink?view=rev&revision=196500

[caseq, 2015-06-01 12:02:37]

caseq@chromium.org changed reviewers:
+ alph@chromium.org, yurys@chromium.org

[pfeldman, 2015-06-01 12:11:58]

On 2015/06/01 12:02:38, caseq wrote:

A test?

[caseq, 2015-06-01 15:12:16]

Patchset #2 (id:20001) has been deleted

[caseq, 2015-06-01 15:12:27]

Added a test, ptal.

[pfeldman, 2015-06-01 16:13:29]

pfeldman@chromium.org changed reviewers:
+ pfeldman@chromium.org

[pfeldman, 2015-06-01 16:13:30]

lgtm

https://codereview.chromium.org/1162223002/diff/40001/LayoutTests/inspector/t...
File LayoutTests/inspector/tracing/paint-profiler-update.html (right):

https://codereview.chromium.org/1162223002/diff/40001/LayoutTests/inspector/t...
LayoutTests/inspector/tracing/paint-profiler-update.html:49:
InspectorTest.addSniffer(panel, "_appendDetailsTabsForTraceEventAndShowDetails",
onRecordDetailsReady, false);
Is there a way to avoid 2 sniffers in this test? I a m a bit worried that it'll
be hard to maintain.

[caseq, 2015-06-01 16:29:37]

The CQ bit was checked by caseq@chromium.org

[commit-bot: I haz the power, 2015-06-01 16:30:16]

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

[commit-bot: I haz the power, 2015-06-01 18:20:58]

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

[commit-bot: I haz the power, 2015-06-01 18:20:59]

Try jobs failed on following builders:
  linux_blink_rel on tryserver.blink (JOB_FAILED,
http://build.chromium.org/p/tryserver.blink/builders/linux_blink_rel/builds/6...)

[caseq, 2015-06-04 12:04:02]

Patchset #3 (id:60001) has been deleted

[caseq, 2015-06-04 12:06:06]

The CQ bit was checked by caseq@chromium.org

[caseq, 2015-06-04 12:06:07]

The patchset sent to the CQ was uploaded after l-g-t-m from
pfeldman@chromium.org
Link to the patchset: https://codereview.chromium.org/1162223002/#ps80001
(title: "skip the test under virtual/deferred")

[commit-bot: I haz the power, 2015-06-04 12:06:40]

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

[commit-bot: I haz the power, 2015-06-04 13:55:54]

Committed patchset #3 (id:80001) as
https://src.chromium.org/viewvc/blink?view=rev&revision=196500