{
"findings": [
{
"category": "clang-tidy",
"fixes": [
{
"replacements": [
{
"location": {
"file_path": "third_party/blink/renderer/core/css/resolver/font_builder.h",
"gerrit_change_ref": {
"change": "6936189",
"host": "chromium-review.googlesource.com",
"patchset": "7",
"project": "chromium/src"
},
"range": {
"end_column": 28,
"end_line": 228,
"start_column": 20,
"start_line": 228
}
},
"new_content": "static_cast<unsigned>"
}
]
}
],
"location": {
"file_path": "third_party/blink/renderer/core/css/resolver/font_builder.h",
"gerrit_change_ref": {
"change": "6936189",
"host": "chromium-review.googlesource.com",
"patchset": "7",
"project": "chromium/src"
},
"range": {
"end_line": 228,
"start_line": 228
}
},
"message": "check: google-readability-casting\n\nC-style casts are discouraged; use static_cast (
https://clang.llvm.org/extra/clang-tidy/checks/google/readability-casting.html)\n\n(Note: You can add `Skip-Clang-Tidy-Checks: google-readability-casting` footer to the CL description to skip the check)",
"severity_level": "SEVERITY_LEVEL_WARNING"
},
{
"category": "clang-tidy",
"fixes": [
{
"replacements": [
{
"location": {
"file_path": "third_party/blink/renderer/core/css/resolver/font_builder.h",
"gerrit_change_ref": {
"change": "6936189",
"host": "chromium-review.googlesource.com",
"patchset": "7",
"project": "chromium/src"
},
"range": {
"end_column": 34,
"end_line": 231,
"start_column": 26,
"start_line": 231
}
},
"new_content": "static_cast<unsigned>"
}
]
}
],
"location": {
"file_path": "third_party/blink/renderer/core/css/resolver/font_builder.h",
"gerrit_change_ref": {
"change": "6936189",
"host": "chromium-review.googlesource.com",
"patchset": "7",
"project": "chromium/src"
},
"range": {
"end_line": 231,
"start_line": 231
}
},
"message": "check: google-readability-casting\n\nC-style casts are discouraged; use static_cast (
https://clang.llvm.org/extra/clang-tidy/checks/google/readability-casting.html)\n\n(Note: You can add `Skip-Clang-Tidy-Checks: google-readability-casting` footer to the CL description to skip the check)",
"severity_level": "SEVERITY_LEVEL_WARNING"
},
{
"category": "clang-tidy",
"fixes": [
{
"replacements": [
{
"location": {
"file_path": "third_party/blink/renderer/core/css/css_rule.h",
"gerrit_change_ref": {
"change": "6936189",
"host": "chromium-review.googlesource.com",
"patchset": "7",
"project": "chromium/src"
},
"range": {
"end_column": 24,
"end_line": 50,
"start_column": 22,
"start_line": 50
}
},
"new_content": "= default;"
}
]
}
],
"location": {
"file_path": "third_party/blink/renderer/core/css/css_rule.h",
"gerrit_change_ref": {
"change": "6936189",
"host": "chromium-review.googlesource.com",
"patchset": "7",
"project": "chromium/src"
},
"range": {
"end_line": 50,
"start_line": 50
}
},
"message": "check: modernize-use-equals-default\n\nuse '= default' to define a trivial destructor (
https://clang.llvm.org/extra/clang-tidy/checks/modernize/use-equals-default.html)\n\n(Note: You can add `Skip-Clang-Tidy-Checks: modernize-use-equals-default` footer to the CL description to skip the check)",
"severity_level": "SEVERITY_LEVEL_WARNING"
},
{
"category": "clang-tidy",
"fixes": [
{
"replacements": [
{
"location": {
"file_path": "third_party/blink/renderer/core/css/css_rule.h",
"gerrit_change_ref": {
"change": "6936189",
"host": "chromium-review.googlesource.com",
"patchset": "7",
"project": "chromium/src"
},
"range": {
"end_column": 35,
"end_line": 158,
"start_column": 35,
"start_line": 158
}
},
"new_content": " = false"
}
]
}
],
"location": {
"file_path": "third_party/blink/renderer/core/css/css_rule.h",
"gerrit_change_ref": {
"change": "6936189",
"host": "chromium-review.googlesource.com",
"patchset": "7",
"project": "chromium/src"
},
"range": {
"end_line": 158,
"start_line": 158
}
},
"message": "check: modernize-use-default-member-init\n\nuse default member initializer for 'parent_is_rule_' (
https://clang.llvm.org/extra/clang-tidy/checks/modernize/use-default-member-init.html)\n\n(Note: You can add `Skip-Clang-Tidy-Checks: modernize-use-default-member-init` footer to the CL description to skip the check)",
"severity_level": "SEVERITY_LEVEL_WARNING"
},
{
"category": "clang-tidy",
"fixes": [
{
"replacements": [
{
"location": {
"file_path": "third_party/blink/renderer/core/css/css_rule.h",
"gerrit_change_ref": {
"change": "6936189",
"host": "chromium-review.googlesource.com",
"patchset": "7",
"project": "chromium/src"
},
"range": {
"end_column": 53,
"end_line": 157,
"start_column": 53,
"start_line": 157
}
},
"new_content": " = false"
}
]
}
],
"location": {
"file_path": "third_party/blink/renderer/core/css/css_rule.h",
"gerrit_change_ref": {
"change": "6936189",
"host": "chromium-review.googlesource.com",
"patchset": "7",
"project": "chromium/src"
},
"range": {
"end_line": 157,
"start_line": 157
}
},
"message": "check: modernize-use-default-member-init\n\nuse default member initializer for 'has_cached_selector_text_' (
https://clang.llvm.org/extra/clang-tidy/checks/modernize/use-default-member-init.html)\n\n(Note: You can add `Skip-Clang-Tidy-Checks: modernize-use-default-member-init` footer to the CL description to skip the check)",
"severity_level": "SEVERITY_LEVEL_WARNING"
},
{
"category": "clang-tidy",
"fixes": [
{
"replacements": [
{
"location": {
"file_path": "third_party/blink/renderer/core/css/resolver/match_result.h",
"gerrit_change_ref": {
"change": "6936189",
"host": "chromium-review.googlesource.com",
"patchset": "7",
"project": "chromium/src"
},
"range": {
"end_column": 18,
"end_line": 125,
"start_column": 16,
"start_line": 125
}
},
"new_content": "= default;"
}
]
}
],
"location": {
"file_path": "third_party/blink/renderer/core/css/resolver/match_result.h",
"gerrit_change_ref": {
"change": "6936189",
"host": "chromium-review.googlesource.com",
"patchset": "7",
"project": "chromium/src"
},
"range": {
"end_line": 125,
"start_line": 125
}
},
"message": "check: modernize-use-equals-default\n\nuse '= default' to define a trivial default constructor (
https://clang.llvm.org/extra/clang-tidy/checks/modernize/use-equals-default.html)\n\n(Note: You can add `Skip-Clang-Tidy-Checks: modernize-use-equals-default` footer to the CL description to skip the check)",
"severity_level": "SEVERITY_LEVEL_WARNING"
},
{
"category": "clang-tidy",
"fixes": [
{
"replacements": [
{
"location": {
"file_path": "third_party/blink/renderer/core/css/resolver/style_resolver.cc",
"gerrit_change_ref": {
"change": "6936189",
"host": "chromium-review.googlesource.com",
"patchset": "7",
"project": "chromium/src"
},
"range": {
"end_column": 34,
"end_line": 645,
"start_column": 32,
"start_line": 645
}
},
"new_content": "= default;"
}
]
}
],
"location": {
"file_path": "third_party/blink/renderer/core/css/resolver/style_resolver.cc",
"gerrit_change_ref": {
"change": "6936189",
"host": "chromium-review.googlesource.com",
"patchset": "7",
"project": "chromium/src"
},
"range": {
"end_line": 645,
"start_line": 645
}
},
"message": "check: modernize-use-equals-default\n\nuse '= default' to define a trivial destructor (
https://clang.llvm.org/extra/clang-tidy/checks/modernize/use-equals-default.html)\n\n(Note: You can add `Skip-Clang-Tidy-Checks: modernize-use-equals-default` footer to the CL description to skip the check)",
"severity_level": "SEVERITY_LEVEL_WARNING"
}
]
}