tree ca4e2ea83c7b2343844c4e60d5d7c03353178d30
parent 2e09d18f78f5772d2d9506ff8024ca8542654358
author mtklein <mtklein@chromium.org> 1404924850 -0700
committer Commit bot <commit-bot@chromium.org> 1404924851 -0700

Always initialize SkDashPathEffect fields.

SkDashPathEffect is flattened into a hash key as part of the old-and-busted
SkPicture paint deduping code.  If all its fields aren't intialized, this hash
will be based on uninitialized data.  This means the hash won't be
deterministic, and worse, Valgrind and MSAN will make us feel bad.

An alternative to this is to have SkDashPath::CalcDashParameters always
guarantee it writes something to all its output parameters, even when the dash
intervals make no sense.  I like it being dumb and its users defensive, but
could go either way.

BUG=391001
R=reed@google.com, mtklein@google.com

Author: mtklein@chromium.org

Review URL: https://codereview.chromium.org/379813004