{"push_id":2600740720,"size":4,"distinct_size":4,"ref":"refs/heads/master","head":"ac3d5886352ec97a24347c07bec28870ac3bb8d2","before":"a102c90593abb6f06680dc4f1f3fa479a75d54a5","commits":[{"sha":"9598bfd62f139a2b99184655ddb4963c43809272","author":{"name":"Marc Treib","email":"610c1df9d78ad5ebf95d7131452edf97cfdb1875@chromium.org"},"message":"ProfileSyncService: TryStart in OnPrimaryAccountSet, not OnRefreshTokenAvailable\n\nAfter https://crrev.com/c/997796, Sync startup doesn't depend on refresh\ntoken availability anymore, so OnRefreshTokenAvailable is not a relevant\nstartup signal. The primary account changing *is* a valid signal though, per\nthe IsSignedIn() condition in CanSyncStart().\n\nThis is another step towards making ProfileSyncService refresh-token-agnostic.\n\nBug: 825190, 842697\nChange-Id: I0e5e001e54ec54301a0a6f1333718b5a7e3b13c4\nReviewed-on: https://chromium-review.googlesource.com/1073413\nReviewed-by: Mikel Astiz <mastiz@chromium.org>\nCommit-Queue: Marc Treib <treib@chromium.org>\nCr-Commit-Position: refs/heads/master@{#562359}","distinct":true,"url":"https://api.github.com/repos/chromium/chromium/commits/9598bfd62f139a2b99184655ddb4963c43809272"},{"sha":"03d5a7880dd33ae9237cbcfb9ba35cbbc130a637","author":{"name":"Jan Krcal","email":"93237df08813c25e1a581e7e2c8cf6e31c2a1037@chromium.org"},"message":"[USS] Rephrase the comment for EntityData\n\nThe CL rephrases a comment for EntityData to remove the reference to the\nModelTypeSyncBridge which is an implementation detail of each model\ntype.\n\nBug: 819993\nChange-Id: I966330419913f70d7a4bbf70567727a4ace7283c\nReviewed-on: https://chromium-review.googlesource.com/1075531\nReviewed-by: Mohamed Amir Yosef <mamir@chromium.org>\nReviewed-by: Marc Treib <treib@chromium.org>\nCommit-Queue: Jan Krcal <jkrcal@chromium.org>\nCr-Commit-Position: refs/heads/master@{#562360}","distinct":true,"url":"https://api.github.com/repos/chromium/chromium/commits/03d5a7880dd33ae9237cbcfb9ba35cbbc130a637"},{"sha":"b2402b2d9027173dc2d221ccfb8247ba7913ce28","author":{"name":"Koji Ishii","email":"cb141bc04912f7082317b2d75e22ecde40953ac2@chromium.org"},"message":"[LayoutNG] Fix text-decoration offset for overlines and under-underlines\n\nThis patch fixes text-decoration offset for underlines with\n'text-underline-position: under' and overlines.\n\nIn the current paint code, these take the farthest position\nwithin the decorating box. That part of the code is not\nincluded in this patch yet.\n\nRebaselines:\n* first-line-text-decoration*: Legacy renders line through on\n  trailing spaces which should be removed, probably a bug.\n* stroking-decorations.html: Overlines render 1px above the\n  legacy. Not sure where that comes from, it may change back\n  when we support decorating box.\n\nBug: 636993\nCq-Include-Trybots: master.tryserver.chromium.linux:linux_layout_tests_layout_ng\nChange-Id: I1c67cbb697703b329179b6e4ec71aaa2047fa934\nReviewed-on: https://chromium-review.googlesource.com/1074570\nReviewed-by: Aleks Totic <atotic@chromium.org>\nReviewed-by: Emil A Eklund <eae@chromium.org>\nCommit-Queue: Emil A Eklund <eae@chromium.org>\nCr-Commit-Position: refs/heads/master@{#562361}","distinct":true,"url":"https://api.github.com/repos/chromium/chromium/commits/b2402b2d9027173dc2d221ccfb8247ba7913ce28"},{"sha":"ac3d5886352ec97a24347c07bec28870ac3bb8d2","author":{"name":"Siddhartha","email":"5cd4c27a3fce3491cd65f5ba040b8b631f008320@chromium.org"},"message":"Reland \"Reland \"Use thresholds based on RAM size for OOM intervention\"\"\n\nThis reverts commit a21bf6f95c3698b9542d8aa3a4ed081e2771d7fd.\n\nReason for revert: Test field trial config does not support low_end_device flag.\nSo, make sure the config does not turn on intervention on high end devices\nwithout new param.\n\nOriginal change's description:\n> Revert \"Reland \"Use thresholds based on RAM size for OOM intervention\"\"\n>\n> This reverts commit 9191004f57d2bbb925d1d48194036bb1230d0f1f.\n>\n> Reason for revert: The failure after the revert was a different failure.\n> Sorry for the reland.\n>\n> Original change's description:\n> > Reland \"Use thresholds based on RAM size for OOM intervention\"\n> >\n> > This reverts commit 50257b9eafc8ec098a9693d4740fd761ace59c35.\n> >\n> > Reason for revert: This was not the cause of failure, see\n> > https://crbug.com/845763\n> >\n> > Original change's description:\n> > > Revert \"Use thresholds based on RAM size for OOM intervention\"\n> > >\n> > > This reverts commit 0c2124b3c9acf283315fb56beea340d050db0acd.\n> > >\n> > > Reason for revert: Speculatively reverting to see if it fixes webgl conformance test timeouts\n> > > Bug: 845411\n> > >\n> > > Original change's description:\n> > > > Use thresholds based on RAM size for OOM intervention\n> > > >\n> > > > This CL does not change behavior of existing experiment on 512 devices.\n> > > > 1. Remove low-end device checks while enabling intervention. This can be\n> > > >    filtered in Finch config.\n> > > > 2. Set threshold based on percentage of RAM instead of absolute number.\n> > > > 3. Disable intervention if device has more than 512MB RAM and threshold\n> > > >    was set to absolute number.\n> > > >\n> > > > BUG=843419\n> > > >\n> > > >\n> > > > Change-Id: Ib40a038d9d2f111a514b3f41ce7f32a3ea3a156d\n> > > > Reviewed-on: https://chromium-review.googlesource.com/1066804\n> > > > Commit-Queue: Siddhartha S <ssid@chromium.org>\n> > > > Reviewed-by: Kenichi Ishibashi <bashi@chromium.org>\n> > > > Cr-Commit-Position: refs/heads/master@{#560430}\n> > >\n> > > TBR=bashi@chromium.org,ssid@chromium.org\n> > >\n> > > Change-Id: I4b50e522dcc6c91542927a9e769e5eee24f486a5\n> > > No-Presubmit: true\n> > > No-Tree-Checks: true\n> > > No-Try: true\n> > > Bug: 843419\n> > > Reviewed-on: https://chromium-review.googlesource.com/1069567\n> > > Reviewed-by: enne <enne@chromium.org>\n> > > Commit-Queue: enne <enne@chromium.org>\n> > > Cr-Commit-Position: refs/heads/master@{#560761}\n> >\n> > TBR=bashi@chromium.org,enne@chromium.org,ssid@chromium.org\n> >\n> > Change-Id: I4d0a08cd44772dc690ac1d93b1d92b4934ed0373\n> > No-Presubmit: true\n> > No-Tree-Checks: true\n> > No-Try: true\n> > Bug: 845411, 843419\n> > Reviewed-on: https://chromium-review.googlesource.com/1069888\n> > Reviewed-by: Siddhartha S <ssid@chromium.org>\n> > Commit-Queue: Siddhartha S <ssid@chromium.org>\n> > Cr-Commit-Position: refs/heads/master@{#560882}\n>\n> TBR=bashi@chromium.org,enne@chromium.org,ssid@chromium.org\n>\n> Change-Id: Ib8841dddaa2a167e674c997ec1577036df26e8cb\n> No-Presubmit: true\n> No-Tree-Checks: true\n> No-Try: true\n> Bug: 845411, 843419\n> Reviewed-on: https://chromium-review.googlesource.com/1069876\n> Reviewed-by: Siddhartha S <ssid@chromium.org>\n> Commit-Queue: Siddhartha S <ssid@chromium.org>\n> Cr-Commit-Position: refs/heads/master@{#560889}\n\nTBR=enne@chromium.org\n\n# Not skipping CQ checks because original CL landed > 1 day ago.\n\nBug: 845411, 843419\nChange-Id: I5c03b1ac806a5f4e0b1516761e3b86c242a3c200\nReviewed-on: https://chromium-review.googlesource.com/1074151\nReviewed-by: Kenichi Ishibashi <bashi@chromium.org>\nReviewed-by: Siddhartha S <ssid@chromium.org>\nCommit-Queue: Siddhartha S <ssid@chromium.org>\nCr-Commit-Position: refs/heads/master@{#562362}","distinct":true,"url":"https://api.github.com/repos/chromium/chromium/commits/ac3d5886352ec97a24347c07bec28870ac3bb8d2"}]}
{"actor":{"display_login":"aarongable"}}