0
Commit Graph

19 Commits

Author SHA1 Message Date
Jonathan Lee
e95877fe72 [wptrunner] Add run_wpt_tests.py support to debug_renderer
Add support for interactively debugging WPTs with upstream's `wpt run`.
`run_wpt_tests.py --verbose` ensures browser logs are dumped to stdout
so that `debug_renderer` can find the PID to attach to.

Bug: 1440021
Test: ./debug_renderer ./run_wpt_tests.py -t Default
Test: external/wpt/badging/badge-success.https.html
Change-Id: I58323aea67f6232bc28b198b8f18d3d7bb7d8a7b
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/4893148
Reviewed-by: Weizhong Xia <weizhong@google.com>
Commit-Queue: Jonathan Lee <jonathanjlee@google.com>
Reviewed-by: Robert Flack <flackr@chromium.org>
Cr-Commit-Position: refs/heads/main@{#1201810}
2023-09-26 21:16:30 +00:00
Taiyo Mizuhashi
d0f90c9d31 doc: Fix link to gdbinit.md & lldbinit.md on linux/debugging.md
- Fix the broken link to gdbinit.md & lldbinit.md on linux/debugging.md.
- Additionally, replace the absolute link to gdbinit.md with a relative
  link to maintain notation consistency with the other links in this
  doc.

Bug: n/a
Change-Id: Idc275f63de05264b66d0fbe022f26dc0e0b63599
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/4573957
Auto-Submit: Taiyo Mizuhashi <taiyo@chromium.org>
Commit-Queue: Hiroki Nakagawa <nhiroki@chromium.org>
Reviewed-by: Hiroki Nakagawa <nhiroki@chromium.org>
Cr-Commit-Position: refs/heads/main@{#1150998}
2023-05-31 00:55:23 +00:00
Brett Wilson
41a7de0afb Clarify debugging docs.
Add a section at the top for build configuration. Move the gdbinit
instructions to the configuration section (it's required before
debugging) and remove the details about why this is required (it's
described on the dedicated page that's linked). Add a link to the
corresponding LLDB page.

Mention flag --gtest_break_on_failure for debugging tests.

Clarify LLDB install (the page used to imply that you had to run apt-get
to get the lldbinit file).

Change-Id: Ib292350ee521bf2bc3c4a836523fe070e4d7694e
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/4482817
Reviewed-by: Yuzhu Shen <yzshen@chromium.org>
Commit-Queue: Brett Wilson <brettw@chromium.org>
Cr-Commit-Position: refs/heads/main@{#1136698}
2023-04-27 18:56:54 +00:00
Robert Flack
6bbefe7e7e Add support for running web tests with debug_renderer
If debug_renderer detects that you're running web tests, modifies the
command line flags and adds appropriate run_web_tests flags for
debugging the renderer spawned for individual web tests.

Bug: None
Change-Id: I85980ff57c85738f8ad8d47e83ba9d61d5411e09
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/4198115
Commit-Queue: Robert Flack <flackr@chromium.org>
Reviewed-by: Xianzhu Wang <wangxianzhu@chromium.org>
Cr-Commit-Position: refs/heads/main@{#1109686}
2023-02-24 18:49:51 +00:00
Robert Flack
cdbf8c4a02 Allow debugging the navigation of a renderer by URL
This adds the --wait-for-debugger-on-navigation flag which pauses
renderers when the target URL is known displaying a message for
each. The debug_renderer script is also updated to automatically
debug the site passed and unblock all other renderer processes.

Documentation in linux/debugging.md is updated to explain how
to use the updated script and flag.

Bug: 730065
Change-Id: I758bf6d10ba9b6a846d216987c20053d4c5ac240
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/4028699
Reviewed-by: Daniel Cheng <dcheng@chromium.org>
Reviewed-by: John Abd-El-Malek <jam@chromium.org>
Commit-Queue: Robert Flack <flackr@chromium.org>
Reviewed-by: Stephen McGruer <smcgruer@chromium.org>
Cr-Commit-Position: refs/heads/main@{#1073426}
2022-11-18 18:12:34 +00:00
Joshua Pawlicki
c2464187d5 Linux: Document symbol file availability.
Change-Id: I3128a25648f97e7c9e53ebf1198d6b01416f82d3
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/3484956
Auto-Submit: Joshua Pawlicki <waffles@chromium.org>
Reviewed-by: Chris Hamilton <chrisha@chromium.org>
Commit-Queue: Chris Hamilton <chrisha@chromium.org>
Cr-Commit-Position: refs/heads/main@{#974159}
2022-02-23 14:27:51 +00:00
Steve Kobes
8ce3e44def Light cleanup of rr docs in debugging.md.
Remove some flags like --no-sandbox that are no longer necessary, and
add links to recent demos.

Change-Id: I5c44047bb1b58ea65231628b14b171073a4b8ba2
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/3424982
Reviewed-by: David Baron <dbaron@chromium.org>
Commit-Queue: Steve Kobes <skobes@chromium.org>
Cr-Commit-Position: refs/heads/main@{#964814}
2022-01-28 22:36:59 +00:00
sisidovski
f270241c38 Fix 404 links
Change-Id: I0dc3fb0f1dc8ddbf27a0c3644fb5c26b822d3c87
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/3066824
Reviewed-by: Takuto Ikuta <tikuta@chromium.org>
Reviewed-by: Kouhei Ueno <kouhei@chromium.org>
Reviewed-by: Kinuko Yasuda <kinuko@chromium.org>
Commit-Queue: Shunya Shishido <sisidovski@google.com>
Cr-Commit-Position: refs/heads/master@{#908361}
2021-08-04 07:07:44 +00:00
John Palmer
046f987e02 [COIL] Change chromium code search type links to use main rather than master for docs
See this doc: go/chromium-coil-change for more info
    BUG=1210385

Change-Id: I6cd8548301e360bae8e71384dbd8ef11aa6c985f
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2902904
Reviewed-by: My Nguyen <myy@chromium.org>
Commit-Queue: My Nguyen <myy@chromium.org>
Auto-Submit: John Palmer <jopalmer@chromium.org>
Cr-Commit-Position: refs/heads/master@{#885851}
2021-05-24 01:24:56 +00:00
Alex Gough
2092674e7b Allow selection of utility sub type in utility-startup-dialog
This CL enhances the --utility-startup-dialog switch to now take an
optional service sub type, and displays the subtype when waiting
for the debugger, or showing the dialog.

`chrome --utility-startup-dialog` works as before, prompting/pausing
for every utility process.

`chrome --utility-startup-dialog=data_decoder.mojom.DataDecoderService`
only prompts for the data decoder service.

We check for the switch first then interrogate its value to limit
any impact on normal startup time.

This should make it easier to debug specific utility processes.

Tests: manual tested on Windows & Linux
Change-Id: I0195432a9f826697a87c687406a30dbdf1bfa378
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2892176
Reviewed-by: Alex Moshchuk <alexmos@chromium.org>
Commit-Queue: Alex Gough <ajgo@chromium.org>
Cr-Commit-Position: refs/heads/master@{#882668}
2021-05-13 20:11:30 +00:00
Zhang Hao
a9522488ac fix a command line error in debugging.md
Change-Id: I7a3a35349fdb44a83616b385d1ce43739da4faf1
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2881748
Reviewed-by: Yaron Friedman <yfriedman@chromium.org>
Auto-Submit: Zhang Hao <15686357310a@gmail.com>
Commit-Queue: Pavol Marko <pmarko@chromium.org>
Cr-Commit-Position: refs/heads/master@{#881970}
2021-05-12 11:26:36 +00:00
L. David Baron
e99d91eb8d Update documentation on using rr.
This makes a few updates to the documentation on using the rr debugger:

 * clarify recommendation to build from source into one about using a
   recent version (which might mean building from source), and explain
   why

 * mention both rr replay -p and -f, and explain the difference

 * remove reference to a year-old bug that's now subsumed by the advice
   on using a recent version

 * add advice on workarounds for calling functions that use LOG() from
   gdb

 * link to relevant Chromium Chronicle episode

Change-Id: I13206a009103ae12f83a3890101a91e02fd78a0a
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2794877
Reviewed-by: Christian Biesinger <cbiesinger@chromium.org>
Commit-Queue: David Baron <dbaron@chromium.org>
Cr-Commit-Position: refs/heads/master@{#867753}
2021-03-30 20:18:00 +00:00
Takuto Ikuta
a093dc26b5 [docs] small fix for debugging.md
use_debug_fission=false doesn't split debug info.

Change-Id: Ib8def944dfcea1345f41284e9d125c082071848c
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2526826
Reviewed-by: Minoru Chikamune <chikamune@chromium.org>
Commit-Queue: Takuto Ikuta <tikuta@chromium.org>
Auto-Submit: Takuto Ikuta <tikuta@chromium.org>
Cr-Commit-Position: refs/heads/master@{#826106}
2020-11-11 03:17:50 +00:00
David Benjamin
e7b279b503 Update instructions for rr and MADV_WIPEONFORK
Bug: 1082304
Change-Id: Ie8acbf67b4304dc67f7c8883b2a418c383b046d1
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2216596
Auto-Submit: David Benjamin <davidben@chromium.org>
Reviewed-by: Robert Flack <flackr@chromium.org>
Commit-Queue: Robert Flack <flackr@chromium.org>
Cr-Commit-Position: refs/heads/master@{#771964}
2020-05-26 21:38:58 +00:00
Robert Flack
c9e69959a3 Document necessary rr patch to support MADV_WIPEONFORK.
Bug: 1082304
Change-Id: Ica7adee109a045813db9cae6dafa2a6c4ec083e8
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2199703
Reviewed-by: David Benjamin <davidben@chromium.org>
Commit-Queue: Robert Flack <flackr@chromium.org>
Cr-Commit-Position: refs/heads/master@{#768409}
2020-05-13 19:52:31 +00:00
Robert Flack
e13e0b1bc2 Document how to find renderer for rr debugging on multi-process chrome.
Bug: 730065
Change-Id: I60f2dc628d00c0cf9c70bdff417f5c8cc07f8243
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2112981
Reviewed-by: Daniel Cheng <dcheng@chromium.org>
Commit-Queue: Robert Flack <flackr@chromium.org>
Cr-Commit-Position: refs/heads/master@{#759701}
2020-04-16 17:03:58 +00:00
Tom Anderson
abdbd6a60e Fix some links in docs/linux
The documents are already in linux/ so they should not have linux/ in the path,
otherwise the link will be invalid.

R=sky

Change-Id: Iff06ba371672bc5f1d7c06d54babb90e3b4edbcf
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1992800
Auto-Submit: Thomas Anderson <thomasanderson@chromium.org>
Commit-Queue: Scott Violet <sky@chromium.org>
Reviewed-by: Scott Violet <sky@chromium.org>
Cr-Commit-Position: refs/heads/master@{#729782}
2020-01-09 16:59:27 +00:00
Tom Anderson
93e49e49ba Update relative links to Linux docs
This is a followup to [1].

[1] https://chromium-review.googlesource.com/c/chromium/src/+/1974901

TBR=sky

Change-Id: I442ffe121378607bdc5e1b16c081b8d66b138955
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1980900
Reviewed-by: Thomas Anderson <thomasanderson@chromium.org>
Commit-Queue: Thomas Anderson <thomasanderson@chromium.org>
Cr-Commit-Position: refs/heads/master@{#727226}
2019-12-23 19:55:37 +00:00
Tom Anderson
9f5be07962 Move Linux documents to a subdir
There were 32 linux documents in docs. It's getting out of hand, so lets
move them to a subdirectory.

R=sky,jorgelo
CC=thestig
BUG=None

Change-Id: Ib7de415c72fe6ec3c51dc2b28f06b34487b5b6d1
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1974901
Commit-Queue: Thomas Anderson <thomasanderson@chromium.org>
Reviewed-by: Scott Violet <sky@chromium.org>
Reviewed-by: Jorge Lucangeli Obes <jorgelo@chromium.org>
Cr-Commit-Position: refs/heads/master@{#726499}
2019-12-19 20:54:32 +00:00