Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(700)

Unified Diff: docs/stability_fixit_week.md

Issue 1309473002: WIP: Migrate Wiki content over to src/docs (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: Created 5 years, 4 months ago
Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.
Jump to:
View side-by-side diff with in-line comments
Download patch
Index: docs/stability_fixit_week.md
diff --git a/docs/stability_fixit_week.md b/docs/stability_fixit_week.md
new file mode 100644
index 0000000000000000000000000000000000000000..d6ace66074974343f40a7100c68fb6a888799d9c
--- /dev/null
+++ b/docs/stability_fixit_week.md
@@ -0,0 +1,62 @@
+# Stability Fixit Week
+
+The Chromium project tries hard to avoid bugs.
+We close the tree if even a tiny bug is detected by our continuous test suite.
+For bugs that can't be fixed right away, we file bug reports so developers can fix them offline.
+But after a while, the stack of crash and leak bug reports can be overwhelming
+for a single engineer to attack. So every now and then, we hold
+Stability Fixit Weeks so we can gang up on those bugs, and
+make some headway on the pile.
+
+We're especially interested in bugs taged with Fixit and Valgrind (click for lists):
+ * [Valgrind+Fixit](http://crbug.com/?can=2&q=label:Valgrind+Fixit&sort=-id)
+
+but any of the following could also use attention:
+ * [Valgrind](http://crbug.com/?can=2&q=label:Valgrind&sort=-id)
+ * [Crash](http://crbug.com/?can=2&q=label:crash&sort=-id)
+ * [Regression](http://crbug.com/?can=2&q=label:regression&sort=-id)
+ * [Fixit](http://crbug.com/?can=2&q=label:fixit&sort=-id)
+ * [LinuxBeta](http://crbug.com/?can=2&q=label:Mstone-LinuxBeta&sort=-id)
+ * [MacBeta](http://crbug.com/?can=2&q=label:Mstone-MacBeta&sort=-id)
+
+Memory leaks aren't as high priority as other sorts of Valgrind errors, but they're still worth fixing.
+
+To work on Valgrind bugs, it helps to have the tool installed.
+
+Valgrind **is** free; see http://dev.chromium.org/developers/how-tos/using-valgrind for how to install and use it with Chrome on Mac and Linux.
+
+The bug reports usually name the test that exposed the problem; to
+reproduce the problem, you have to run the same test under Valgrind.
+That can be slow, and these bugs are subtle, so while you're waiting, try to understand the bug by inspection of the source code.
+
+If the bug in question is already being suppressed, you may need to remove the suppression before you can see it.
+With Valgrind, it's easy to find and comment out suppressions; they're mostly in tools/valgrind/memcheck/suppressions.txt, and every suppression has an associated bug number. (Mac-specific suppressions are in a nearby file, suppressions\_mac.txt.)
+
+To try to reproduce a problem in a UI test named Foo.Bar with Valgrind, give the command
+```
+tools/valgrind/chrome_tests.sh -t ui --gtest_filter=Foo.Bar
+```
+
+(Generally you should be in the chromium/src directory to run any of these tests.)
+
+Layout tests are a bit different.
+
+To try to reproduce a problem in a test named LayoutTests/fast/loader/local-css-from-local-html.html with Valgrind, you would give the command
+```
+tools/valgrind/valgrind_webkit_tests.sh LayoutTests/fast/loader/local-css-from-local-html.html
+```
+This will eventually produce output on stdout, but rather than wait, you can look in the current directory for its temporary log files while it's still producing them.
+
+Sometimes problems only happen occasionally; you might want to write a little script to run your test in a loop. Or you might run a wide overnight fishing expedition to see if you can find any valgrind errors
+at all in one area of the tests.
+
+Note that the only kind of leak reports we care about from Valgrind are those marked "definitely leaked". Ones marked "possibly leaked" should usually be ignored.
+
+## See Also
+
+ * StabilizeTrunk
+ * LayoutTestsLinux
+ * LinuxDebugging
+ * [RunningChromeUITests](RunningChromeUITests.md)
+ * http://dev.chromium.org/developers/how-tos/using-valgrind
+ * http://dev.chromium.org/developers/testing

Powered by Google App Engine
This is Rietveld 408576698