Update rebaselining process to be less Florin intensive

site/dev/sheriffing/index.md

 Tree Sheriffs Documentation
 ===========================
 
 ### Contents ###
 
 *   [What does a sheriff do?](#what_is_a_sheriff)
     +   [Skia tree](#skia_tree)
     +   [DEPS rolls](#deps_rolls)
     +   [Gold and Perf](#gold_and_perf)
     +   [Documentation](#sheriff_doc)
@@ skipping 112 matching lines @@
   * goto [crbug.com](https://crbug.com)
   * Make sure you’re logged in with your Chromium credentials
   * Click “New Issue”
   * Summary: “Skia image rebaseline”
   * Description:
       * DEPS roll #,
       * Helpful message about what went wrong (e.g., “Changes to how lighting is scaled in Skia r#### changed the following images:”)
       * Layout tests effected
       * You should copy the list of affected from stdio of the failing bot
   * Status: Assigned
-  * Owner: fmalita@
+  * Owner: yourself
   * cc: reed@, bsalomon@, robertphillips@ & developer responsible for changes
   * Labels: OS-All & Cr-Blink-LayoutTests
   * If it is filter related, cc senorblanco@
 
-* Edit [skia/skia_test_expectations.txt](https://chromium.googlesource.com/chromium/src/+/master/skia/skia_test_expectations.txt)
+* (Dispreferred but faster) Edit [skia/skia_test_expectations.txt](https://chromium.googlesource.com/chromium/src/+/master/skia/skia_test_expectations.txt)
   * Add # comment about what has changed (I usually paraphrase the crbug text)
-  * Add line(s) like after the comment:
+  * Add line(s) like the following after the comment:
       * crbug.com/<bug#youjustcreated> foo/bar/test-name.html [ ImageOnlyFailure ]
+  * Note: this change is usually done in the DEPS roll patch itself
 
-* Commit the changes and fire off new try bots (you usually only need to fire off the layout bots)
+* (Preferred but slower) Make a separate Blink patch by editing LayoutTests/TestExpectations

[jcgregorio, 2015/04/13 14:14:07]

Maybe move the Preferred option to appear before the Dispreferred option?

+  * Add # comment about what has changed (I usually paraphrase the crbug text)
+  * Add line(s) like the following after the comment:
+      * crbug.com/<bug#youjustcreated> foo/bar/test-name.html [ NeedsManualRebaseline ]
+  * Commit the patch you created and wait until it lands and rolls into Chrome
+
+* Retry the DEPS roll (for the 1st/dispreferred option this usually means just retrying the layout bots)
+* Make a Blink patch by editing LayoutTests/TestExpectations
+  * Add # comment about what has changed 
+  * Add line(s) like the following after the comment:
+      * crbug.com/<bug#youjustcreated> foo/bar/test-name.html [ NeedsRebaseline ]
+        * (if you took the second option above you can just edit the existing line(s))
+
+* If you took the first/dispreferred option above:
+  * Wait for the Blink patch to roll into Chrome
+  * Create a Chrome patch that removes your suppressions from skia/skia_test_expectations.txt
+
+