Page MenuHomePhabricator
People reviewbot

reviewbot (Code Review Bot)
User

Projects

User does not belong to any projects.

Calendar

Today

  • Clear sailing ahead.

Tomorrow

  • Clear sailing ahead.

Wednesday

  • Clear sailing ahead.

User Details

User Since
Mar 5 2018, 2:16 PM (169 w, 2 h)
Availability
Available

If you have any questions, please contact :bastien (babadie@mozilla.com) and :marco (mcastelluccio@mozilla.com).

Recent Activity
View All

Today

reviewbot added a comment to D116357: Bug 1703812 - Part 12 - Add logging in failure paths in FFmpegAudioDecoder.cpp r?bryce.

Code analysis found 151 defects in the diff 444866:

  • 1 defect found by clang-format
  • 150 build errors found by clang-tidy
Mon, May 31, 3:01 PM
reviewbot added a comment to D111865: Bug 1703812 - Part 13 - When no explicit encoder delay is found, use the implicit encoder delay of 2112 frames. r?bryce.

Code analysis found 161 defects in the diff 444867:

  • 161 build errors found by clang-tidy
Mon, May 31, 2:56 PM
reviewbot added a comment to D116356: Bug 1703812 - Part 11 - Put the test file name in the assertion message in test_decoderDelay.html r?bryce.

Code analysis found 159 defects in the diff 444865:

  • 159 build errors found by clang-tidy
Mon, May 31, 2:54 PM
reviewbot added a comment to D111866: Bug 1703812 - Part 10 - Don't spam the log with AudioTrimmer log when activating PlatformDecoderModule:4. r?bryce.

Code analysis found 153 defects in the diff 444864:

  • 153 build errors found by clang-tidy
Mon, May 31, 2:53 PM · testing-exception-unchanged (Doesn't change behavior for users)
reviewbot added a comment to D116386: Bug 1713610: Require PyPI-vendored packages be added to requirements.in.

You have touched the documentation in diff 444868, you can find it rendered here for a week.

Mon, May 31, 2:51 PM · testing-exception-unchanged (Doesn't change behavior for users)
reviewbot added a comment to D113118: WIP: Bug 1703812 - Part 9 - Don't require that when decoding is a success, an output must be received..

Code analysis found 158 defects in the diff 444863:

  • 158 build errors found by clang-tidy
Mon, May 31, 2:48 PM
reviewbot added a comment to D113117: WIP: Bug 1703812 - Part 8 - Trim the encoder delay and padding of AAC files when decoding using WMF..

Code analysis found 153 defects in the diff 444862:

  • 153 build errors found by clang-tidy
Mon, May 31, 2:46 PM
reviewbot added a comment to D116355: Bug 1703812 - Part 7 - Change the way decoder delay is trimmed when using ffmpeg to decode audio to be more compatible with looping files. r?bryce.

Code analysis found 162 defects in the diff 444861:

  • 1 defect found by clang-format
  • 161 build errors found by clang-tidy
Mon, May 31, 2:44 PM
reviewbot added a comment to D111823: Bug 1703812 - Part 6 - Test that the decoder delay and the padding are trimmed correctly with various test files encoded with ffmpeg. r?bryce.

Code analysis found 159 defects in the diff 444860:

  • 159 build errors found by clang-tidy
Mon, May 31, 2:43 PM · testing-approved
reviewbot added a comment to D116386: Bug 1713610: Require PyPI-vendored packages be added to requirements.in.

You have touched the documentation in diff 444854, you can find it rendered here for a week.

Mon, May 31, 2:34 PM · testing-exception-unchanged (Doesn't change behavior for users)
reviewbot added a comment to D111822: WIP: Bug 1703812 - Part 5 - Trim the decoder delay and the padding when decoding AAC using the AT decoder..

Code analysis found 147 defects in the diff 444859:

  • 1 defect found by clang-format
  • 146 build errors found by clang-tidy
Mon, May 31, 2:33 PM
reviewbot added a comment to D111821: Bug 1703812 - Part 4 - Add logging statement in error path in ffmpeg init code. r?bryce.

Code analysis found 149 defects in the diff 444858:

  • 149 build errors found by clang-tidy
Mon, May 31, 2:29 PM · testing-exception-unchanged (Doesn't change behavior for users)
reviewbot added a comment to D111820: Bug 1703812 - Part 3 - Trim decoder delay and padding when ffmpeg is decoding AAC. r?bryce.

Code analysis found 138 defects in the diff 444857:

  • 1 defect found by clang-format
  • 136 build errors and 1 defect found by clang-tidy
Mon, May 31, 2:26 PM · testing-exception-elsewhere (Please comment explaining where the tests are)
reviewbot added a comment to D111819: Bug 1703812 - Part 2 - Find the decoder delay and the total media duration when demuxing an mp4 file that contains an AAC track, and pass that to the codec implementation. r?bryce.

Code analysis found 3 defects in the diff 444856:

  • 3 defects found by clang-tidy
Mon, May 31, 2:20 PM · testing-exception-elsewhere (Please comment explaining where the tests are)
reviewbot added a comment to D115188: Bug 1675753 Schedule high value mobile tests against Samsung Galaxy S7 in CI r=#perftest.

You have touched the documentation in diff 444850, you can find it rendered here for a week.

Mon, May 31, 2:16 PM
reviewbot added a comment to D116384: No bug: Add helpful suggestion to ./mach taskgraph --diff for some circumstances r=ahal.

Code analysis found 1 defect in the diff 444852:

  • 1 defect found by py-flake8 (Mozlint)
Mon, May 31, 2:09 PM
reviewbot added a comment to D115509: Bug 1710317 - Part 2: Add mButton[s] to WidgetTouchEvent r=aklotz,edgar.

Code analysis found 2 defects in the diff 444840:

  • 2 defects found by clang-tidy
Mon, May 31, 1:58 PM · testing-exception-unchanged (Doesn't change behavior for users)
reviewbot added a comment to D116383: Bug 1713607 - [devtools] ensure emitting DOCUMENT_EVENT will-navigate in case of bfcache navigations..

Code analysis found 6 defects in the diff 444841:

  • 6 defects found by eslint (Mozlint)
Mon, May 31, 1:55 PM
reviewbot added a comment to D115188: Bug 1675753 Schedule high value mobile tests against Samsung Galaxy S7 in CI r=#perftest.

Code analysis found 8 defects in the diff 444845:

  • 8 defects found by perfdocs-verify (Mozlint)
Mon, May 31, 1:50 PM
reviewbot added a comment to D114498: Bug 1693486 - Implement new event extra API. r?chutten.

You have touched the documentation in diff 444834, you can find it rendered here for a week.

Mon, May 31, 1:48 PM
reviewbot added a comment to D113273: Bug 1707489 - Retrieve entry registers and stop areas from JIT - r?canaltinova!.

Code analysis found 2 defects in the diff 444830:

  • 2 build errors found by clang-tidy
Mon, May 31, 1:42 PM · testing-exception-other (Please leave a comment explaining why)
reviewbot added a comment to D114498: Bug 1693486 - Implement new event extra API. r?chutten.

Code analysis found 2 defects in the diff 444824:

  • 2 build errors found by clang-tidy
Mon, May 31, 1:36 PM
reviewbot added a comment to D116377: Bug 1693486 - Update glean-parser to v3.4.0. r?chutten.

You have touched the documentation in diff 444823, you can find it rendered here for a week.

Mon, May 31, 1:36 PM
reviewbot added a comment to D111865: Bug 1703812 - Part 13 - When no explicit encoder delay is found, use the implicit encoder delay of 2112 frames. r?bryce.

Code analysis found 147 defects in the diff 444805:

  • 147 build errors found by clang-tidy
Mon, May 31, 12:56 PM
reviewbot added a comment to D116370: Bug 1713589. If double tap zoom can't find a rect to zoom in to and we are zoomed out then zoom in some small default amount. r?botond.

Code analysis found 1 defect in the diff 444789:

  • 1 defect found by code coverage analysis
Mon, May 31, 12:11 PM
reviewbot added a comment to D112787: WIP: Bug 1704115 - Always sniff to determine the MIME type when attempting to play a media file..

Code analysis found 1 defect in the diff 444792:

  • 1 defect found by clang-format
Mon, May 31, 12:07 PM
reviewbot added a comment to D116367: Bug 1713584. In calculating if a rect is zoomed in shift the rect and the composited area to a common origin. r?botond,emilio.

Code analysis found 1 defect in the diff 444781:

  • 1 defect found by code coverage analysis
Mon, May 31, 11:54 AM
reviewbot added a comment to D106412: WIP: Bug 1432901 - (WIP - updated prototype) Support ES Modules in ChromeUtils.import.

Code analysis found 1 defect in the diff 444758:

  • 1 defect found by clang-tidy
Mon, May 31, 10:48 AM
reviewbot added a comment to D116355: Bug 1703812 - Part 7 - Change the way decoder delay is trimmed when using ffmpeg to decode audio to be more compatible with looping files. r?bryce.

Code analysis found 161 defects in the diff 444729:

  • 1 defect found by clang-format
  • 160 build errors found by clang-tidy
Mon, May 31, 9:26 AM
reviewbot added a comment to D111823: Bug 1703812 - Part 6 - Test that the decoder delay and the padding are trimmed correctly with various test files encoded with ffmpeg. r?bryce.

Code analysis found 145 defects in the diff 444728:

  • 145 build errors found by clang-tidy
Mon, May 31, 9:23 AM · testing-approved
reviewbot added a comment to D111822: WIP: Bug 1703812 - Part 5 - Trim the decoder delay and the padding when decoding AAC using the AT decoder..

Code analysis found 160 defects in the diff 444727:

  • 1 defect found by clang-format
  • 159 build errors found by clang-tidy
Mon, May 31, 9:16 AM
reviewbot added a comment to D111820: Bug 1703812 - Part 3 - Trim decoder delay and padding when ffmpeg is decoding AAC. r?bryce.

Code analysis found 142 defects in the diff 444725:

  • 1 defect found by clang-format
  • 140 build errors and 1 defect found by clang-tidy
Mon, May 31, 9:15 AM · testing-exception-elsewhere (Please comment explaining where the tests are)
reviewbot added a comment to D112787: WIP: Bug 1704115 - Always sniff to determine the MIME type when attempting to play a media file..

Code analysis found 1 defect in the diff 444731:

  • 1 defect found by clang-format
Mon, May 31, 9:14 AM
reviewbot added a comment to D111821: Bug 1703812 - Part 4 - Add logging statement in error path in ffmpeg init code. r?bryce.

Code analysis found 158 defects in the diff 444726:

  • 158 build errors found by clang-tidy
Mon, May 31, 9:13 AM · testing-exception-unchanged (Doesn't change behavior for users)
reviewbot added a comment to D111819: Bug 1703812 - Part 2 - Find the decoder delay and the total media duration when demuxing an mp4 file that contains an AAC track, and pass that to the codec implementation. r?bryce.

Code analysis found 3 defects in the diff 444724:

  • 3 defects found by clang-tidy
Mon, May 31, 9:04 AM · testing-exception-elsewhere (Please comment explaining where the tests are)
reviewbot added a comment to D113119: ConnectionEntry should hold reference to DnsAndConnectSocket-s.

Code analysis found 3 defects in the diff 444719:

  • 3 defects found by clang-format
Mon, May 31, 9:00 AM · testing-exception-unchanged (Doesn't change behavior for users)
reviewbot added a comment to D116337: Bug 1713271 - Move initial maps code to a new class r=glandium.

Code analysis found 1 defect in the diff 444678:

  • 1 defect found by clang-tidy
Mon, May 31, 7:48 AM
reviewbot added a comment to D116338: Bug 1713271 - Provide our own isspace implementation r=glandium.

The analysis task source-test-coverity-coverity failed, but we could not detect any issue.
Please check this task manually.

Mon, May 31, 7:47 AM
reviewbot added a comment to D116339: Bug 1713271 - Capture the initial maps as jemalloc starts r=glandium.

Code analysis found 1 defect in the diff 444673:

  • 1 defect found by clang-format
Mon, May 31, 6:26 AM
reviewbot added a comment to D116337: Bug 1713271 - Move initial maps code to a new class r=glandium.

Code analysis found 3 defects in the diff 444671:

  • 3 defects found by clang-tidy
Mon, May 31, 6:21 AM
reviewbot added a comment to D115754: Bug 1711878 - Remove unused JSContext arguments from PrintErrorImpl and PrintSingleError. r?mgaudet.

Code analysis found 2 defects in the diff 444649:

  • 2 defects found by clang-format
Mon, May 31, 2:41 AM

Yesterday

reviewbot added a comment to D116322: Bug 1713485 - Remove code for completed migration. r?botond.

Code analysis found 1 defect in the diff 444619:

  • 1 defect found by clang-format
Sun, May 30, 2:28 PM · testing-exception-unchanged (Doesn't change behavior for users)
reviewbot added a comment to D116317: Bug 1696093 - AVIF image transform support. r=aosmond.

Code analysis found 5 defects in the diff 444581:

  • 1 defect found by clang-format
  • 4 defects found by Coverity
Sun, May 30, 1:25 AM
reviewbot added a comment to D116317: Bug 1696093 - AVIF image transform support. r=aosmond.

Code analysis found 5 defects in the diff 444580:

  • 1 defect found by clang-format
  • 4 defects found by Coverity
Sun, May 30, 1:22 AM

Sat, May 29

reviewbot added a comment to D116315: Bug 1713468 - Support RenderCompositorEGL on X11-only builds, r=stransky.

Code analysis found 1 defect in the diff 444571:

  • 1 defect found by code coverage analysis
Sat, May 29, 8:44 PM
reviewbot added a comment to D113702: Bug 1325335 - Adding Ed25519 implementation..

Code analysis found 1 defect in the diff 444563:

  • 1 defect found by Coverity
Sat, May 29, 4:33 PM
reviewbot added a comment to D116301: Bug 1713409 - Always support a single update rect for SwCompositor. r?gw.

Code analysis found 1 defect in the diff 444544:

  • 1 defect found by code coverage analysis
Sat, May 29, 3:10 AM · testing-exception-unchanged (Doesn't change behavior for users)
reviewbot added a comment to D116302: Bug 1340422 - Part 3-1: Update getPathSegListAtLength..

Code analysis found 6 defects in the diff 444538:

  • 1 defect found by clang-format
  • 5 defects found by eslint (Mozlint)
Sat, May 29, 2:05 AM · testing-approved
reviewbot added a comment to D116310: Bug 1713404 - Only apply unwriteable margins to the total width of a page..

Code analysis found 1 defect in the diff 444535:

  • 1 defect found by clang-format
Sat, May 29, 2:00 AM
reviewbot added a comment to D116309: Bug 1624792: Remove Debugger interface r=tcampbell!.

You have touched the documentation in diff 444534, you can find it rendered here for a week.

Sat, May 29, 1:57 AM · testing-exception-other (Please leave a comment explaining why)
reviewbot added a comment to D115975: Bug 1712408 - Move results composition doc into mozilla-central address bar docs..

You have touched the documentation in diff 444527, you can find it rendered here for a week.

Sat, May 29, 1:29 AM · testing-exception-unchanged (Doesn't change behavior for users)
reviewbot added a comment to D116190: Bug 1712491 - Avoid re-recording parts of blob images which haven't changed..

Code analysis found 2 defects in the diff 444519:

  • 1 defect found by clang-tidy
  • 1 defect found by code coverage analysis
Sat, May 29, 12:18 AM · testing-exception-other (Please leave a comment explaining why)
reviewbot added a comment to D116190: Bug 1712491 - Avoid re-recording parts of blob images which haven't changed..

Code analysis found 2 defects in the diff 444516:

  • 1 defect found by clang-tidy
  • 1 defect found by code coverage analysis
Sat, May 29, 12:16 AM · testing-exception-other (Please leave a comment explaining why)

Fri, May 28

reviewbot added a comment to D116301: Bug 1713409 - Always support a single update rect for SwCompositor. r?gw.

Code analysis found 1 defect in the diff 444512:

  • 1 defect found by code coverage analysis
Fri, May 28, 11:42 PM · testing-exception-unchanged (Doesn't change behavior for users)
reviewbot added a comment to D113065: Bug 1524662 - Add --enable-path-remapping producing compiled objects with generic paths. r?#build.

Code analysis found 2 defects in the diff 444510:

  • 2 defects found by py-black (Mozlint)
Fri, May 28, 11:20 PM
reviewbot added a comment to D115246: Bug 1710316 - Pass modifiers in DispatchTouchEventFromWMPointer r=aklotz.

Code analysis found 1 defect in the diff 444507:

  • 1 defect found by code coverage analysis
Fri, May 28, 10:57 PM · testing-exception-other (Please leave a comment explaining why)
reviewbot added a comment to D116296: Bug 1712874 - Back out bug 1704080 (ability to overscroll is non-scrollable directions). r=tnikkel,hiro.

Code analysis found 11 defects in the diff 444489:

  • 11 build errors found by clang-tidy
Fri, May 28, 10:37 PM · testing-approved
reviewbot added a comment to D115656: Bug 1712204 - Pass through requested job count from `mach hazards gather -jN`, and set a better default.

Code analysis found 1 defect in the diff 444485:

  • 1 defect found by py-black (Mozlint)
Fri, May 28, 10:26 PM · needs-testing-tag
reviewbot added a comment to D116293: Bug 1712754 - Do not autogrant permissions in GV push notification code. r=aklotz!.

Code analysis found 1 defect in the diff 444451:

  • 1 defect found by eslint (Mozlint)
Fri, May 28, 10:10 PM · testing-exception-unchanged (Doesn't change behavior for users)
reviewbot added a comment to D105391: Bug 1672317 - [l10nfilesource] part4: Fix preferences browser_languages_subdialog.js text to be not racy..

You have touched the documentation in diff 444423, you can find it rendered here for a week.

Fri, May 28, 9:20 PM · testing-approved
reviewbot added a comment to D103002: Bug 1672317 - [l10nfilesource] part3: Introduce L10nFileSource backed by l10nregistry-rs..

You have touched the documentation in diff 444422, you can find it rendered here for a week.

Fri, May 28, 9:15 PM · testing-exception-elsewhere (Please comment explaining where the tests are)
reviewbot added a comment to D102102: Bug 1672317 - [l10nfilesource] part2: Vendor in l10nregistry-rs and fluent-fallback..

You have touched the documentation in diff 444421, you can find it rendered here for a week.

Fri, May 28, 9:11 PM · testing-approved
reviewbot added a comment to D103184: Bug 1672317 - [l10nfilesource] part1: Update fluent-rs to 0.15.0..

You have touched the documentation in diff 444420, you can find it rendered here for a week.

Fri, May 28, 9:10 PM · testing-approved
reviewbot added a comment to D115593: Bug 1651133 - Double-check the build ID to avoid spurious about:restartrequired r?jld,glandium.

Code analysis found 1 defect in the diff 444384:

  • 1 defect found by clang-tidy
Fri, May 28, 7:35 PM
reviewbot added a comment to D115596: Bug 1711479 - Implement CSS support for the optional adjustment-basis metric keywords for the font-size-adjust property (enabled on Nightly only for now). r=#firefox-style-system-reviewers.

Code analysis found 2 defects in the diff 444362:

  • 1 defect found by clang-format
  • 1 defect found by clang-tidy
Fri, May 28, 7:03 PM · testing-exception-elsewhere (Please comment explaining where the tests are)
reviewbot added a comment to D116281: Bug 1713122 - Update Telemetry products value documentation r?TravisLong!.

You have touched the documentation in diff 444365, you can find it rendered here for a week.

Fri, May 28, 6:52 PM
reviewbot added a comment to D116085: Bug 1711136 Add CNN desktop page for arewefastyet/raptor browsertime -- live site test r=#perftest.

You have touched the documentation in diff 444336, you can find it rendered here for a week.

Fri, May 28, 5:41 PM · testing-exception-unchanged (Doesn't change behavior for users)
reviewbot added a comment to D116270: Bug 1713350 - Ensure we reset the dirty flag for SVG blob recordings during moves..

Code analysis found 1 defect in the diff 444338:

  • 1 defect found by code coverage analysis
Fri, May 28, 5:31 PM · testing-exception-other (Please leave a comment explaining why)
reviewbot added a comment to D116271: Bug 1681769: Set UiCompositorControllerParent's "other" pid when initializing for in-process; r=botond.

Code analysis found 1 defect in the diff 444339:

  • 1 defect found by code coverage analysis
Fri, May 28, 5:28 PM · testing-exception-unchanged (Doesn't change behavior for users)
reviewbot added a comment to D116260: Bug 1713343 - Land strings for CFR Full Video Support message.

Code analysis found 3 defects in the diff 444310:

  • 3 defects found by l10n-conflicts (Mozlint)
Fri, May 28, 4:31 PM · testing-exception-ui (Only changes ui styling, images, or localized strings)
reviewbot added a comment to D110681: Bug 1702761 - Basic implementation of CallMonitor r?ckerschb.

Code analysis found 1 defect in the diff 444248:

  • 1 defect found by file-whitespace (Mozlint)
Fri, May 28, 2:36 PM
reviewbot added a comment to D112434: Bug 1696251 - Refactor constructors in MachCommandBase subclasses to remove them. r=mhentges.

You have touched the documentation in diff 444245, you can find it rendered here for a week.

Fri, May 28, 2:11 PM · testing-exception-unchanged (Doesn't change behavior for users)
reviewbot added a comment to D116008: Bug 1693575 - Part 3: Implement support for "calendar" option. r=tcampbell!.

Code analysis found 1 defect in the diff 444240:

  • 1 defect found by clang-tidy
Fri, May 28, 1:58 PM · testing-approved
reviewbot added a comment to D110681: Bug 1702761 - Basic implementation of CallMonitor r?ckerschb.

Code analysis found 1 defect in the diff 444236:

  • 1 defect found by file-whitespace (Mozlint)
Fri, May 28, 1:31 PM
reviewbot added a comment to D115997: Bug 1710933 - Save LaunchWithApplication downloads into downloads folder for windows and unix r?mtigley.

Code analysis found 1 defect in the diff 444237:

  • 1 defect found by test-manifest (Mozlint)
Fri, May 28, 1:22 PM
reviewbot added a comment to D116190: Bug 1712491 - Avoid re-recording parts of blob images which haven't changed..

Code analysis found 2 defects in the diff 444232:

  • 1 defect found by clang-tidy
  • 1 defect found by code coverage analysis
Fri, May 28, 1:10 PM · testing-exception-other (Please leave a comment explaining why)
reviewbot added a comment to D115934: Bug 1712803 - Ensure external surface lifetimes match that of their referencing blob..

Code analysis found 4 defects in the diff 444233:

  • 4 defects found by code coverage analysis
Fri, May 28, 1:07 PM
reviewbot added a comment to D115934: Bug 1712803 - Ensure external surface lifetimes match that of their referencing blob..

Code analysis found 6 defects in the diff 444231:

  • 1 build error found by clang-tidy
  • 1 defect found by clang-format
  • 4 defects found by code coverage analysis
Fri, May 28, 1:04 PM
reviewbot added a comment to D116190: Bug 1712491 - Avoid re-recording parts of blob images which haven't changed..

Code analysis found 2 defects in the diff 444230:

  • 1 defect found by clang-tidy
  • 1 defect found by code coverage analysis
Fri, May 28, 1:04 PM · testing-exception-other (Please leave a comment explaining why)
reviewbot added a comment to D84684: Bug 1682632 - part2.3: ExtensionEventListener callListener promised return value..

Code analysis found 2 defects in the diff 444225:

  • 2 build errors found by clang-tidy
Fri, May 28, 12:58 PM · testing-exception-elsewhere (Please comment explaining where the tests are)
reviewbot added a comment to D115593: Bug 1651133 - Double-check the build ID to avoid spurious about:restartrequired r?jld,glandium.

Code analysis found 9 defects in the diff 444211:

  • 1 defect found by clang-format
  • 1 defect found by clang-tidy
  • 7 defects found by eslint (Mozlint)
Fri, May 28, 12:04 PM
reviewbot added a comment to D115987: Bug 1710045 - Static analysis auto fixes r=#necko.

Code analysis found 1 defect in the diff 444167:

  • 1 defect found by code coverage analysis
Fri, May 28, 10:07 AM · testing-exception-unchanged (Doesn't change behavior for users)
reviewbot added a comment to D116126: Bug 1713082 - Add APIs to get helper thread system thread and CPU counts r?sfink.

Code analysis found 2 defects in the diff 444154:

  • 2 defects found by clang-tidy
Fri, May 28, 9:49 AM · testing-exception-unchanged (Doesn't change behavior for users)
reviewbot added a comment to D116161: Bug 1712701 - Assert hasValue in PropertyDescriptor before allowing access to the value. r?jandem.

Code analysis found 1 defect in the diff 444155:

  • 1 defect found by code coverage analysis
Fri, May 28, 9:48 AM · testing-exception-unchanged (Doesn't change behavior for users)
reviewbot added a comment to D116085: Bug 1711136 Add CNN desktop page for arewefastyet/raptor browsertime -- live site test r=#perftest.

You have touched the documentation in diff 444141, you can find it rendered here for a week.

Fri, May 28, 8:56 AM · testing-exception-unchanged (Doesn't change behavior for users)
reviewbot added a comment to D108635: Bug 1616239 - Enable http3 tests for asan, r=#necko.

Code analysis found 15 defects in the diff 444129:

  • 15 defects found by test-manifest (Mozlint)
Fri, May 28, 8:10 AM · testing-exception-unchanged (Doesn't change behavior for users)
reviewbot added a comment to D116209: WIP: Bug 1709136 - Remove the explicit remoteness switch that happens during the restore process.

Code analysis found 8 defects in the diff 444127:

  • 1 defect found by clang-format
  • 7 defects found by eslint (Mozlint)
Fri, May 28, 7:29 AM
reviewbot added a comment to D116208: Bug 1710381 - Stop using this argument in ChromeUtils.import for importing modules in toolkit/components/telemetry/tests/unit/. r?Standard8.

Code analysis found 1 defect in the diff 444126:

  • 1 defect found by eslint (Mozlint)
Fri, May 28, 7:24 AM
reviewbot added a comment to D116197: Bug 1695116 part 7: Support MSAA accParent, accChildCount and accNavigate for RemoteAccessibles..

Code analysis found 2 defects in the diff 444124:

  • 2 defects found by code coverage analysis
Fri, May 28, 7:21 AM
reviewbot added a comment to D116196: Bug 1695116 part 6: Support retrieval of remote MsaaAccessibles via MSAA child id..

Code analysis found 1 defect in the diff 444120:

  • 1 defect found by code coverage analysis
Fri, May 28, 7:07 AM
reviewbot added a comment to D116206: Bug 1695116 part 11: When the cache is enabled, don't generate an MSAA id for each content process..

Code analysis found 1 defect in the diff 444119:

  • 1 defect found by code coverage analysis
Fri, May 28, 6:41 AM
reviewbot added a comment to D115783: WIP: Bug 1704509 - Add Unified PuralRules Impl For SpiderMonkey ?dminor!,gregtatum,anba.

Code analysis found 1 defect in the diff 444108:

  • 1 defect found by clang-tidy
Fri, May 28, 6:03 AM
reviewbot added a comment to D115783: WIP: Bug 1704509 - Add Unified PuralRules Impl For SpiderMonkey ?dminor!,gregtatum,anba.

Code analysis found 6 defects in the diff 444106:

  • 6 defects found by clang-tidy
Fri, May 28, 5:18 AM
reviewbot added a comment to D116196: Bug 1695116 part 6: Support retrieval of remote MsaaAccessibles via MSAA child id..

Code analysis found 1 defect in the diff 444103:

  • 1 defect found by code coverage analysis
Fri, May 28, 5:01 AM
reviewbot added a comment to D116199: Bug 1695116 part 9: Fail gracefully for MSAA methods which don't support RemoteAccessible yet instead of crashing..

Code analysis found 2 defects in the diff 444083:

  • 2 defects found by code coverage analysis
Fri, May 28, 4:14 AM
reviewbot added a comment to D116196: Bug 1695116 part 6: Support retrieval of remote MsaaAccessibles via MSAA child id..

Code analysis found 2 defects in the diff 444080:

  • 1 defect found by clang-format
  • 1 defect found by code coverage analysis
Fri, May 28, 4:13 AM
reviewbot added a comment to D116193: Bug 1695116 part 3: Id management for remote MsaaAccessibles with the cache enabled..

Code analysis found 1 defect in the diff 444077:

  • 1 defect found by code coverage analysis
Fri, May 28, 4:13 AM
reviewbot added a comment to D116198: Bug 1695116 part 8: Support MSAA accRole for RemoteAccessibles..

Code analysis found 1 defect in the diff 444082:

  • 1 defect found by code coverage analysis
Fri, May 28, 4:12 AM
reviewbot added a comment to D116194: Bug 1695116 part 4: Support events for RemoteAccessibles with the cache enabled..

Code analysis found 3 defects in the diff 444078:

  • 3 defects found by code coverage analysis
Fri, May 28, 4:12 AM
reviewbot added a comment to D116195: Bug 1695116 part 5: Initial support for RemoteAccessibles in QueryInterface..

Code analysis found 2 defects in the diff 444079:

  • 2 defects found by code coverage analysis
Fri, May 28, 4:10 AM
Privacy · Cookies · Legal