[DevTools] Prepare to move Popover to shadow DOM.

[DevTools] Prepare to move Popover to shadow DOM.

This patch encapsulates styles of ObjectPopoverHelper, image preview,
network timing table and UISourceCodeFrame's message popover.

BUG=none

Review-Url: https://codereview.chromium.org/2703143002
Cr-Commit-Position: refs/heads/master@{#452308}
Committed: https://chromium.googlesource.com/chromium/src/+/51dd8015de2619e1ef86f01f215ffe2dee297a6e

[dgozman, 2017-02-20 06:29:16]

The CQ bit was checked by dgozman@chromium.org to run a CQ dry run

[commit-bot: I haz the power, 2017-02-20 06:29:30]

Dry run: CQ is trying da patch. Follow status at
 
https://chromium-cq-status.appspot.com/v2/patch-status/codereview.chromium.or...

[dgozman, 2017-02-20 06:29:32]

dgozman@chromium.org changed reviewers:
+ alph@chromium.org

[dgozman, 2017-02-20 06:29:32]

Could you please take a look?

[commit-bot: I haz the power, 2017-02-20 07:47:44]

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

[commit-bot: I haz the power, 2017-02-20 07:47:45]

Dry run: Try jobs failed on following builders:
  linux_chromium_rel_ng on master.tryserver.chromium.linux (JOB_FAILED,
http://build.chromium.org/p/tryserver.chromium.linux/builders/linux_chromium_...)

[alph, 2017-02-21 21:21:33]

lgtm

https://codereview.chromium.org/2703143002/diff/1/third_party/WebKit/LayoutTe...
File
third_party/WebKit/LayoutTests/inspector/tracing/timeline-network/timeline-network-resource-details.html
(right):

https://codereview.chromium.org/2703143002/diff/1/third_party/WebKit/LayoutTe...
third_party/WebKit/LayoutTests/inspector/tracing/timeline-network/timeline-network-resource-details.html:40:
value = Array.prototype.slice.call(value.children, 1).map(element =>
element.textContent).join('');
nit: Array.from(value.children, element =>
element.textContent)).slice(1).join('')

[dgozman, 2017-02-21 21:34:57]

Thank you for review!

https://codereview.chromium.org/2703143002/diff/1/third_party/WebKit/LayoutTe...
File
third_party/WebKit/LayoutTests/inspector/tracing/timeline-network/timeline-network-resource-details.html
(right):

https://codereview.chromium.org/2703143002/diff/1/third_party/WebKit/LayoutTe...
third_party/WebKit/LayoutTests/inspector/tracing/timeline-network/timeline-network-resource-details.html:40:
value = Array.prototype.slice.call(value.children, 1).map(element =>
element.textContent).join('');
On 2017/02/21 21:21:32, alph wrote:
> nit: Array.from(value.children, element =>
> element.textContent)).slice(1).join('')

Acknowledged.

[dgozman, 2017-02-21 21:35:09]

The CQ bit was checked by dgozman@chromium.org

[commit-bot: I haz the power, 2017-02-21 21:36:44]

CQ is trying da patch. Follow status at
 
https://chromium-cq-status.appspot.com/v2/patch-status/codereview.chromium.or...

[commit-bot: I haz the power, 2017-02-21 23:42:17]

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

[commit-bot: I haz the power, 2017-02-21 23:42:18]

Try jobs failed on following builders:
  cast_shell_linux on master.tryserver.chromium.linux (JOB_TIMED_OUT, no build
URL)
  chromeos_amd64-generic_chromium_compile_only_ng on
master.tryserver.chromium.linux (JOB_TIMED_OUT, no build URL)
  chromeos_daisy_chromium_compile_only_ng on master.tryserver.chromium.linux
(JOB_TIMED_OUT, no build URL)
  linux_chromium_asan_rel_ng on master.tryserver.chromium.linux (JOB_TIMED_OUT,
no build URL)
  linux_chromium_chromeos_rel_ng on master.tryserver.chromium.linux
(JOB_TIMED_OUT, no build URL)
  linux_chromium_rel_ng on master.tryserver.chromium.linux (JOB_TIMED_OUT, no
build URL)

[dgozman, 2017-02-22 00:04:41]

The CQ bit was checked by dgozman@chromium.org

[commit-bot: I haz the power, 2017-02-22 00:06:41]

CQ is trying da patch. Follow status at
 
https://chromium-cq-status.appspot.com/v2/patch-status/codereview.chromium.or...

[commit-bot: I haz the power, 2017-02-22 02:09:01]

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

[commit-bot: I haz the power, 2017-02-22 02:09:02]

Try jobs failed on following builders:
  chromium_presubmit on master.tryserver.chromium.linux (JOB_TIMED_OUT, no build
URL)

[dgozman, 2017-02-22 02:09:28]

The CQ bit was checked by dgozman@chromium.org

[commit-bot: I haz the power, 2017-02-22 02:10:52]

CQ is trying da patch. Follow status at
 
https://chromium-cq-status.appspot.com/v2/patch-status/codereview.chromium.or...

[commit-bot: I haz the power, 2017-02-22 04:42:26]

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

[commit-bot: I haz the power, 2017-02-22 04:42:27]

Try jobs failed on following builders:
  linux_chromium_asan_rel_ng on master.tryserver.chromium.linux (JOB_TIMED_OUT,
no build URL)

[dgozman, 2017-02-22 15:27:35]

The CQ bit was checked by dgozman@chromium.org

[commit-bot: I haz the power, 2017-02-22 15:27:58]

CQ is trying da patch. Follow status at
 
https://chromium-cq-status.appspot.com/v2/patch-status/codereview.chromium.or...

[commit-bot: I haz the power, 2017-02-22 15:31:40]

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

[commit-bot: I haz the power, 2017-02-22 15:31:41]

Try jobs failed on following builders:
  chromium_presubmit on master.tryserver.chromium.linux (JOB_FAILED,
http://build.chromium.org/p/tryserver.chromium.linux/builders/chromium_presub...)
  linux_chromium_asan_rel_ng on master.tryserver.chromium.linux (JOB_FAILED,
http://build.chromium.org/p/tryserver.chromium.linux/builders/linux_chromium_...)
  linux_chromium_rel_ng on master.tryserver.chromium.linux (JOB_FAILED,
http://build.chromium.org/p/tryserver.chromium.linux/builders/linux_chromium_...)

[dgozman, 2017-02-22 20:09:00]

The CQ bit was checked by dgozman@chromium.org to run a CQ dry run

[commit-bot: I haz the power, 2017-02-22 20:09:47]

Dry run: CQ is trying da patch. Follow status at
 
https://chromium-cq-status.appspot.com/v2/patch-status/codereview.chromium.or...

[dgozman, 2017-02-22 21:49:00]

The CQ bit was checked by dgozman@chromium.org to run a CQ dry run

[commit-bot: I haz the power, 2017-02-22 21:49:40]

Dry run: CQ is trying da patch. Follow status at
 
https://chromium-cq-status.appspot.com/v2/patch-status/codereview.chromium.or...

[commit-bot: I haz the power, 2017-02-23 00:06:14]

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

[commit-bot: I haz the power, 2017-02-23 00:06:15]

Dry run: This issue passed the CQ dry run.

[dgozman, 2017-02-23 00:20:00]

The CQ bit was checked by dgozman@chromium.org

[dgozman, 2017-02-23 00:20:00]

The patchset sent to the CQ was uploaded after l-g-t-m from alph@chromium.org
Link to the patchset: https://codereview.chromium.org/2703143002/#ps40001
(title: "fix")

[commit-bot: I haz the power, 2017-02-23 00:20:37]

CQ is trying da patch. Follow status at
 
https://chromium-cq-status.appspot.com/v2/patch-status/codereview.chromium.or...

[commit-bot: I haz the power, 2017-02-23 00:35:00]

CQ is committing da patch.

Bot data: {"patchset_id": 40001, "attempt_start_ts": 1487809200125430,
"parent_rev": "e1a5f4de7e19dd33e3781609dd325fa715f354d2", "commit_rev":
"51dd8015de2619e1ef86f01f215ffe2dee297a6e"}

[commit-bot: I haz the power, 2017-02-23 00:35:44]

Description was changed from

==========
[DevTools] Prepare to move Popover to shadow DOM.

This patch encapsulates styles of ObjectPopoverHelper, image preview,
network timing table and UISourceCodeFrame's message popover.

BUG=none
==========

to

==========
[DevTools] Prepare to move Popover to shadow DOM.

This patch encapsulates styles of ObjectPopoverHelper, image preview,
network timing table and UISourceCodeFrame's message popover.

BUG=none

Review-Url: https://codereview.chromium.org/2703143002
Cr-Commit-Position: refs/heads/master@{#452308}
Committed:
https://chromium.googlesource.com/chromium/src/+/51dd8015de2619e1ef86f01f215f...
==========

[commit-bot: I haz the power, 2017-02-23 00:35:46]

Committed patchset #3 (id:40001) as
https://chromium.googlesource.com/chromium/src/+/51dd8015de2619e1ef86f01f215f...