[polymer] Merge of master into polymer10-migration

[polymer] Merge of master into polymer10-migration

This merge contains all Catapult changes until 10:30am on 7/19/16.

There are still 5 trace viewer tests failing, but I'll fix those up in
later CLs.

Files with major work include:

  - tracing/tracing/value/ui/value_set_table.html
  - tracing/tracing/value/ui/value_set_view[_test].html
  - tracing/tracing/ui/base/table[_test].html

NOPRESUBMIT=true
NOTRY=true
BUG=catapult:#2285
R=sullivan@chromium.org

Committed: https://chromium.googlesource.com/external/github.com/catapult-project/catapult/+/d779e0ae7c7f4006536886830e89e3df8633f1c5

[charliea (OOO until 10-5), 2016-07-19 14:57:05]

Description was changed from

==========
[polymer] Merge of master into polymer10-migration

This merge contains all Catapult changes until 10:30am on 7/19/16.

NOPRESUBMIT=true
NOTRY=true
BUG=catapult:#2285
==========

to

==========
[polymer] Merge of master into polymer10-migration

This merge contains all Catapult changes until 10:30am on 7/19/16.

There are still 5 trace viewer tests failing, but I'll fix those up in
later CLs.

Files with major work include:

  - tracing/tracing/ui/base/table.html
  - tracing/tracing/value/ui/value_set_table.html
  - tracing/tracing/value/ui/value_set_view[_test].html
  - tracing/tracing/ui/base/table[_test].html

NOPRESUBMIT=true
NOTRY=true
BUG=catapult:#2285
==========

[charliea (OOO until 10-5), 2016-07-19 14:58:31]

Patchset #1 (id:1) has been deleted

[charliea (OOO until 10-5), 2016-07-19 14:58:59]

Patchset #1 (id:1) has been deleted

[charliea (OOO until 10-5), 2016-07-19 14:59:00]

Patchset #1 (id:1) has been deleted

[charliea (OOO until 10-5), 2016-07-19 15:02:32]

Description was changed from

==========
[polymer] Merge of master into polymer10-migration

This merge contains all Catapult changes until 10:30am on 7/19/16.

There are still 5 trace viewer tests failing, but I'll fix those up in
later CLs.

Files with major work include:

  - tracing/tracing/ui/base/table.html
  - tracing/tracing/value/ui/value_set_table.html
  - tracing/tracing/value/ui/value_set_view[_test].html
  - tracing/tracing/ui/base/table[_test].html

NOPRESUBMIT=true
NOTRY=true
BUG=catapult:#2285
==========

to

==========
[polymer] Merge of master into polymer10-migration

This merge contains all Catapult changes until 10:30am on 7/19/16.

There are still 5 trace viewer tests failing, but I'll fix those up in
later CLs.

Files with major work include:

  - tracing/tracing/value/ui/value_set_table.html
  - tracing/tracing/value/ui/value_set_view[_test].html
  - tracing/tracing/ui/base/table[_test].html

NOPRESUBMIT=true
NOTRY=true
BUG=catapult:#2285
==========

[charliea (OOO until 10-5), 2016-07-19 15:08:03]

charliea@chromium.org changed reviewers:
+ sullivan@chromium.org

[charliea (OOO until 10-5), 2016-07-19 15:08:08]

https://codereview.chromium.org/2162963002/diff/10001/tracing/tracing/ui/base...
File tracing/tracing/ui/base/tab_view.html (left):

https://codereview.chromium.org/2162963002/diff/10001/tracing/tracing/ui/base...
tracing/tracing/ui/base/tab_view.html:125: if (!(subView instanceof HTMLElement)
||
When I rewrote the tab view, I assumed that it would only ever be used in the
analysis pane. Because the value set view started using it, I had to stop
enforcing this constraint.

https://codereview.chromium.org/2162963002/diff/10001/tracing/tracing/ui/base...
File tracing/tracing/ui/base/tab_view.html (right):

https://codereview.chromium.org/2162963002/diff/10001/tracing/tracing/ui/base...
tracing/tracing/ui/base/tab_view.html:60: <div id='tabs'
hidden="[[tabsHidden]]">
The value set view that Ben created for results2.html was the second user of
tab_view.html, which I completely rewrote on the migration branch. His use of it
included an otherwise unused feature that I deleted that allowed the user to
hide the tabs. I had to add this back in to support his use case.

[sullivan, 2016-07-19 15:42:05]

lgtm after dashboard merge fixes.

https://codereview.chromium.org/2162963002/diff/20001/dashboard/dashboard/ele...
File dashboard/dashboard/elements/triage-dialog.html (right):

https://codereview.chromium.org/2162963002/diff/20001/dashboard/dashboard/ele...
dashboard/dashboard/elements/triage-dialog.html:132: </div>
Looks like this is duplicated.

https://codereview.chromium.org/2162963002/diff/20001/dashboard/dashboard/tem...
File dashboard/dashboard/templates/bad_bisect.html (left):

https://codereview.chromium.org/2162963002/diff/20001/dashboard/dashboard/tem...
dashboard/dashboard/templates/bad_bisect.html:6: <link rel="import"
href="/dashboard/elements/primary-button.html">
The CL at master removed this primary-button.html file as well. Should that be
merged in too?

[charliea (OOO until 10-5), 2016-07-19 15:49:07]

https://codereview.chromium.org/2162963002/diff/20001/dashboard/dashboard/ele...
File dashboard/dashboard/elements/triage-dialog.html (right):

https://codereview.chromium.org/2162963002/diff/20001/dashboard/dashboard/ele...
dashboard/dashboard/elements/triage-dialog.html:132: </div>
On 2016/07/19 at 15:42:04, sullivan wrote:
> Looks like this is duplicated.

Done.

[charliea (OOO until 10-5), 2016-07-19 15:49:13]

The CQ bit was checked by charliea@chromium.org

[charliea (OOO until 10-5), 2016-07-19 15:49:14]

The patchset sent to the CQ was uploaded after l-g-t-m from
sullivan@chromium.org
Link to the patchset: https://codereview.chromium.org/2162963002/#ps30001
(title: "Code review")

[commit-bot: I haz the power, 2016-07-19 15:49:26]

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, 2016-07-19 15:50:51]

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

[commit-bot: I haz the power, 2016-07-19 15:50:54]

Failed to apply patch for third_party/vinn/third_party/v8/README.chromium:
While running git apply --index -3 -p1;
  error: patch failed: third_party/vinn/third_party/v8/README.chromium:1
  Falling back to three-way merge...
  error: patch failed: third_party/vinn/third_party/v8/README.chromium:1
  error: third_party/vinn/third_party/v8/README.chromium: patch does not apply

Patch:       third_party/vinn/third_party/v8/README.chromium
Index: third_party/vinn/third_party/v8/README.chromium
diff --git a/third_party/vinn/third_party/v8/README.chromium
b/third_party/vinn/third_party/v8/README.chromium
index
435184c9d3db656ac77a1fccd132da1ffbdbb58b..f8d78d63e686c30e7fe46af993e46f47ac2bb4da
100644
--- a/third_party/vinn/third_party/v8/README.chromium
+++ b/third_party/vinn/third_party/v8/README.chromium
@@ -1,7 +1,7 @@
 Name: Chrome d8 for executing javascript
 Short Name: d8
 URL: https://developers.google.com/v8/
-Version: 5.1.281.59 (linux), 5.1.281.65 (mac), 4.8.271.20 (win)
+Version: 5.1.281.59 (linux), 5.1.281.65 (mac), 5.2.361.30 (win)
 License: BSD
 License File: LICENSE, LICENSE.v8, LICENSE.strongtalk, LICENSE.valgrind
 Security Critical: no

[charliea (OOO until 10-5), 2016-07-19 20:26:10]

Description was changed from

==========
[polymer] Merge of master into polymer10-migration

This merge contains all Catapult changes until 10:30am on 7/19/16.

There are still 5 trace viewer tests failing, but I'll fix those up in
later CLs.

Files with major work include:

  - tracing/tracing/value/ui/value_set_table.html
  - tracing/tracing/value/ui/value_set_view[_test].html
  - tracing/tracing/ui/base/table[_test].html

NOPRESUBMIT=true
NOTRY=true
BUG=catapult:#2285
==========

to

==========
[polymer] Merge of master into polymer10-migration

This merge contains all Catapult changes until 10:30am on 7/19/16.

There are still 5 trace viewer tests failing, but I'll fix those up in
later CLs.

Files with major work include:

  - tracing/tracing/value/ui/value_set_table.html
  - tracing/tracing/value/ui/value_set_view[_test].html
  - tracing/tracing/ui/base/table[_test].html

NOPRESUBMIT=true
NOTRY=true
BUG=catapult:#2285
R=sullivan@chromium.org

Committed:
https://chromium.googlesource.com/external/github.com/catapult-project/catapu...
==========

[charliea (OOO until 10-5), 2016-07-19 20:26:13]

Committed patchset #5 (id:50001) manually as
d779e0ae7c7f4006536886830e89e3df8633f1c5.