- 14 Sep, 2018 6 commits
-
-
Kyle Milka authored
New test data created by download_test_doodles.sh Bug: None Change-Id: I56ca865cc616135022b9661142a7cec55e85c386 Reviewed-on: https://chromium-review.googlesource.com/1225699Reviewed-by:
Kristi Park <kristipark@chromium.org> Commit-Queue: Kyle Milka <kmilka@chromium.org> Cr-Commit-Position: refs/heads/master@{#591220}
-
chromium-autoroll authored
https://pdfium.googlesource.com/pdfium.git/+log/a2fc263db880..61f1d62daa12 git log a2fc263db880..61f1d62daa12 --date=short --no-merges --format='%ad %ae %s' 2018-09-13 tsepez@chromium.org Reland "Introduce FX_Number class as a replacement for FX_atonum()." 2018-09-13 tsepez@chromium.org Disable FPDFEditEmbeddertest.AddCIDFontText Created with: gclient setdep -r src/third_party/pdfium@61f1d62daa12 The AutoRoll server is located here: https://autoroll.skia.org/r/pdfium-autoroll Documentation for the AutoRoller is here: https://skia.googlesource.com/buildbot/+/master/autoroll/README.md If the roll is causing failures, please contact the current sheriff, who should be CC'd on the roll, and stop the roller if necessary. TBR=dsinclair@chromium.org Change-Id: I125526efe825c0e9f246a95051f141893e5ee14b Reviewed-on: https://chromium-review.googlesource.com/1226090Reviewed-by:
chromium-autoroll <chromium-autoroll@skia-public.iam.gserviceaccount.com> Commit-Queue: chromium-autoroll <chromium-autoroll@skia-public.iam.gserviceaccount.com> Cr-Commit-Position: refs/heads/master@{#591219}
-
Han Leon authored
This CL moves the blink::mojom::ServiceWorkerHostAssociatedPtr from Content into Blink, now Blink can use it directly to do IPCs rather than requesting Content to do so via public Web interfaces. To avoid too big a CL, this CL tries to do the movement only and is leaving some refactoring work as TODOs, i.e. some public Web structs/callbacks no longer need to cross the boundary of Content and Blink, so some of them should be just removed and some should be hidden inside Blink. Follow-up CLs will do these cleanup. BUG=789857 Change-Id: I080e3f40b4f1e5a79cf37cd62bc6e8d3172b54c4 Reviewed-on: https://chromium-review.googlesource.com/1214709 Commit-Queue: Leon Han <leon.han@intel.com> Reviewed-by:
Makoto Shimazu <shimazu@chromium.org> Cr-Commit-Position: refs/heads/master@{#591218}
-
Daniel Cheng authored
Pathnames are delimited with \ on Windows by default, while the script only expects / Change-Id: Ie15090dd45440dbb4eaad1e5feb48d18a0ed971f Reviewed-on: https://chromium-review.googlesource.com/1225933Reviewed-by:
Kentaro Hara <haraken@chromium.org> Commit-Queue: Daniel Cheng <dcheng@chromium.org> Cr-Commit-Position: refs/heads/master@{#591217}
-
Kevin Marshall authored
The AutoRoll server is located here: https://autoroll.skia.org/r/fuchsia-sdk-chromium-autoroll Documentation for the AutoRoller is here: https://skia.googlesource.com/buildbot/+/master/autoroll/README.md If the roll is causing failures, please contact the current sheriff, who should be CC'd on the roll, and stop the roller if necessary. CQ_INCLUDE_TRYBOTS=luci.chromium.try:fuchsia_arm64_cast_audio;luci.chromium.try:fuchsia_x64_cast_audio TBR=cr-fuchsia+bot@chromium.org Change-Id: Iee9c1a2f249f0b660ed6d2f9e88f7e43eec2378d Reviewed-on: https://chromium-review.googlesource.com/1226271 Commit-Queue: Kevin Marshall <kmarshall@chromium.org> Reviewed-by:
Kevin Marshall <kmarshall@chromium.org> Cr-Commit-Position: refs/heads/master@{#591216}
-
sebsg authored
Bug: 883923 Change-Id: I3e35456bda1347ae060fe2748b0126dfa312f27e Reviewed-on: https://chromium-review.googlesource.com/1224659Reviewed-by:
Michael Giuffrida <michaelpg@chromium.org> Commit-Queue: Sebastien Seguin-Gagnon <sebsg@chromium.org> Cr-Commit-Position: refs/heads/master@{#591215}
-
- 13 Sep, 2018 34 commits
-
-
Chong Zhang authored
Consistently failing since https://ci.chromium.org/p/chromium/builders/luci.chromium.ci/Mojo%20ChromiumOS/34744 Bug: 866991 Change-Id: Idd9ea206c37b53fd0b2d8fa516c8cb486d1eb7b6 Reviewed-on: https://chromium-review.googlesource.com/1225100Reviewed-by:
Scott Violet <sky@chromium.org> Commit-Queue: Chong Zhang <chongz@chromium.org> Cr-Commit-Position: refs/heads/master@{#591214}
-
Lukasz Anforowicz authored
This CL adds extra suffixes for navigation-timing UMAs, based on whether the renderer handling the navigation is backgrounded or not. Since we plan to restrict the hang-renderer-dialog to only show up for foreground tabs, the navigation commit timeout should be based on numbers that exclude backgrounded renderers. The new UMAs should help with getting such numbers. Bug: 881812 Change-Id: I68fd4f83dbb09ee036d1a4fd23591c8255c66398 Reviewed-on: https://chromium-review.googlesource.com/1220470Reviewed-by:
Nasko Oskov <nasko@chromium.org> Reviewed-by:
Steven Holte <holte@chromium.org> Commit-Queue: Łukasz Anforowicz <lukasza@chromium.org> Cr-Commit-Position: refs/heads/master@{#591213}
-
Emircan Uysaler authored
This reverts commit b968ecb7. Reason for revert: WebRtcAudioQualityBrowserTests on chromium.webrtc.MacTester have been failing consistently since this CL. https://ci.chromium.org/buildbot/chromium.webrtc/Mac%20Tester/83171 Original change's description: > Reland "Enable sandboxing for audio service field trial testing on Windows and Mac." > > This is a reland of 5fddb84e > Maybe this CL was innocent after all. > > Original change's description: > > Enable sandboxing for audio service field trial testing on Windows and Mac. > > > > Bug: 832666 > > Change-Id: Ic64a983498156bba73c458f2abc872db337d1999 > > Reviewed-on: https://chromium-review.googlesource.com/1216142 > > Reviewed-by: Robert Kaplow <rkaplow@chromium.org> > > Commit-Queue: Marina Ciocea <marinaciocea@chromium.org> > > Cr-Commit-Position: refs/heads/master@{#589911} > > Tbr: rkaplow > Bug: 832666 > Change-Id: I2874346b66b9dd686470fef13aabae6ae8e21eff > Reviewed-on: https://chromium-review.googlesource.com/1224382 > Reviewed-by: Max Morin <maxmorin@chromium.org> > Commit-Queue: Max Morin <maxmorin@chromium.org> > Cr-Commit-Position: refs/heads/master@{#591010} TBR=rkaplow@chromium.org,maxmorin@chromium.org,marinaciocea@chromium.org Change-Id: Ieeafee37da28999055e0250c6db934ac2b0e8df1 No-Presubmit: true No-Tree-Checks: true No-Try: true Bug: 832666 Reviewed-on: https://chromium-review.googlesource.com/1226207Reviewed-by:
Emircan Uysaler <emircan@chromium.org> Commit-Queue: Emircan Uysaler <emircan@chromium.org> Cr-Commit-Position: refs/heads/master@{#591212}
-
Weidong Guo authored
Bug: 883873 Change-Id: I50d4059b22cd8baefeebabc44e8c128c2efb7374 Reviewed-on: https://chromium-review.googlesource.com/1225865Reviewed-by:
Alex Newcomer <newcomer@chromium.org> Commit-Queue: Alex Newcomer <newcomer@chromium.org> Cr-Commit-Position: refs/heads/master@{#591211}
-
Ben Pastene authored
It's leaving around crash dumps that are filling up the disk and causing proceeding tests to fail. crbug.com/881638 fixed this, but that hasn't rolled into chromium yet. Disable the test until it does. Bug: 876587 Change-Id: Iacfdfb6f2f84a32027d4518a15949ab1e3d88677 Reviewed-on: https://chromium-review.googlesource.com/1225934Reviewed-by:
John Budorick <jbudorick@chromium.org> Commit-Queue: Ben Pastene <bpastene@chromium.org> Cr-Commit-Position: refs/heads/master@{#591210}
-
chromium-autoroll authored
https://chromium.googlesource.com/catapult.git/+log/572c25286dc6..62c9dfc9791f git log 572c25286dc6..62c9dfc9791f --date=short --no-merges --format='%ad %ae %s' 2018-09-13 sadrul@chromium.org rendering: Ignore trace-events for canceled draws. Created with: gclient setdep -r src/third_party/catapult@62c9dfc9791f The AutoRoll server is located here: https://autoroll.skia.org/r/catapult-autoroll Documentation for the AutoRoller is here: https://skia.googlesource.com/buildbot/+/master/autoroll/README.md If the roll is causing failures, please contact the current sheriff, who should be CC'd on the roll, and stop the roller if necessary. CQ_INCLUDE_TRYBOTS=luci.chromium.try:android_optional_gpu_tests_rel;luci.chromium.try:linux_optional_gpu_tests_rel;luci.chromium.try:mac_optional_gpu_tests_rel;luci.chromium.try:win_optional_gpu_tests_rel BUG=chromium:883592 TBR=sullivan@chromium.org Change-Id: If075924a071ef37decabb41bca148cc1111d2600 Reviewed-on: https://chromium-review.googlesource.com/1226010Reviewed-by:
-
Takumi Fujimoto authored
When the user selects a sink and its status gets changed via sink list updates, we focus on the selected sink so that the updated status text can be read by screen readers. We stop focusing when the route to it is terminated. Bug: 881564 Change-Id: Icb9e76d9e1f18f63b84611339f1ac9f0ce624b39 Reviewed-on: https://chromium-review.googlesource.com/1217520Reviewed-by:
mark a. foltz <mfoltz@chromium.org> Commit-Queue: mark a. foltz <mfoltz@chromium.org> Cr-Commit-Position: refs/heads/master@{#591208}
-
Chris Palmer authored
It's not safe, so call that out. Bug: 880974 Cq-Include-Trybots: luci.chromium.try:linux_mojo Change-Id: I9c2e4eb8116c12270aa776fd17db893b46b473d7 Reviewed-on: https://chromium-review.googlesource.com/1222284Reviewed-by:
Tom Sepez <tsepez@chromium.org> Reviewed-by:
-
David Black authored
Sometimes, but not always, we receive an empty speech recognition result when an interaction is resolving due to mic timeout. When the mic times out we discard the interaction, so we shouldn't commit the pending query. Bug: b:115513980 Change-Id: I0122c3b6a21d2df0cd79a87410dc15396178cf4c Reviewed-on: https://chromium-review.googlesource.com/1222250Reviewed-by:
Xiaohui Chen <xiaohuic@chromium.org> Commit-Queue: David Black <dmblack@google.com> Cr-Commit-Position: refs/heads/master@{#591206}
-
chromium-autoroll authored
https://chromium.googlesource.com/chromiumos/chromite.git/+log/d7d11650ca37..71ddeb9859d4 git log d7d11650ca37..71ddeb9859d4 --date=short --no-merges --format='%ad %ae %s' 2018-09-13 bhthompson@google.com Adjust release branch timing Created with: gclient setdep -r src/third_party/chromite@71ddeb9859d4 The AutoRoll server is located here: https://autoroll.skia.org/r/chromite-chromium-autoroll Documentation for the AutoRoller is here: https://skia.googlesource.com/buildbot/+/master/autoroll/README.md If the roll is causing failures, please contact the current sheriff, who should be CC'd on the roll, and stop the roller if necessary. TBR=chrome-os-gardeners@chromium.org Change-Id: If4428c75f6bc7bfeff5f611c70267f2ab6c06a23 Reviewed-on: https://chromium-review.googlesource.com/1225311Reviewed-by:
-
Ryan Sturm authored
This reflects changes in https://chromium-review.googlesource.com/c/chromium/src/+/1225095 that added Bloom filter to the blacklist component. Change-Id: Icfb9e266a54c88c5306811feec07d30df0a63680 Reviewed-on: https://chromium-review.googlesource.com/1225491 Commit-Queue: Ryan Sturm <ryansturm@chromium.org> Reviewed-by:
Doug Arnett <dougarnett@chromium.org> Cr-Commit-Position: refs/heads/master@{#591204}
-
Istiaque Ahmed authored
The root cause is fixed by https://crrev.com/b8e24bdbf6bbe43bc32fd684f2a0aad914e804d2 and the test doesn't flake locally or on trybots anymore. Bug: 834200 Change-Id: I0bca3a12374cb70554c73063d7913dfbeec78e07 Reviewed-on: https://chromium-review.googlesource.com/1225700Reviewed-by:
Devlin <rdevlin.cronin@chromium.org> Commit-Queue: Istiaque Ahmed <lazyboy@chromium.org> Cr-Commit-Position: refs/heads/master@{#591203}
-
Sean Topping authored
This is necessary for content apps which obscured by an opaque overlay (such as the remote control overlay), but still want to be treated as fully visible. Bug: internal b/113950519 Test: None Change-Id: Ifc2923e91900845e33a90bb832d05586ea711990 Reviewed-on: https://chromium-review.googlesource.com/1226261Reviewed-by:
Luke Halliwell <halliwell@chromium.org> Commit-Queue: Sean Topping <seantopping@chromium.org> Cr-Commit-Position: refs/heads/master@{#591202}
-
Darren Shen authored
Bug: 882340 Change-Id: I0897afd6e048816630c22a3e714829f3c685a0e4 Reviewed-on: https://chromium-review.googlesource.com/1215460 Commit-Queue: Darren Shen <shend@chromium.org> Reviewed-by:
Alice Boxhall <aboxhall@chromium.org> Reviewed-by:
Trent Apted <tapted@chromium.org> Cr-Commit-Position: refs/heads/master@{#591201}
-
Weilun Shi authored
Now, the browser will not load the rest of the tile images until the first N tile images finished loading. Now the N set to 9 which is the default number of tiles will show in the custom background dialog. Screencast on fast network: https://screencast.googleplex.com/cast/NDc0OTAyNTE5MDM0Njc1Mnw1OTJjZmIyNy1kOA Screencast on slow network: https://screencast.googleplex.com/cast/NjY0MDQ4NzU4MjY2MjY1Nnw3ZmZkNGY5Ni00Zg Bug: 876814 Change-Id: Id7f9e3e9538399f7fb64a68b7723ca39d6f19d65 Reviewed-on: https://chromium-review.googlesource.com/1220183 Commit-Queue: Weilun Shi <sweilun@chromium.org> Reviewed-by:
-
Kevin Strohbehn authored
This reverts commit b08f82e9. Reason for revert: Causing issues with framerate and keyboard while folder is open Original change's description: > Folder Should no longer travel with users between pages > > Bug: 880975 > Change-Id: I4a4a41833e4312c8a78ff6b74897e539bf456d1f > Reviewed-on: https://chromium-review.googlesource.com/1220057 > Reviewed-by: Weidong Guo <weidongg@chromium.org> > Commit-Queue: Kevin Strohbehn <ginko@google.com> > Cr-Commit-Position: refs/heads/master@{#590751} TBR=weidongg@chromium.org,ginko@google.com # Not skipping CQ checks because original CL landed > 1 day ago. Bug: 880975 Change-Id: I107c456e6654b9808316314d5f53ded082041fc3 Reviewed-on: https://chromium-review.googlesource.com/1226197Reviewed-by:
Weidong Guo <weidongg@chromium.org> Commit-Queue: Kevin Strohbehn <ginko@google.com> Cr-Commit-Position: refs/heads/master@{#591199}
-
Ehsan Karamad authored
This reverts commit d3358a99. Reason for revert: The test seems to be very flaky. Bug: 883965 Original change's description: > Make 'web_view_browsertest.cc' BrowserPlugin-free > > This CL removes and converts the last remaining BrowserPlugin-based test > to chrome_mime_handler_view_browsertest.cc (the new test uses a > MimeHandlerViewGuest). > > Bug: 553069, 330264 > Change-Id: I487fbd47ca0c989e746d087c72fe396f9e9ac867 > Reviewed-on: https://chromium-review.googlesource.com/1221693 > Reviewed-by: James MacLean <wjmaclean@chromium.org> > Commit-Queue: Ehsan Karamad <ekaramad@chromium.org> > Cr-Commit-Position: refs/heads/master@{#591115} TBR=ekaramad@chromium.org,wjmaclean@chromium.org Change-Id: I60e4bc6872645cb1be4de70d19f41c1c4e6e51b6 No-Presubmit: true No-Tree-Checks: true No-Try: true Bug: 553069, 330264 Reviewed-on: https://chromium-review.googlesource.com/1226202Reviewed-by:
Ehsan Karamad <ekaramad@chromium.org> Commit-Queue: Ehsan Karamad <ekaramad@chromium.org> Cr-Commit-Position: refs/heads/master@{#591198}
-
Takumi Fujimoto authored
Put the Cast toolbar icon in pressed state while showing the Media Remoting dialog. Bug: 854374 Change-Id: Ie75f7905e7c796375ec85075a56bbdf15c73d36c Reviewed-on: https://chromium-review.googlesource.com/1191063Reviewed-by:
-
Taylor Bergquist authored
Sets the DPI awareness context on the UI thread to per monitor v2 at startup, and on file dialog threads immediately after creation. Bug: 624992 Change-Id: I86ae30e7deeedc207159c5a0672ad727d55ff388 Reviewed-on: https://chromium-review.googlesource.com/1212184Reviewed-by:
Robert Liao <robliao@chromium.org> Commit-Queue: Taylor Bergquist <tbergquist@chromium.org> Cr-Commit-Position: refs/heads/master@{#591196}
-
Christian Biesinger authored
Many users of constraint space never use it. This is accomplished by introducing a wrapper class (thanks Ian for the suggestion) which internally manages the actual exclusion space pointer (now called NGExclusionSpaceInternal). It has the same interface as the old NGExclusionSpace but does not require memory allocation until you add an exclusion (comparatively rare). As a minor cleanup, this also changes two NGLayoutResult constructor arguments to use rvalue references, which they effectively already were. Cq-Include-Trybots: luci.chromium.try:linux_layout_tests_layout_ng Change-Id: I2c83ba64443bc45f7e328f8d5663d5a4867885a5 Reviewed-on: https://chromium-review.googlesource.com/1222270 Commit-Queue: Christian Biesinger <cbiesinger@chromium.org> Reviewed-by:
Ian Kilpatrick <ikilpatrick@chromium.org> Reviewed-by:
Emil A Eklund <eae@chromium.org> Cr-Commit-Position: refs/heads/master@{#591195}
-
Aga Wronska authored
The subtitle text is not suitable for demo mode setup as there is no data restoration performed in demo mode. Bug: 883530 Test: Manually verified network screen in OOBE and in demo mode setup. Change-Id: I3e70b4a963f336e59547b64b7a710c99a57ce0f6 Reviewed-on: https://chromium-review.googlesource.com/1222448Reviewed-by:
-
Jamie Madill authored
This affects a few EGL enums ANGLE was using via exposed extensions. We were incorrectly using enums from an unallocated block instead of the ones allocated to ANGLE. Reported by dan.glastonbury@gmail.com. Bug: angleproject:2806 Cq-Include-Trybots: luci.chromium.try:android_optional_gpu_tests_rel;luci.chromium.try:linux_optional_gpu_tests_rel;luci.chromium.try:mac_optional_gpu_tests_rel;luci.chromium.try:win_optional_gpu_tests_rel Change-Id: I4e96fa81faf4092ec9ea4d17e0ee24ad172fb042 Tbr: kbr@chromium.org Reviewed-on: https://chromium-review.googlesource.com/1225863Reviewed-by:
Jamie Madill <jmadill@chromium.org> Commit-Queue: Jamie Madill <jmadill@chromium.org> Cr-Commit-Position: refs/heads/master@{#591193}
-
Raymond Toy authored
Speculative fix similar to the fix for issue 882734. Bug: 882669 Change-Id: I128ed1b1d9af368237cc35ad30daaeba1fae5583 Reviewed-on: https://chromium-review.googlesource.com/1220688Reviewed-by:
Hongchan Choi <hongchan@chromium.org> Commit-Queue: Raymond Toy <rtoy@chromium.org> Cr-Commit-Position: refs/heads/master@{#591192}
-
chromium-autoroll authored
https://chromium.googlesource.com/chromium/tools/depot_tools.git/+log/77ed83d5a5e4..d42dfd429be3 git log 77ed83d5a5e4..d42dfd429be3 --date=short --no-merges --format='%ad %ae %s' 2018-09-13 recipe-roller@chromium.org Roll recipe dependencies (trivial). 2018-09-13 recipe-roller@chromium.org Roll recipe dependencies (trivial). Created with: gclient setdep -r src/third_party/depot_tools@d42dfd429be3 The AutoRoll server is located here: https://autoroll.skia.org/r/depot-tools-chromium-autoroll Documentation for the AutoRoller is here: https://skia.googlesource.com/buildbot/+/master/autoroll/README.md If the roll is causing failures, please contact the current sheriff, who should be CC'd on the roll, and stop the roller if necessary. TBR=agable@chromium.org Change-Id: If80ae0dc26a36ed142c1819f1111ca596214db99 Reviewed-on: https://chromium-review.googlesource.com/1226011Reviewed-by:
-
David Black authored
Locale needed to display proper I18N strings. Bug: b:115667092 Change-Id: Id5f19ba545d488695a4142b7210b386ddb56af1b Reviewed-on: https://chromium-review.googlesource.com/1225932Reviewed-by:
-
Christian Fremerey authored
This fixes a deadlock issue in test-only code fake_v4l2_impl.cc. Bug: 883941 Change-Id: Ie6de0e026d79cd3559852e255e464e1abbf4a564 Reviewed-on: https://chromium-review.googlesource.com/1225973Reviewed-by:
-
Istiaque Ahmed authored
The root cause is fixed by https://crrev.com/b8e24bdbf6bbe43bc32fd684f2a0aad914e804d2 and the test doesn't flake locally or on trybots anymore. Bug: 855800 Change-Id: I316b876de5e993f1d41bb12cc040ac9e1dccd0e0 Reviewed-on: https://chromium-review.googlesource.com/1226071Reviewed-by:
-
chromium-autoroll authored
https://skia.googlesource.com/skia.git/+log/bc6b9cb6ac40..9548ea9d6ce1 git log bc6b9cb6ac40..9548ea9d6ce1 --date=short --no-merges --format='%ad %ae %s' 2018-09-13 caryclark@skia.org fix bookmaker 2018-09-13 caryclark@skia.org update generated includes 2018-09-13 mtklein@google.com pin in Sk4f_toL32() 2018-09-13 michaelludwig@google.com Implement sweep gradient layout 2018-09-13 stani@google.com Add GrSurfaceOrigin to MakeFromAHardwareBuffer Created with: gclient setdep -r src/third_party/skia@9548ea9d6ce1 The AutoRoll server is located here: https://autoroll.skia.org/r/skia-autoroll Documentation for the AutoRoller is here: https://skia.googlesource.com/buildbot/+/master/autoroll/README.md If the roll is causing failures, please contact the current sheriff, who should be CC'd on the roll, and stop the roller if necessary. CQ_INCLUDE_TRYBOTS=master.tryserver.blink:linux_trusty_blink_rel;luci.chromium.try:android_optional_gpu_tests_rel;luci.chromium.try:linux_optional_gpu_tests_rel;luci.chromium.try:mac_optional_gpu_tests_rel;luci.chromium.try:win_optional_gpu_tests_rel BUG=chromium:b/115610873 TBR=caryclark@chromium.org Change-Id: Ic34aa7a6b9c022c7a68c8b4f0ed7820714dc7605 Reviewed-on: https://chromium-review.googlesource.com/1225314Reviewed-by:
-
James Cook authored
It isn't used. It was added when the app list code wasn't in ash. Also remove some comments mentioning "bubble" mode, which I don't think we support any more. Bug: none Change-Id: I4f3bf461dfe3dca00399290274f6a7be2520f224 Reviewed-on: https://chromium-review.googlesource.com/1225870Reviewed-by:
Xiyuan Xia <xiyuan@chromium.org> Commit-Queue: James Cook <jamescook@chromium.org> Cr-Commit-Position: refs/heads/master@{#591186}
-
James Cook authored
It was the only subclass of WorkspaceEventHandler, so fold it into the superclass. This is left over from pre-WS2 mustash, when we had a separate ash config for mash. Bug: 842365, 866528 Test: ash_unittests, double-click window borders to resize windows Change-Id: I5de319afb6e8e00b51130a9099fc7c3e9ef70cfc Reviewed-on: https://chromium-review.googlesource.com/1225596Reviewed-by:
Jun Mukai <mukai@chromium.org> Commit-Queue: James Cook <jamescook@chromium.org> Cr-Commit-Position: refs/heads/master@{#591185}
-
Shakti Sahu authored
Added ability to add observers interested in only a single OfflineItem. Bug: 881499 Change-Id: I0b221071a0d7e258778fd37f668c8e0c5a8c2c26 Reviewed-on: https://chromium-review.googlesource.com/1213963Reviewed-by:
Min Qin <qinmin@chromium.org> Reviewed-by:
David Trainor <dtrainor@chromium.org> Commit-Queue: Shakti Sahu <shaktisahu@chromium.org> Cr-Commit-Position: refs/heads/master@{#591184}
-
Yiming Zhou authored
In the first attempt to add a DidCommitAndDrawCompositorFrame event to Chrome, I used the wrong message mechanism and inadvertently caused a massive performance regression (Change Ic1d349fd80cc1f877a3a8a604ec4266452a8186d). A subsequent fix (change I20e6fa6dbe1fd30f1529d6dda20e6208ae719bd1) removed the faulty queue message call, but also silenced the event. This fix reintroduces the event so that WebContentsObserver can monitor Chrome paint events. Bug: 854828 Change-Id: I65b31f3328a8882fa96a544a737f5213dac9a680 Reviewed-on: https://chromium-review.googlesource.com/1154139Reviewed-by:
Fady Samuel <fsamuel@chromium.org> Commit-Queue: Yiming Zhou <uwyiming@google.com> Cr-Commit-Position: refs/heads/master@{#591183}
-
v8-ci-autoroll-builder authored
Summary of changes available at: https://chromium.googlesource.com/v8/v8/+log/1eb6866c..6d775183 Please follow these instructions for assigning/CC'ing issues: https://github.com/v8/v8/wiki/Triaging%20issues Please close rolling in case of a roll revert: https://v8-roll.appspot.com/ This only works with a Google account. CQ_INCLUDE_TRYBOTS=master.tryserver.blink:linux_trusty_blink_rel;luci.chromium.try:linux_optional_gpu_tests_rel;luci.chromium.try:mac_optional_gpu_tests_rel;luci.chromium.try:win_optional_gpu_tests_rel;luci.chromium.try:android_optional_gpu_tests_rel TBR=hablich@chromium.org,v8-waterfall-sheriff@grotations.appspotmail.com Change-Id: If7b183949ef876eab578dc2e53fb538be89043cb Reviewed-on: https://chromium-review.googlesource.com/1225305Reviewed-by:
V8 Autoroller <v8-ci-autoroll-builder@chops-service-accounts.iam.gserviceaccount.com> Commit-Queue: V8 Autoroller <v8-ci-autoroll-builder@chops-service-accounts.iam.gserviceaccount.com> Cr-Commit-Position: refs/heads/master@{#591182}
-
Eric Karl authored
This reverts commit 0dd63a33. Reason for revert: Causes flakiness for a large number of tests. See crbug.com/882951 Original change's description: > Canvas/WebGL: Defer commits to limit queued frames. > > This change adds more backpressure to Canvas/WebGL rendering in GPU > bound situations. > > Before, the renderer could have three or four frames of drawing commands > queued and waiting for execution by the GPU process. The GPU process > does not know which drawing commands are associated with which display > frame. Sometimes it would run multiple frames of queued drawing commands > in one display frame, causing frame rate hiccups. > > Now when the TextureLayer has more than one buffer in flight, it calls > DeferCommits to block the next requestAnimationFrame callback from > running until one of the buffers is returned, indicating that the GPU > process has executed the commands for that frame. The shorter pipeline > reduces latency and prevents the GPU process from trying to execute > multiple Canvas/WebGL frames in one display frame. > > Bug: 835353, 879658 > Cq-Include-Trybots: luci.chromium.try:android_optional_gpu_tests_rel;luci.chromium.try:linux_layout_tests_slimming_paint_v2;luci.chromium.try:win_optional_gpu_tests_rel;master.tryserver.blink:linux_trusty_blink_rel > Change-Id: I7e5484add5fbd79757b205b1124ed7c2c5836dbb > Reviewed-on: https://chromium-review.googlesource.com/1199699 > Commit-Queue: Eric Karl <ericrk@chromium.org> > Reviewed-by: Sunny Sachanandani <sunnyps@chromium.org> > Reviewed-by: Chris Harrelson <chrishtr@chromium.org> > Cr-Commit-Position: refs/heads/master@{#589784} TBR=chrishtr@chromium.org,sunnyps@chromium.org,ericrk@chromium.org # Not skipping CQ checks because original CL landed > 1 day ago. Bug: 835353, 879658, 882951, 883364, 883358, 883057, 882618 Change-Id: I3bee10c8f43c2868966cedb638afbb4b0d9babab Cq-Include-Trybots: luci.chromium.try:android_optional_gpu_tests_rel;luci.chromium.try:linux_layout_tests_slimming_paint_v2;luci.chromium.try:win_optional_gpu_tests_rel;master.tryserver.blink:linux_trusty_blink_rel Reviewed-on: https://chromium-review.googlesource.com/1224456 Commit-Queue: Eric Karl <ericrk@chromium.org> Reviewed-by:
Eric Karl <ericrk@chromium.org> Cr-Commit-Position: refs/heads/master@{#591181}
-
