{"push_id":670320963,"size":15,"distinct_size":15,"ref":"refs/heads/err","head":"73b6001b211b1e0f4444dd05d8d1639c9d2b8b01","before":"0933cc0d0a544319f4ec115b93cea2ae213cbc0f","commits":[{"sha":"8a6e6190e299df702c1b1279bb6aad6f6d181b4e","author":{"email":"53b4baf7beed764e759decbbaacbec9351f2869e@chromium.org","name":"Wei-Yin Chen (陳威尹)"},"message":"Skip tests using URL() in continuous integration\n\ndrone.io uses older version of Chrome browser, which doesn't fully\nsupport JavaScript URL() object.\n\n\"ant test\" still runs all the tests locally.\n\nBUG=483149\nR=kuan@chromium.org\n\nReview URL: https://codereview.chromium.org/1115303002","distinct":true,"url":"https://api.github.com/repos/wychen/dom-distiller/commits/8a6e6190e299df702c1b1279bb6aad6f6d181b4e"},{"sha":"d4854eb5387e769863c5a26d5e2ac2af5f86f8e8","author":{"email":"53b4baf7beed764e759decbbaacbec9351f2869e@chromium.org","name":"Wei-Yin Chen (陳威尹)"},"message":"Catch errors in continuous integration script\n\nThis fixes a bug introduced in the following revision:\n\"Put steps in continuous integration into a bash script\".\nhttps://codereview.chromium.org/952783006\n\nAlso print the commands being run.\n\nAllow \"start xvfb\" to fail in order to run locally.\n\nBUG=483149\nR=nyquist@chromium.org\n\nReview URL: https://codereview.chromium.org/1115583005","distinct":true,"url":"https://api.github.com/repos/wychen/dom-distiller/commits/d4854eb5387e769863c5a26d5e2ac2af5f86f8e8"},{"sha":"31359c23bbf0a418f1e38d814bab97d6296543e0","author":{"email":"53b4baf7beed764e759decbbaacbec9351f2869e@chromium.org","name":"Wei-Yin Chen (陳威尹)"},"message":"Allow test.filter to begin with a dash\n\nThis allows the following command to run:\n$ ant test -Dtest.filter '-*Foo*'\n\nR=cjhopman@chromium.org\n\nReview URL: https://codereview.chromium.org/1115143002","distinct":true,"url":"https://api.github.com/repos/wychen/dom-distiller/commits/31359c23bbf0a418f1e38d814bab97d6296543e0"},{"sha":"0650d4134e8a7c03d28d84e89ecd007bfb4f169e","author":{"email":"4439ff048f4288e04f9446ae0ea13621e0f61f68@igalia.com","name":"mrobinson"},"message":"Never regard HTML elements as comments\n\nLike BODY elements, HTML elements should not be regarded as comment\nsections and marked with STRICTLY_NOT_CONTENT. This fixes display of\nsome New York Times articles.\n\nBUG=483036\nR=cjhopman@chromium.org\n\nReview URL: https://codereview.chromium.org/1121623002\n\nPatch from mrobinson <mrobinson@igalia.com>.","distinct":true,"url":"https://api.github.com/repos/wychen/dom-distiller/commits/0650d4134e8a7c03d28d84e89ecd007bfb4f169e"},{"sha":"a03cbd2ff8b4989367eb20c9c32009c4dd175c85","author":{"email":"53b4baf7beed764e759decbbaacbec9351f2869e@chromium.org","name":"Wei-Yin Chen (陳威尹)"},"message":"Disregard \"comment\" appearing in className when too many classes\n\nIf an element has many classes, having \"comment\" in it is no longer a\nstrong signal.\n\nThis is inspired by this change:\n\"Never regard HTML elements as comments\"\nhttps://codereview.chromium.org/1121623002/\n\nScoring changes:\nhttps://x20web.corp.google.com/~wychen/domdistillerscore/comment/reader-mode.html\n\n1 URL changed:\nhttp://www.bbcamerica.com/anglophenia/2014/07/rupert-graves-joins-cast-last-tango-halifax/\n  P 0.308 -> 1.000, R 0.014 -> 0.938\n\nBUG=483036\nR=cjhopman@chromium.org\n\nReview URL: https://codereview.chromium.org/1122623002","distinct":true,"url":"https://api.github.com/repos/wychen/dom-distiller/commits/a03cbd2ff8b4989367eb20c9c32009c4dd175c85"},{"sha":"fa4d011c4445f9bea9d1b1cbeccc99fcbc0259e1","author":{"email":"53b4baf7beed764e759decbbaacbec9351f2869e@chromium.org","name":"Wei-Yin Chen (陳威尹)"},"message":"Support running tests in random order\n\nThis helps catch bad dependencies between test cases.\n\nExample usage:\n$ ant [run]test -Dtest.shuffle=1\n\nInspired by Testing on the Toilet (TotT) Episode 376:\nDon't Alter Shared Environments Between Tests.\nhttp://tott/2015/04/episode-376-dont-alter-shared.html\n\nBUG=483172\nR=nyquist@chromium.org\n\nReview URL: https://codereview.chromium.org/1113733002","distinct":true,"url":"https://api.github.com/repos/wychen/dom-distiller/commits/fa4d011c4445f9bea9d1b1cbeccc99fcbc0259e1"},{"sha":"7a0544c0393e052df504d62c16ccb339e7459761","author":{"email":"53b4baf7beed764e759decbbaacbec9351f2869e@chromium.org","name":"Wei-Yin Chen (陳威尹)"},"message":"Support running jstests repeatedly\n\nThis can be combined with random order to try different combinations,\nor just repeat the deterministic order to detect flakyness.\n\nExample usage:\n$ ant [run]test -Dtest.repeat=10\n\nBUG=483172\nR=nyquist@chromium.org\n\nReview URL: https://codereview.chromium.org/1115523002","distinct":true,"url":"https://api.github.com/repos/wychen/dom-distiller/commits/7a0544c0393e052df504d62c16ccb339e7459761"},{"sha":"f5070fdb238790d5457c9a329cb4c57548f62e6a","author":{"email":"53b4baf7beed764e759decbbaacbec9351f2869e@chromium.org","name":"Wei-Yin Chen (陳威尹)"},"message":"Properly restore test environment between tests\n\nBefore this fix, the following command would fail:\n$ ant test -Dtest.shuffle=1 -Dtest.repeat=100\n\nThe root cause is that OpenGraphProtocolParserTest set attributes on\nmRoot, and they are not removed after the end of each test.\n\nBUG=483172\nR=nyquist@chromium.org\n\nReview URL: https://codereview.chromium.org/1119573003","distinct":true,"url":"https://api.github.com/repos/wychen/dom-distiller/commits/f5070fdb238790d5457c9a329cb4c57548f62e6a"},{"sha":"270058c9cd66312694c85989b558e62d9191b819","author":{"email":"53b4baf7beed764e759decbbaacbec9351f2869e@chromium.org","name":"Wei-Yin Chen (陳威尹)"},"message":"Shuffle tests in continuous integration\n\nThe order of tests and test cases are randomly shuffled, and repeat the\ntest with different random seeds for 10 times. This is to increase the\nrobustness of our continuous integration, with less than 10 seconds more\nof run time.\n\nBUG=483172\nR=nyquist@chromium.org\n\nReview URL: https://codereview.chromium.org/1125813003","distinct":true,"url":"https://api.github.com/repos/wychen/dom-distiller/commits/270058c9cd66312694c85989b558e62d9191b819"},{"sha":"b9e2b72d5a4204f9e5fdc5d6bba199fdbb5c89f6","author":{"email":"53b4baf7beed764e759decbbaacbec9351f2869e@chromium.org","name":"Wei-Yin Chen (陳威尹)"},"message":"Update doc about Developing on Mac OS X\n\npip command either needs sudo, or --user option.\n\nR=nyquist@chromium.org\n\nReview URL: https://codereview.chromium.org/1123403002","distinct":true,"url":"https://api.github.com/repos/wychen/dom-distiller/commits/b9e2b72d5a4204f9e5fdc5d6bba199fdbb5c89f6"},{"sha":"1f211a44c0a888b1434c08db0406653e9643ac7b","author":{"email":"53b4baf7beed764e759decbbaacbec9351f2869e@chromium.org","name":"Wei-Yin Chen (陳威尹)"},"message":"Simplify testImageWithSrcset\n\nUse mRoot instead of creating a new document.\n\nR=cjhopman@chromium.org\n\nReview URL: https://codereview.chromium.org/1126383002","distinct":true,"url":"https://api.github.com/repos/wychen/dom-distiller/commits/1f211a44c0a888b1434c08db0406653e9643ac7b"},{"sha":"ec27418ebb3d2066489ad365f11a68c16abd2fa3","author":{"email":"53b4baf7beed764e759decbbaacbec9351f2869e@chromium.org","name":"Wei-Yin Chen (陳威尹)"},"message":"Show distilled page in Chrome Extension\n\nAfter clicking the Reader Mode icon in the omnibox, the distilled page\nwould replace the original one, just for debugging purpose.\nClicking \"Profile Extraction\" button in the devtool panel would not\nchange the original page, so it is still possible to do profiling for\nmultiple times.\n\nThe outer console.log() is not very useful, so removed.\n\nR=cjhopman@chromium.org\n\nReview URL: https://codereview.chromium.org/1133543002","distinct":true,"url":"https://api.github.com/repos/wychen/dom-distiller/commits/ec27418ebb3d2066489ad365f11a68c16abd2fa3"},{"sha":"aa79b394e7508c3b384dc0cc653d97a8f9f12658","author":{"email":"4439ff048f4288e04f9446ae0ea13621e0f61f68@igalia.com","name":"mrobinson"},"message":"Strip bgcolor attribute from all table elements\n\nThis is a companion change to stripping color attributes from font tags.\nOnce the color is stripped from the font tag, any remaining background\ncolors might obscure black text. Later, this can be removed or made\nconfigurable by tuning the luminosity of text upon the original\nbackground color.\n\nBUG=382993\nR=cjhopman@chromium.org, wychen@chromium.org\n\nReview URL: https://codereview.chromium.org/1111213003\n\nPatch from mrobinson <mrobinson@igalia.com>.","distinct":true,"url":"https://api.github.com/repos/wychen/dom-distiller/commits/aa79b394e7508c3b384dc0cc653d97a8f9f12658"},{"sha":"ccdfba128152c30702aa81dd28f2b59f901a1cab","author":{"email":"53b4baf7beed764e759decbbaacbec9351f2869e@chromium.org","name":"Wei-Yin Chen (陳威尹)"},"message":"Avoid chromedriver elsewhere if any\n\nMake sure the chromedriver installed by install-build-deps.sh is used.\nBefore this change, if chromedriver is already installed and is in $PATH,\nthat would be used instead of the one installed by install-build-deps.sh.","distinct":true,"url":"https://api.github.com/repos/wychen/dom-distiller/commits/ccdfba128152c30702aa81dd28f2b59f901a1cab"},{"sha":"73b6001b211b1e0f4444dd05d8d1639c9d2b8b01","author":{"email":"53b4baf7beed764e759decbbaacbec9351f2869e@chromium.org","name":"Wei-Yin Chen (陳威尹)"},"message":"run all tests","distinct":true,"url":"https://api.github.com/repos/wychen/dom-distiller/commits/73b6001b211b1e0f4444dd05d8d1639c9d2b8b01"}]}