- 19 Jan, 2018 40 commits
-
-
Mostyn Bramley-Moore authored
These same constant names are used in MediaRemotingInterstitial.cpp, which causes the jumbo build to fail. Bug: 726619 TBR=jochen@chromium.org Change-Id: I2c2cc45f010b1212185d59a58dcf070ef0e16ee1 Reviewed-on: https://chromium-review.googlesource.com/876015 Commit-Queue: Mostyn Bramley-Moore <mostynb@vewd.com> Reviewed-by:
apacible <apacible@chromium.org> Cr-Commit-Position: refs/heads/master@{#530643}
-
Troy Hildebrandt authored
Fixes CatchFail warnings by throwing AssertionErrors with the original exception instead of calling Assert.fail with a message and no stack trace. A couple tests were using assert instead of the appropriate junit.Assert method, and causing a UseCorrectAssertInTests warning. These were fixed. Change-Id: I72089ab76197b048baa9cb319f8269613a4674fe Reviewed-on: https://chromium-review.googlesource.com/876982Reviewed-by:
Maria Khomenko <mariakhomenko@chromium.org> Commit-Queue: Troy Hildebrandt <thildebr@chromium.org> Cr-Commit-Position: refs/heads/master@{#530642}
-
Julia Tuttle authored
Cronet doesn't support controlling base::Features, so: 1. For Chromium, move the base::Feature check to NetworkContext. 2. For Cronet, add an experimental feature. TBR=mkwst@chromium.org Bug: 799616 Cq-Include-Trybots: master.tryserver.chromium.linux:linux_mojo Change-Id: I5baab77588d8f1ef9d292febca826777820b6a23 Reviewed-on: https://chromium-review.googlesource.com/858176 Commit-Queue: Julia Tuttle <juliatuttle@chromium.org> Reviewed-by:
Matt Menke <mmenke@chromium.org> Reviewed-by:
Bernhard Bauer <bauerb@chromium.org> Reviewed-by:
Misha Efimov <mef@chromium.org> Cr-Commit-Position: refs/heads/master@{#530641}
-
Troy Hildebrandt authored
Throws an AssertionError containing the stack trace instead of calling Assert.fail, and gets rid of a try/catch block for an exception that won't occur. Change-Id: Ia6342346599dd1042c555dc144fab24808129660 Reviewed-on: https://chromium-review.googlesource.com/876824Reviewed-by:
Yusuf Ozuysal <yusufo@chromium.org> Commit-Queue: Troy Hildebrandt <thildebr@chromium.org> Cr-Commit-Position: refs/heads/master@{#530640}
-
Emil A Eklund authored
Based on the existing Slimming Paint 1.75 implementation in BoxPainter. Bug: 714962 Cq-Include-Trybots: master.tryserver.blink:linux_trusty_blink_rel;master.tryserver.chromium.linux:linux_layout_tests_layout_ng;master.tryserver.chromium.linux:linux_layout_tests_slimming_paint_v2 Change-Id: Iab68c32f6f681b6775e07b0a10d960dbd0be3aaa Reviewed-on: https://chromium-review.googlesource.com/875149 Commit-Queue: Emil A Eklund <eae@chromium.org> Reviewed-by:
Koji Ishii <kojii@chromium.org> Cr-Commit-Position: refs/heads/master@{#530639}
-
Xiaocheng Hu authored
Change-Id: I7380a6b36ed8212c2fe78ba0a1ea5f68237e8efa Reviewed-on: https://chromium-review.googlesource.com/875320 Commit-Queue: Xiaocheng Hu <xiaochengh@chromium.org> Reviewed-by:
Yoshifumi Inoue <yosin@chromium.org> Cr-Commit-Position: refs/heads/master@{#530638}
-
Scott Chen authored
Bug: 801755 Change-Id: Ia22d9182283ae16abb6f73f602371b914b060a2e Reviewed-on: https://chromium-review.googlesource.com/872200Reviewed-by:
Demetrios Papadopoulos <dpapad@chromium.org> Commit-Queue: Scott Chen <scottchen@chromium.org> Cr-Commit-Position: refs/heads/master@{#530637}
-
Pavel Feldman authored
Change-Id: Ide61510df5eb10a41b608f78710aa278040586cc Reviewed-on: https://chromium-review.googlesource.com/876518Reviewed-by:
Kentaro Hara <haraken@chromium.org> Commit-Queue: Pavel Feldman <pfeldman@chromium.org> Cr-Commit-Position: refs/heads/master@{#530636}
-
Bill Budge authored
- Raises the limit of direct mapped allocations. WASM needs to be able to allocate 2GiB regions. - Manual reland. Fixed another test. Bug: chromium:801604 Change-Id: I1e19246cbc47eff22e770db84b29e6aac91e7ac9 Reviewed-on: https://chromium-review.googlesource.com/876484Reviewed-by:
Chris Palmer <palmer@chromium.org> Reviewed-by:
Kentaro Hara <haraken@chromium.org> Commit-Queue: Bill Budge <bbudge@chromium.org> Cr-Commit-Position: refs/heads/master@{#530635}
-
Jiaquan He authored
Notifies chrome that the app list view is shown so that it can take proper actions, e.g. counting badged items to record metrics. Bug: 733662 Change-Id: I5e8c7a7f9a61b9e8df347cf9349c01221a86bfa8 Reviewed-on: https://chromium-review.googlesource.com/876487Reviewed-by:
Xiyuan Xia <xiyuan@chromium.org> Reviewed-by:
Scott Violet <sky@chromium.org> Commit-Queue: Jiaquan He <hejq@google.com> Cr-Commit-Position: refs/heads/master@{#530634}
-
catapult-deps-roller@chromium.org authored
https://chromium.googlesource.com/catapult.git/+log/3318544c51d4..c4b36e2d9b77 $ git log 3318544c5..c4b36e2d9 --date=short --no-merges --format='%ad %ae %s' 2018-01-19 dtu [pinpoint] Cache Gitiles requests. 2018-01-19 dtu Make services.gitiles_service use services.request. 2018-01-19 dtu [dashboard] Add optional caching to the services.request module. 2018-01-04 eakuefner [TBMv2] Move Python Breakdown into diagnostics.Breakdown 2018-01-19 charliea Reraise exception when wmic fails Created with: roll-dep src/third_party/catapult 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=master.tryserver.chromium.android:android_optional_gpu_tests_rel TBR=sullivan@chromium.org Change-Id: I6ebd5fee93926f61a6e5a9f1e8e2fb533d42fb59 Reviewed-on: https://chromium-review.googlesource.com/876784 Commit-Queue: <catapult-deps-roller@chromium.org> Reviewed-by: <catapult-deps-roller@chromium.org> Cr-Commit-Position: refs/heads/master@{#530633}
-
Peter Collingbourne authored
All known blockers have now been fixed. This CL is likely to uncover unknown blockers, and is a likely culprit for any issues involving crash dumping or symbolization on Android. MIPS is excluded because of an incompatibility between Android MIPS prebuilts and lld (https://github.com/android-ndk/ndk/issues/617). Bug: 742655 Change-Id: Ic12eebce867f3804bb97d18566afc011fa962a26 Reviewed-on: https://chromium-review.googlesource.com/872225 Commit-Queue: Peter Collingbourne <pcc@chromium.org> Reviewed-by:
agrieve <agrieve@chromium.org> Cr-Commit-Position: refs/heads/master@{#530632}
-
dpapad authored
TBR=dtrainor Bug: 800502 Cq-Include-Trybots: master.tryserver.chromium.linux:closure_compilation Change-Id: Ie8c3da370937be820ac0622947f4693c68e03fea Reviewed-on: https://chromium-review.googlesource.com/871570 Commit-Queue: Demetrios Papadopoulos <dpapad@chromium.org> Reviewed-by:
Devlin <rdevlin.cronin@chromium.org> Cr-Commit-Position: refs/heads/master@{#530631}
-
Chromium WPT Sync authored
Using wpt-import in Chromium 52d83c98. With Chromium commits locally applied on WPT: e8a79119 "Fixed wpt csp reporting tests and moved over chrome internal ones" ea343b08 "[css-grid] Fix for flexible tracks, gutters and spanning items" 245bf895 "Catch unexpected exceptions in sync-xhr test." Build: https://ci.chromium.org/buildbot/chromium.infra.cron/wpt-importer/10384 Note to sheriffs: This CL imports external tests and adds expectations for those tests; if this CL is large and causes a few new failures, please fix the failures by adding new lines to TestExpectations rather than reverting. See: https://chromium.googlesource.com/chromium/src/+/master/docs/testing/web_platform_tests.md TBR=kereliuk No-Export: true Change-Id: Ifadf0fa750538f96c6b66b411dc55ad5785df253 Reviewed-on: https://chromium-review.googlesource.com/876308 Commit-Queue: Blink WPT Bot <blink-w3c-test-autoroller@chromium.org> Reviewed-by:
Blink WPT Bot <blink-w3c-test-autoroller@chromium.org> Cr-Commit-Position: refs/heads/master@{#530630}
-
Khushal authored
Use Read instead of ReadSimple on PaintOpReader during matrix serialization to ensure the type and matrix values are set correctly. In general, ReadSimple directly reads and sets the memory for the value and might miss type specific sanity checks, so use the Read version for available types. R=vmpstr@chromium.org Bug: 803432 Cq-Include-Trybots: master.tryserver.blink:linux_trusty_blink_rel;master.tryserver.chromium.android:android_optional_gpu_tests_rel Change-Id: I27c13806f3426fa1cad919af7a5f76f357f79d52 Reviewed-on: https://chromium-review.googlesource.com/875186 Commit-Queue: Khushal <khushalsagar@chromium.org> Reviewed-by:
vmpstr <vmpstr@chromium.org> Cr-Commit-Position: refs/heads/master@{#530629}
-
Xiaochu Liu authored
PopBool sets its parameter to be false on a failure (dbus optional parameter is missing). I reset the mount variable to be true on a failure return of PopBool. BUG=chromium:803681 TEST=test_that -b ${BOARD} 100.127.0.229 platform_AddPrinter.epson Change-Id: Ibc8981df635b121057b6e43dc3fe3dc7b317fd75 Reviewed-on: https://chromium-review.googlesource.com/876654 Commit-Queue: Xiaochu Liu <xiaochu@chromium.org> Reviewed-by:
Dan Erat <derat@chromium.org> Cr-Commit-Position: refs/heads/master@{#530628}
-
Randy Smith authored
TBR=kinuko@chromium.org Bug: 803958 Cq-Include-Trybots: master.tryserver.chromium.linux:linux_mojo Change-Id: I80bf960f2d4ec54bf60ff83c5d734571cd946c47 Reviewed-on: https://chromium-review.googlesource.com/876901 Commit-Queue: Randy Smith <rdsmith@chromium.org> Reviewed-by:
Randy Smith <rdsmith@chromium.org> Cr-Commit-Position: refs/heads/master@{#530627}
-
Fredrik Söderquist authored
Rather than passing a LayoutObject around we can pass around the <Document, ComputedStyle> tuple that is the bits of data that is needed. While a it, change CSSGradientValue::GetStopColors to return the vector of Colors instead of using an out-variable. Bug: 803224 Change-Id: Ib89bf3c0e47495886b5b715504a90b262348c6f2 Reviewed-on: https://chromium-review.googlesource.com/875925Reviewed-by:
Stephen Chenney <schenney@chromium.org> Commit-Queue: Fredrik Söderquist <fs@opera.com> Cr-Commit-Position: refs/heads/master@{#530626}
-
Daniel Cheng authored
I made the mistake of believing a DCHECK that was in the code… Bug: 803811 Change-Id: I536ff17c7bbb2ff23f199c9b51637f184d08966d Reviewed-on: https://chromium-review.googlesource.com/876649Reviewed-by:
Nate Chapin <japhet@chromium.org> Commit-Queue: Daniel Cheng <dcheng@chromium.org> Cr-Commit-Position: refs/heads/master@{#530625}
-
Sorin Jianu authored
BUG=803321 Cq-Include-Trybots: master.tryserver.chromium.mac:ios-simulator-cronet;master.tryserver.chromium.mac:ios-simulator-full-configs Change-Id: Iaf24028c071606c1565127b92372de6bc3c62da1 Reviewed-on: https://chromium-review.googlesource.com/868753 Commit-Queue: Sorin Jianu <sorin@chromium.org> Reviewed-by:
Joshua Pawlicki <waffles@chromium.org> Reviewed-by:
Scott Violet <sky@chromium.org> Reviewed-by:
Julian Pastarmov <pastarmovj@chromium.org> Reviewed-by:
Sylvain Defresne <sdefresne@chromium.org> Reviewed-by:
Devlin <rdevlin.cronin@chromium.org> Cr-Commit-Position: refs/heads/master@{#530624}
-
erikchen authored
Previously, there was a race between starting profiling for a renderer process, and acquiring a heap dump from the renderer process. This CL adds some logic to wait for the renderer to start. This CL also re-adds the GPU test, as no flakiness has been observed there. Bug: 803696 Change-Id: I6ddc00dcf42e9d2c4292dcec50f77033de26ad7d Reviewed-on: https://chromium-review.googlesource.com/876304Reviewed-by:
Maria Khomenko <mariakhomenko@chromium.org> Commit-Queue: Erik Chen <erikchen@chromium.org> Cr-Commit-Position: refs/heads/master@{#530623}
-
Charles Vazac authored
Per spec (https://w3c.github.io/server-timing/#processing-model), resources that fail the TAO check should return an empty array for the serverTiming attribute. Bug: 802321 Change-Id: I0bca0f38f0a975f7c85f4d8837a901bd8475d9b7 Reviewed-on: https://chromium-review.googlesource.com/870990 Commit-Queue: Yoav Weiss <yoav@yoav.ws> Reviewed-by:
Yoav Weiss <yoav@yoav.ws> Cr-Commit-Position: refs/heads/master@{#530622}
-
v8-autoroll authored
Summary of changes available at: https://chromium.googlesource.com/v8/v8/+log/71997847..a998a1a6 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;master.tryserver.chromium.linux:linux_optional_gpu_tests_rel;master.tryserver.chromium.mac:mac_optional_gpu_tests_rel;master.tryserver.chromium.win:win_optional_gpu_tests_rel;master.tryserver.chromium.android:android_optional_gpu_tests_rel TBR=hablich@chromium.org,machenbach@chromium.org,kozyatinskiy@chromium.org,sergiyb@chromium.org Change-Id: I4e9d79cd3d7dc789b43bd3a1f1f5cf9bdde813df Reviewed-on: https://chromium-review.googlesource.com/876064Reviewed-by:
v8 autoroll <v8-autoroll@chromium.org> Commit-Queue: v8 autoroll <v8-autoroll@chromium.org> Cr-Commit-Position: refs/heads/master@{#530621}
-
David Grogan authored
This reverts the logic, but not tests, from: 7f7beb7d ca3ed704 842ea8f4 The bottom caused netflix to render wrong, the top 2 were built on the bottom. Bug: 417223,702927,798164 Change-Id: Icb350aa0ea385937294040ce681345e3314da0e2 Reviewed-on: https://chromium-review.googlesource.com/875015Reviewed-by:
Morten Stenshorne <mstensho@chromium.org> Commit-Queue: David Grogan <dgrogan@chromium.org> Cr-Commit-Position: refs/heads/master@{#530620}
-
Chris Harrelson authored
1. Don't copy PropertyTreeState for source state, as it is not modified. This saves 1% of runtime on the referenced bug's regression test. 2. Reduce copying of LayoutPoint and LayoutRect. Cq-Include-Trybots: master.tryserver.blink:linux_trusty_blink_rel;master.tryserver.chromium.linux:linux_layout_tests_slimming_paint_v2 Change-Id: I73c584e7f9b14770abde0c5922d766f7d2d5b02e Reviewed-on: https://chromium-review.googlesource.com/875164 Commit-Queue: Chris Harrelson <chrishtr@chromium.org> Reviewed-by:
Stephen Chenney <schenney@chromium.org> Cr-Commit-Position: refs/heads/master@{#530619}
-
Troy Hildebrandt authored
Throws an AssertionError with the stack trace instead of calling Assert.fail() with just a message. Change-Id: I7ade80ee96e5feb4e5fc7a2b92f04af9c148a14b Reviewed-on: https://chromium-review.googlesource.com/876244Reviewed-by:
Ted Choc <tedchoc@chromium.org> Commit-Queue: Troy Hildebrandt <thildebr@chromium.org> Cr-Commit-Position: refs/heads/master@{#530618}
-
bsheedy authored
Adds vr_common_unittests and vr_pixeltests to chromium.linux bots now that VR is enabled by default on Linux builds. Bug: 770853 Change-Id: I4588f661713d60335569366e1ad05fb79645a41a Reviewed-on: https://chromium-review.googlesource.com/875312Reviewed-by:
Dirk Pranke <dpranke@chromium.org> Commit-Queue: Brian Sheedy <bsheedy@chromium.org> Cr-Commit-Position: refs/heads/master@{#530617}
-
Xiaocheng Hu authored
Change-Id: If483a1ac2b02eabfc9ff7d9f3a5a9619b505d5b0 Reviewed-on: https://chromium-review.googlesource.com/875315 Commit-Queue: Xiaocheng Hu <xiaochengh@chromium.org> Reviewed-by:
Yoshifumi Inoue <yosin@chromium.org> Cr-Commit-Position: refs/heads/master@{#530616}
-
Chris Harrelson authored
This emphasizes the fact that the enclosingLayer call is actually cheap because the object in question is guaranteed to have one, by the logic earlier in the method. Bug:682032 Cq-Include-Trybots: master.tryserver.blink:linux_trusty_blink_rel;master.tryserver.chromium.linux:linux_layout_tests_slimming_paint_v2 Change-Id: Ifaba3ba951acf52380a52fe1111f20d5ad2d61cd Reviewed-on: https://chromium-review.googlesource.com/876731Reviewed-by:
Stephen Chenney <schenney@chromium.org> Commit-Queue: Chris Harrelson <chrishtr@chromium.org> Cr-Commit-Position: refs/heads/master@{#530615}
-
pdfium-deps-roller@chromium.org authored
https://pdfium.googlesource.com/pdfium.git/+log/27d718ebb298..f668ed47ae54 $ git log 27d718ebb..f668ed47a --date=short --no-merges --format='%ad %ae %s' 2018-01-19 thestig Remove NeedAlpha() in CFX_ImageTransformer. Created with: roll-dep src/third_party/pdfium 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. TBR=dsinclair@chromium.org Change-Id: I3e84d02e83fea4a8f0d16329721f7d83eb636171 Reviewed-on: https://chromium-review.googlesource.com/876519 Reviewed-by: <pdfium-deps-roller@chromium.org> Commit-Queue: <pdfium-deps-roller@chromium.org> Cr-Commit-Position: refs/heads/master@{#530614}
-
Ken Rockot authored
This flakily times-out on ASAN bots. Bug: 803814 Change-Id: I2639e4b36c6dff9021744bc753805b085e161a4a TBR=jamescook Change-Id: I2639e4b36c6dff9021744bc753805b085e161a4a Reviewed-on: https://chromium-review.googlesource.com/876727Reviewed-by:
Ken Rockot <rockot@chromium.org> Commit-Queue: Ken Rockot <rockot@chromium.org> Cr-Commit-Position: refs/heads/master@{#530613}
-
Zhenyao Mo authored
BUG=744658 TEST=bots R=halliwell@chromium.org,piman@chromium.org,kbr@chromium.org Cq-Include-Trybots: master.tryserver.chromium.android:android_optional_gpu_tests_rel;master.tryserver.chromium.linux:linux_optional_gpu_tests_rel;master.tryserver.chromium.mac:mac_optional_gpu_tests_rel;master.tryserver.chromium.win:win_optional_gpu_tests_rel Change-Id: Ib8e4e5e874025d78c7cdc2cc3b7af6c9e20ca9c4 Reviewed-on: https://chromium-review.googlesource.com/874612 Commit-Queue: Zhenyao Mo <zmo@chromium.org> Reviewed-by:
Stephen Lanham <slan@chromium.org> Reviewed-by:
Kenneth Russell <kbr@chromium.org> Reviewed-by:
Antoine Labour <piman@chromium.org> Reviewed-by:
Luke Halliwell <halliwell@chromium.org> Cr-Commit-Position: refs/heads/master@{#530612}
-
Andrew Grieve authored
There appears to be no use of it outside of testdata: https://cs.chromium.org/search/?q=file:%5C.grd%5Cb+-output_all_resource_defines Change-Id: I053cd83d18c19cacea0eb1e6b31f3c0eec5e4f17 Reviewed-on: https://chromium-review.googlesource.com/786121 Commit-Queue: agrieve <agrieve@chromium.org> Reviewed-by:
Robert Flack <flackr@chromium.org> Cr-Commit-Position: refs/heads/master@{#530611}
-
Marijn Kruisselbrink authored
Before the caller of BuildBlob/AddFinishedBlob retained ownership of the BlobDataBuilder, only passing in a reference. After this change instead ownership is passed to BlobStorageContext. This change will help with some follow-up refactoring of BlobDataBuilder. TBR=kinuko@chromium.org, xingliu@chromium.org Bug: 803585 Change-Id: Iccbad9b3ecd3c6e510afc9fe6e65c34a2c738dbf Reviewed-on: https://chromium-review.googlesource.com/876499 Commit-Queue: Marijn Kruisselbrink <mek@chromium.org> Reviewed-by:
Daniel Murphy <dmurph@chromium.org> Cr-Commit-Position: refs/heads/master@{#530610}
-
Kenneth Russell authored
BUG=801578 TBR=nednguyen@google.com, perezju@chromium.org Cq-Include-Trybots: master.tryserver.chromium.android:android_optional_gpu_tests_rel;master.tryserver.chromium.linux:linux_optional_gpu_tests_rel;master.tryserver.chromium.mac:mac_optional_gpu_tests_rel;master.tryserver.chromium.win:win_optional_gpu_tests_rel Change-Id: I0e5f31cb6912dcd1bc5da54108b66639beda546a Reviewed-on: https://chromium-review.googlesource.com/876653 Commit-Queue: Kenneth Russell <kbr@chromium.org> Reviewed-by:
Kenneth Russell <kbr@chromium.org> Cr-Commit-Position: refs/heads/master@{#530609}
-
Nate Chapin authored
Bug: 624061 Change-Id: I0b863938467b81dae3fbecd17b23aaf6faffe2a7 Reviewed-on: https://chromium-review.googlesource.com/876508Reviewed-by:
Ojan Vafai <ojan@chromium.org> Reviewed-by:
Nasko Oskov <nasko@chromium.org> Commit-Queue: Nate Chapin <japhet@chromium.org> Cr-Commit-Position: refs/heads/master@{#530608}
-
Ahmed Fakhry authored
Fix a bug resulting in `onUpdatedWithMemory` listeners getting the memory stats only without the other stats that `onUpdated` listeners get. This used to happen when there are no `onUpdated` listeners. BUG=802885 TEST=Added new test Change-Id: I3db2673273391b97b509529ad04a6d0c1a8251c2 Reviewed-on: https://chromium-review.googlesource.com/868838 Commit-Queue: Ahmed Fakhry <afakhry@chromium.org> Reviewed-by:
Devlin <rdevlin.cronin@chromium.org> Reviewed-by:
Toni Barzic <tbarzic@chromium.org> Reviewed-by:
Charlie Reis <creis@chromium.org> Cr-Commit-Position: refs/heads/master@{#530607}
-
Vladimir Levin authored
This patch adds two use counters: 1. ImageDecodeAPI measuring the usage of the image decode api 2. ImageDecodingAttribute measuring the usage of the <img> decoding attribute. R=chrishtr@chromium.org Bug: 803707 Cq-Include-Trybots: master.tryserver.blink:linux_trusty_blink_rel;master.tryserver.chromium.linux:linux_layout_tests_slimming_paint_v2 Change-Id: I398131cb29409d5f6aa1c5f8e0f613d545c288a8 Reviewed-on: https://chromium-review.googlesource.com/876517Reviewed-by:
Chris Harrelson <chrishtr@chromium.org> Commit-Queue: vmpstr <vmpstr@chromium.org> Cr-Commit-Position: refs/heads/master@{#530606}
-
xlai authored
The restoration of canvas matrix clip stack should happen after the image drawing operation from old buffer to new buffer, when canvas 2d acceleration is being disabled. Follow-up tests would be landed in later CL. Bug: 803950 Change-Id: I8a393381f9813853ab966da71244b2e9f4aceb48 Reviewed-on: https://chromium-review.googlesource.com/876528 Commit-Queue: Olivia Lai <xlai@chromium.org> Reviewed-by:
Justin Novosad <junov@chromium.org> Cr-Commit-Position: refs/heads/master@{#530605}
-
Emircan Uysaler authored
This CL fixes the iteration bug. Additionally, it refactors classes so that this can be testable by adding an interface: RenderProcessHostDelegate. This interface is mocked to track number of function calls. Also, it changes |render_process_id| from int to uint32_t. Bug: 802216 Change-Id: I31fb05111ab1d63e5caa1a81e0b6feee55bcecae Reviewed-on: https://chromium-review.googlesource.com/869251 Commit-Queue: Emircan Uysaler <emircan@chromium.org> Reviewed-by:
Christian Fremerey <chfremer@chromium.org> Cr-Commit-Position: refs/heads/master@{#530604}
-