{"push_id":996257325,"size":1,"distinct_size":1,"ref":"refs/heads/master","head":"0fe12bcfb79de1d1066a4c3213184dbf2aac1321","before":"e027f1705eb29560df334bcc02a27e5774277984","commits":[{"sha":"0fe12bcfb79de1d1066a4c3213184dbf2aac1321","author":{"email":"fe9e26f3989304bf7275ed139c57fd059c63d22d@chromium.org","name":"brucedawson"},"message":"Revert of Add non-inline SkRTreeFactory constructor to fix clang-cl (patchset #2 id:20001 of https://codereview.chromium.org/1676833002/ )\n\nReason for revert:\nThis workaround should no longer be needed, per comment#16 from\nthakis@ on the original CL.\n\nOriginal issue's description:\n> Add non-inline SkRTreeFactory constructor to fix clang-cl\n>\n> When Chromium is switched to build with VS 2015 that switches clang-cl\n> to build in -fmsc-version=1900 mode. This causes some changes in\n> importing of constructors and vftables which leads to link errors in\n> blink_platform.dll. Adding a non-inline constructor makes 1900 mode\n> behave consistently with 1800 mode and avoids the link error.\n>\n> The root cause needs investigation but this will unblock the VS 2015\n> switch.\n>\n> BUG=440500,498544\n> GOLD_TRYBOT_URL= https://gold.skia.org/search2?unt=true&query=source_type%3Dgm&master=false&issue=1676833002\n>\n> Committed: https://skia.googlesource.com/skia/+/964eec67760196585954203ba625e440607f7e92\n\nTBR=reed@google.com,bsalomon@google.com,mtklein@chromium.org,thakis@chromium.org\n# Not skipping CQ checks because original CL landed more than 1 days ago.\nBUG=440500,498544\n\nReview URL: https://codereview.chromium.org/1739363002","distinct":true,"url":"https://api.github.com/repos/HalCanary/skia-hc/commits/0fe12bcfb79de1d1066a4c3213184dbf2aac1321"}]}