- 11 Aug, 2018 23 commits
-
-
siyua authored
This is the 1st CL adding view skeleton. For project details please refer to go/ib-paradise. Bug: 852904 Change-Id: Icc10a2598bb15659a6478fc2dbf2734094111494 Reviewed-on: https://chromium-review.googlesource.com/1149514 Commit-Queue: Siyu An <siyua@chromium.org> Reviewed-by:
Fabio Tirelo <ftirelo@chromium.org> Reviewed-by:
Sebastien Seguin-Gagnon <sebsg@chromium.org> Reviewed-by:
Bret Sepulveda <bsep@chromium.org> Reviewed-by:
Cait Phillips <caitkp@chromium.org> Reviewed-by:
Jared Saul <jsaul@google.com> Cr-Commit-Position: refs/heads/master@{#582413}
-
Antoine Labour authored
This generalizes the FlushParams to contain sync tokens + IPC::Message instead of ad-hoc fields transformed into Messages on the receiving side. This allows any Message (that doesn't have handles) to be included into the list, and simplifies the service-side handling. Existing AsyncFlush and DestroyTransferBuffer messages are now serialized client-side, and CreateSharedImage/DestroySharedImage are converted to this mechanism, to avoid extra IPCs per tile. Bug: 870116 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: Ib11298c03e249f49aec8e710a3db4195aea33e81 Reviewed-on: https://chromium-review.googlesource.com/1168731Reviewed-by:
Daniel Cheng <dcheng@chromium.org> Reviewed-by:
Sunny Sachanandani <sunnyps@chromium.org> Commit-Queue: Antoine Labour <piman@chromium.org> Cr-Commit-Position: refs/heads/master@{#582412}
-
Scott Violet authored
This makes Ash create an Env if passed SingleProcessMash. In addition this makes all window creation go through a factory that ensures the right env is supplied. See https://docs.google.com/document/d/11ha_KioDdXe4iZS2AML1foKnCJlNKm7Q1hFr6VW8dV4/edit for more details. This also adds the feature to ui/base. BUG=847992 TEST=covered by tests Change-Id: Ica3c433627222f0e076dddaf19b1ab09168cebeb Reviewed-on: https://chromium-review.googlesource.com/1168282 Commit-Queue: Scott Violet <sky@chromium.org> Reviewed-by:
James Cook <jamescook@chromium.org> Cr-Commit-Position: refs/heads/master@{#582411}
-
Jun Mukai authored
Previously window_finder_chromeos (for mash) looks through the list of the toplevel browser windows, but it is too simple and doesn't work well on several cases. This CL moves the window finding code to ash, and let WindowTree notify the list of windows at the cursor to the client upon window moving. BUG=867074 TEST=ash_unittests Change-Id: Ib44ad1a927a31c50cc108eee92ddba4d9bd918ee Reviewed-on: https://chromium-review.googlesource.com/1159346 Commit-Queue: Jun Mukai <mukai@chromium.org> Reviewed-by:
Tom Sepez <tsepez@chromium.org> Reviewed-by:
Scott Violet <sky@chromium.org> Cr-Commit-Position: refs/heads/master@{#582410}
-
Matthew Cary authored
Bug: 873380 Change-Id: Iaa194b57f57985a6bc851a2b826555a6d087a57c Reviewed-on: https://chromium-review.googlesource.com/1171234 Commit-Queue: Matthew Cary <mattcary@chromium.org> Reviewed-by:
Benoit L <lizeb@chromium.org> Cr-Commit-Position: refs/heads/master@{#582409}
-
depot-tools-chromium-autoroll authored
https://chromium.googlesource.com/chromium/tools/depot_tools.git/+log/284b5e0c60ac..6a4e31b4882f git log 284b5e0c60ac..6a4e31b4882f --date=short --no-merges --format='%ad %ae %s' 2018-08-10 ehmaldonado@chromium.org gclient_scm: Add option to specify the branch the patch is based on. Created with: gclient setdep -r src/third_party/depot_tools@6a4e31b4882f The AutoRoll server is located here: https://depot-tools-chromium-roll.skia.org 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. BUG=chromium:870279 TBR=agable@chromium.org Change-Id: I4563fb1540b017267659c6d67ca8dcbf6f3feb8f Reviewed-on: https://chromium-review.googlesource.com/1171443Reviewed-by:
depot-tools-chromium-autoroll <depot-tools-chromium-autoroll@skia-buildbots.google.com.iam.gserviceaccount.com> Commit-Queue: depot-tools-chromium-autoroll <depot-tools-chromium-autoroll@skia-buildbots.google.com.iam.gserviceaccount.com> Cr-Commit-Position: refs/heads/master@{#582408}
-
Wenzhao Zang authored
Bug: 873340 Change-Id: Ia5979ba5305b154a27689157db573f8bc6803801 Reviewed-on: https://chromium-review.googlesource.com/1171790Reviewed-by:
Alexander Alekseev <alemate@chromium.org> Commit-Queue: Wenzhao (Colin) Zang <wzang@chromium.org> Cr-Commit-Position: refs/heads/master@{#582407}
-
Stephen Martinis authored
This reverts commit 2db41802. Reason for revert: Bots are purple, will reland and make sure they're green later. Original change's description: > Add debug chromium bots > > These are mirrors of the existing release bots > > Bug: 867725 > Change-Id: I6f59d04a82103572e801273e294534e822d3e03d > Reviewed-on: https://chromium-review.googlesource.com/1159172 > Commit-Queue: Stephen Martinis <martiniss@chromium.org> > Reviewed-by: Nodir Turakulov <nodir@chromium.org> > Reviewed-by: Dirk Pranke <dpranke@chromium.org> > Reviewed-by: Kenneth Russell <kbr@chromium.org> > Cr-Commit-Position: refs/heads/master@{#582390} TBR=iannucci@chromium.org,vadimsh@chromium.org,dpranke@chromium.org,kbr@chromium.org,nodir@chromium.org,martiniss@chromium.org,jbudorick@chromium.org Change-Id: I28a27093f09db79c83239070325254df6a1a0dca No-Presubmit: true No-Tree-Checks: true No-Try: true Bug: 867725 Reviewed-on: https://chromium-review.googlesource.com/1171430Reviewed-by:
Stephen Martinis <martiniss@chromium.org> Commit-Queue: Stephen Martinis <martiniss@chromium.org> Cr-Commit-Position: refs/heads/master@{#582406}
-
Lei Zhang authored
- Make some methods const. - Update some comments. - Remove an unimplemented method declaration. - Remove an unused method parameter. Change-Id: I678d740f4915b4d55d5be5fdc6482b7bac18778b Reviewed-on: https://chromium-review.googlesource.com/1170401 Commit-Queue: Lei Zhang <thestig@chromium.org> Reviewed-by:
Wei Li <weili@chromium.org> Cr-Commit-Position: refs/heads/master@{#582405}
-
John Chen authored
The previous change (r581945) to update version string used by ChromeDriver in release branches didn't work, because the top-most commit doesn't have the necessary branch info. This CL adds the same commit filter used to generate the version string shown by Chrome. Bug: chromedriver:2080 Change-Id: I4d7b57a139968d3e5e0f1ebad26d6ed648ab5d72 Reviewed-on: https://chromium-review.googlesource.com/1171843Reviewed-by:
Caleb Rouleau <crouleau@chromium.org> Commit-Queue: John Chen <johnchen@chromium.org> Cr-Commit-Position: refs/heads/master@{#582404}
-
Adolfo Victoria authored
* Add function GetEndOfCurrent interval to obtain the last end of the interval which contains the current time. * Add function MaterializeIntervalsToLocalTimezone, which materializes the given intervals to the current local timezone. * Add function TimeIsInsideCrosSettingsIntervals, which checks if the current time is inside the intervals set in cros settings. It then returns the localized string of the end of the interval that contains the current time. BUG=chromium:852860 TEST=added unittests Change-Id: Ife32e31c8a929827fd14ab9f108c49eacb9c32c3 Reviewed-on: https://chromium-review.googlesource.com/1132676Reviewed-by:
Maksim Ivanov <emaxx@chromium.org> Commit-Queue: Maksim Ivanov <emaxx@chromium.org> Cr-Commit-Position: refs/heads/master@{#582403}
-
angle-chromium-autoroll authored
https://chromium.googlesource.com/angle/angle.git/+log/077cd858b2d4..ea926a362b77 git log 077cd858b2d4..ea926a362b77 --date=short --no-merges --format='%ad %ae %s' 2018-08-10 jmadill@chromium.org Add angle_perftests README file. 2018-08-10 jmadill@chromium.org D3D: Use angle::Result error pattern. 2018-08-10 jmadill@chromium.org Fix AMD detection in angle_end2end_tests. Created with: gclient setdep -r src/third_party/angle@ea926a362b77 The AutoRoll server is located here: https://angle-chromium-roll.skia.org 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 TBR=ynovikov@chromium.org Change-Id: I66ef52bae109017756c16854706a5a4b95d73c23 Reviewed-on: https://chromium-review.googlesource.com/1171403Reviewed-by:
angle-chromium-autoroll <angle-chromium-autoroll@skia-buildbots.google.com.iam.gserviceaccount.com> Commit-Queue: angle-chromium-autoroll <angle-chromium-autoroll@skia-buildbots.google.com.iam.gserviceaccount.com> Cr-Commit-Position: refs/heads/master@{#582402}
-
Emil A Eklund authored
https://chromium.googlesource.com/external/github.com/harfbuzz/harfbuzz/+log/2b76767bf572364d3d647cdd139f2044a7ad06b2..b6fdcf4f8bd09e065c767939125861c9dc8ff18f Tbr: drott@chromium.org Change-Id: Ie65cb46846224bd90ffa86517a946c44f321f2c7 Reviewed-on: https://chromium-review.googlesource.com/1170170Reviewed-by:
Emil A Eklund <eae@chromium.org> Commit-Queue: Emil A Eklund <eae@chromium.org> Cr-Commit-Position: refs/heads/master@{#582401}
-
Tatiana Buldina authored
Fix a test failure in Python test case testWindowMinimize, and clean up code related to not supported version. Change-Id: I076cdc90cdaca322bc64530c6bb9eabe6ae9069a Reviewed-on: https://chromium-review.googlesource.com/1171805Reviewed-by:
John Chen <johnchen@chromium.org> Commit-Queue: Tatiana Buldina <buldina@chromium.org> Cr-Commit-Position: refs/heads/master@{#582400}
-
catapult-chromium-autoroll authored
https://chromium.googlesource.com/catapult.git/+log/5a4110982f31..21cd5a309631 git log 5a4110982f31..21cd5a309631 --date=short --no-merges --format='%ad %ae %s' 2018-08-10 dtu@chromium.org [pinpoint] Ignore comparison_magnitude == 'None' for reruns. Created with: gclient setdep -r src/third_party/catapult@21cd5a309631 The AutoRoll server is located here: https://catapult-roll.skia.org 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:873331 TBR=sullivan@chromium.org Change-Id: Ie322f1501b9f5411006fd4d603edf90941097a93 Reviewed-on: https://chromium-review.googlesource.com/1171404Reviewed-by:
catapult-chromium-autoroll <catapult-chromium-autoroll@skia-buildbots.google.com.iam.gserviceaccount.com> Commit-Queue: catapult-chromium-autoroll <catapult-chromium-autoroll@skia-buildbots.google.com.iam.gserviceaccount.com> Cr-Commit-Position: refs/heads/master@{#582399}
-
Dale Curtis authored
Prevents any calls into the native bridge once release() has been called on MediaCodecBridge. I've also reordered some member variables of the bridge to match their lifetime. BUG=873094 TEST=none Change-Id: Iccf648275bac2d466ce62c5e4e792f9a088b09a0 Reviewed-on: https://chromium-review.googlesource.com/1171836Reviewed-by:
Thomas Guilbert <tguilbert@chromium.org> Commit-Queue: Dale Curtis <dalecurtis@chromium.org> Cr-Commit-Position: refs/heads/master@{#582398}
-
Zhenyao Mo authored
Excluding some large size data in third_party/catapult/ that are not needed in running the telemetry tests. R=kbr@chromium.org,nednguyen@chromium.org Change-Id: I10a1968b5a9b02415f5d7f66b33a247d64791578 Bug: 872521 Reviewed-on: https://chromium-review.googlesource.com/1166261Reviewed-by:
Ned Nguyen <nednguyen@google.com> Reviewed-by:
Kenneth Russell <kbr@chromium.org> Commit-Queue: Zhenyao Mo <zmo@chromium.org> Cr-Commit-Position: refs/heads/master@{#582397}
-
Cathy Li authored
Bug: 873124 Change-Id: I5f5f5af481e9f8e31a1dab141b9b4dac93acc121 Reviewed-on: https://chromium-review.googlesource.com/1171742Reviewed-by:
Peter Williamson <petewil@chromium.org> Commit-Queue: Cathy Li <chili@chromium.org> Cr-Commit-Position: refs/heads/master@{#582396}
-
Krishna Govind authored
R=amineer@chromium.org Cq-Include-Trybots: luci.chromium.try:ios-simulator-full-configs;master.tryserver.chromium.mac:ios-simulator-cronet Change-Id: I5429bacaf530263c0fe78c4403624d6773d12866 Reviewed-on: https://chromium-review.googlesource.com/1171858Reviewed-by:
Krishna Govind <govind@chromium.org> Cr-Commit-Position: refs/heads/master@{#582395}
-
Sebastien Marchand authored
This moves the PageAlmostIdle and SiteCharacteristicsDatabase features to their own experiment so they can easily be shipped to Beta only. Change-Id: I56c65e4ffb8b389b502a8c03d661ca2c26a66abe Reviewed-on: https://chromium-review.googlesource.com/1168107Reviewed-by:
Chris Hamilton <chrisha@chromium.org> Reviewed-by:
Robert Kaplow (slow) <rkaplow@chromium.org> Commit-Queue: Sébastien Marchand <sebmarchand@chromium.org> Cr-Commit-Position: refs/heads/master@{#582394}
-
Manu Cornet authored
This will allow us to differentiate between "running" and "active". The logic to set this properly depending on status will come in a separate CL. Note that no shelf item is ever marked as STATUS_ACTIVE yet, so this CL has no effect on behavior. Bug: 805612 Change-Id: I50f985d66659bd35e2b59ab19a898ae5a3634179 Reviewed-on: https://chromium-review.googlesource.com/1170394 Commit-Queue: Manu Cornet <manucornet@chromium.org> Reviewed-by:
Xiyuan Xia <xiyuan@chromium.org> Reviewed-by:
Dominick Ng <dominickn@chromium.org> Cr-Commit-Position: refs/heads/master@{#582393}
-
Kenneth Russell authored
It looks like an oversight that they weren't run on this configuration previously. Tbr: geofflang@chromium.org Bug: 872302 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: I77e4e175a449cea70254dcc3bab8ea3686fc74d9 Reviewed-on: https://chromium-review.googlesource.com/1168464Reviewed-by:
Geoff Lang <geofflang@chromium.org> Reviewed-by:
Kenneth Russell <kbr@chromium.org> Cr-Commit-Position: refs/heads/master@{#582392}
-
Matthew Jones authored
This patch changes the container that the new tab animation runs in when Duet is enabled. The container changes position from running on top of the toolbars to running beneath them (since the bottom toolbar is a persistent fixture). This would negatively impact the top toolbar if there were still icons there (since the top toolbar has an appearance animation to compensate for the tab appearing on top of it). However, since there are no icons, it is not an issue. Bug: 852012 Change-Id: I19422b70844f3fd45e7d9318a3c9c9768b407991 Reviewed-on: https://chromium-review.googlesource.com/1171819 Commit-Queue: Matthew Jones <mdjones@chromium.org> Reviewed-by:
Pedro Amaral <amaralp@chromium.org> Cr-Commit-Position: refs/heads/master@{#582391}
-
- 10 Aug, 2018 17 commits
-
-
Stephen Martinis authored
These are mirrors of the existing release bots Bug: 867725 Change-Id: I6f59d04a82103572e801273e294534e822d3e03d Reviewed-on: https://chromium-review.googlesource.com/1159172 Commit-Queue: Stephen Martinis <martiniss@chromium.org> Reviewed-by:
Nodir Turakulov <nodir@chromium.org> Reviewed-by:
Dirk Pranke <dpranke@chromium.org> Reviewed-by:
Kenneth Russell <kbr@chromium.org> Cr-Commit-Position: refs/heads/master@{#582390}
-
Pete Williamson authored
Sometimes when sharing temporary offline pages, we fail to create the content URI. If we do, fall back to sharing by URL with the URL of the page. Bug: 871697 Change-Id: Ie2e895fd708fdf4e5e992ebc2317ee49fd5bcee3 Reviewed-on: https://chromium-review.googlesource.com/1168587Reviewed-by:
Yafei Duan <romax@chromium.org> Commit-Queue: Peter Williamson <petewil@chromium.org> Cr-Commit-Position: refs/heads/master@{#582389}
-
Boris Sazonov authored
This CL updates the layout of the streamlined sign-in screen and the consent bump screen to accommodate last changes to mocks. Bug: 814728 Change-Id: If8f2cdaf39ba175ca60584f9015a4b2ccaf23217 Reviewed-on: https://chromium-review.googlesource.com/1171226Reviewed-by:
Theresa <twellington@chromium.org> Commit-Queue: Boris Sazonov <bsazonov@chromium.org> Cr-Commit-Position: refs/heads/master@{#582388}
-
Lei Zhang authored
Change-Id: I3c0930f22e7ff3ff72611bfb70415b3e122f398e Reviewed-on: https://chromium-review.googlesource.com/1171378Reviewed-by:
Wei Li <weili@chromium.org> Commit-Queue: Lei Zhang <thestig@chromium.org> Cr-Commit-Position: refs/heads/master@{#582387}
-
kristipark authored
The "Restore default shortcuts" and "Restore default background" options are swapped when navigating with arrow keys. Bug: 872978 Change-Id: I6078a6fb0a004a81e321a39f08e1f024bf53e9e1 Reviewed-on: https://chromium-review.googlesource.com/1170195 Commit-Queue: Kristi Park <kristipark@chromium.org> Reviewed-by:
Mathieu Perreault <mathp@chromium.org> Cr-Commit-Position: refs/heads/master@{#582386}
-
Justin DeWitt authored
This will host the ExploreSites datastore as well as the downloader. BUG=867488 Change-Id: Ifff84b6c8316033929ccbd52c32c2f175af117c8 Reviewed-on: https://chromium-review.googlesource.com/1169588Reviewed-by:
Cathy Li <chili@chromium.org> Commit-Queue: Justin DeWitt <dewittj@chromium.org> Cr-Commit-Position: refs/heads/master@{#582385}
-
Wei-Yin Chen (陳威尹) authored
third_party/ is excluded, except for third_party/WebKit and third_party/blink. This is uncovered by: https://chromium-swarm.appspot.com/task?id=3f2afb7b4290f910 Bug: None Cq-Include-Trybots: luci.chromium.try:android_optional_gpu_tests_rel;luci.chromium.try:ios-simulator-full-configs;luci.chromium.try:win_optional_gpu_tests_rel;master.tryserver.blink:linux_trusty_blink_rel;master.tryserver.chromium.mac:ios-simulator-cronet Change-Id: Ie17ea50e7f6aba3538acb488df92b4926b52f4c4 Reviewed-on: https://chromium-review.googlesource.com/1170441Reviewed-by:
Philip Jägenstedt <foolip@chromium.org> Reviewed-by:
Scott Violet <sky@chromium.org> Commit-Queue: Wei-Yin Chen (陳威尹) <wychen@chromium.org> Cr-Commit-Position: refs/heads/master@{#582384}
-
Chris Lu authored
Change-Id: Ib6bcee51114a9fcb05aacd9079d52b65356e18d2 Reviewed-on: https://chromium-review.googlesource.com/1171680 Commit-Queue: Chris Lu <thegreenfrog@chromium.org> Reviewed-by:
Sergio Collazos <sczs@chromium.org> Cr-Commit-Position: refs/heads/master@{#582383}
-
pdfium-chromium-autoroll authored
https://pdfium.googlesource.com/pdfium.git/+log/e1a7e8589a8c..f36a4644b828 git log e1a7e8589a8c..f36a4644b828 --date=short --no-merges --format='%ad %ae %s' 2018-08-10 rharrison@chromium.org Remove direct calls to time 2018-08-10 rharrison@chromium.org Add proxy for syscall time 2018-08-10 tsepez@chromium.org Remove |typedef struct| in favor of just |struct|. 2018-08-10 tsepez@chromium.org Consolidate const_casts<> in FXFT_Get_Name_Index() 2018-08-10 thestig@chromium.org Make CFX_ImageTransformer::CalcData's bitmap non-const. 2018-08-10 thestig@chromium.org Fix a JBIG2 decoding regression. 2018-08-10 thestig@chromium.org Remove deprecated flags from fpdf_progressive.h. Created with: gclient setdep -r src/third_party/pdfium@f36a4644b828 The AutoRoll server is located here: https://pdfium-roll.skia.org 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. BUG=chromium:871848 TBR=dsinclair@chromium.org Change-Id: I22721ff61265f49fddd6ffc647ef37cc2f70bc0c Reviewed-on: https://chromium-review.googlesource.com/1171442Reviewed-by:
pdfium-chromium-autoroll <pdfium-chromium-autoroll@skia-buildbots.google.com.iam.gserviceaccount.com> Commit-Queue: pdfium-chromium-autoroll <pdfium-chromium-autoroll@skia-buildbots.google.com.iam.gserviceaccount.com> Cr-Commit-Position: refs/heads/master@{#582382}
-
Takuto Ikuta authored
I increased shards for some tests took time on https://ci.chromium.org/p/chromium/builders/luci.chromium.try/android-marshmallow-arm64-rel/59189 Bug: 868114 Change-Id: I9ae5c80e0fe641f8946e0d00d8a322721d2de079 Reviewed-on: https://chromium-review.googlesource.com/1170922Reviewed-by:
Nico Weber <thakis@chromium.org> Commit-Queue: Takuto Ikuta <tikuta@chromium.org> Cr-Commit-Position: refs/heads/master@{#582381}
-
Mason Freed authored
Especially within paint, most calls to LayoutObject.Style() are assuming a non-null pointer and dereferencing it. To better document the fact that these calls (properly, in most cases) assume the layout object contains style, they should be replaced with calls to the StyleRef() function, which dereferences the pointer. Bug: 872508 Change-Id: Id6a04871c12a2c09d1029ca5aa04ae49099f7fd0 Cq-Include-Trybots: luci.chromium.try:linux_layout_tests_slimming_paint_v2;master.tryserver.blink:linux_trusty_blink_rel Reviewed-on: https://chromium-review.googlesource.com/1171144Reviewed-by:
Chris Harrelson <chrishtr@chromium.org> Commit-Queue: Mason Freed <masonfreed@google.com> Cr-Commit-Position: refs/heads/master@{#582380}
-
Xi Cheng authored
Since SampleOrderingSpec::MAY_SHUFFLE is the only one used in production code, there is no need to specify the SampleOrderingSpec parameter hence SampleOrderingSpec enum is also retired. Bug: 851163 Change-Id: Ie9bf7c39d036db55b1533c4fe3b3f3293a2a8dbf Reviewed-on: https://chromium-review.googlesource.com/1171584 Commit-Queue: Xi Cheng <chengx@chromium.org> Reviewed-by:
Mike Wittman <wittman@chromium.org> Cr-Commit-Position: refs/heads/master@{#582379}
-
Christopher Cameron authored
Disalbing the CATransactionGPUCoordinator caused memory corruption bugs to go away. Fix two potential bugs: 1. Don't post tasks (and potentially change the reference count) inside CATransactionGPUCoordinator's constructor. Do this in a separate explicit Init function. 1a. Move this initialization to the end of GpuProcessHost:Init (instead of being at the beginning of the constructor). 2. Make CATransactionCoordinator explicitly retain PostCommitObserver (which includes CATransactionGPUCoordinator). This fixes a bug whereby at shutdown, destroying not-yet-executed posted tasks caused the CATransactionGPUCoordinator to be destroyed while the CATransactionCoordinator still had a raw pointer to it. Bug: 871430 Change-Id: Ie144071cce9ce48e0187cdaf1fcf32df7b62ed75 Reviewed-on: https://chromium-review.googlesource.com/1171657 Commit-Queue: ccameron <ccameron@chromium.org> Reviewed-by:Antoine Labour <piman@chromium.org> Cr-Commit-Position: refs/heads/master@{#582378}
-
Donn Denman authored
Change-Id: I100542f77eb1ebebf28db6c716733467be61c209 Reviewed-on: https://chromium-review.googlesource.com/1168503Reviewed-by:
Theresa <twellington@chromium.org> Commit-Queue: Donn Denman <donnd@chromium.org> Cr-Commit-Position: refs/heads/master@{#582377}
-
Eric Roman authored
Change-Id: Ibbde1b31fdeade51caf0cb70fefbe915bd7a9af6 Reviewed-on: https://chromium-review.googlesource.com/1169693 Commit-Queue: Eric Roman <eroman@chromium.org> Reviewed-by:
Ken Rockot <rockot@chromium.org> Cr-Commit-Position: refs/heads/master@{#582376}
-
Erik Chen authored
Before this CL, DevTools would crash the browser if it did not get a response in 5 seconds. This timeout is too low for Windows. On Catapult perf tests on Windows, it takes ~3 seconds for the GPU process to respond on average, and sometimes it takes more than 5 seconds. Bug: 873112 Change-Id: I67ee82a1a75df0ba9e1b483a58c66c56a8bf2213 Reviewed-on: https://chromium-review.googlesource.com/1170584Reviewed-by:
Kenneth Russell <kbr@chromium.org> Reviewed-by:
Andrey Kosyakov <caseq@chromium.org> Commit-Queue: Erik Chen <erikchen@chromium.org> Cr-Commit-Position: refs/heads/master@{#582375}
-
Sarah Hu authored
This will be used by both oobe and settings. Bug: 873229 Change-Id: I1bb3ff8e9666e967ff5445d68085801d009894d5 Reviewed-on: https://chromium-review.googlesource.com/1171243 Commit-Queue: Xiaoyin Hu <xiaoyinh@chromium.org> Reviewed-by:
Dan Erat <derat@chromium.org> Cr-Commit-Position: refs/heads/master@{#582374}
-