- 18 Oct, 2017 40 commits
-
-
Kunihiko Sakamoto authored
This patch - Moves the face_ member of CSSFontFaceSource into RemoteFontFaceSource - Sets face_ in the constructor so it's non-nullable - Moves PruneTable() from RemoteFontFaceSource to CSSFontFaceSource Pure refactoring, no behavior change. Change-Id: I1e4ec4951c1856d74d37a8a8164de3bc5d9c5399 Reviewed-on: https://chromium-review.googlesource.com/722141Reviewed-by:
Takashi Toyoshima <toyoshim@chromium.org> Reviewed-by:
Kinuko Yasuda <kinuko@chromium.org> Commit-Queue: Kunihiko Sakamoto <ksakamoto@chromium.org> Cr-Commit-Position: refs/heads/master@{#509730}
-
Andy Paicu authored
Bug: 651742 Change-Id: I5d03992fa5cbbb492c5091f008b1df8967113bfe Reviewed-on: https://chromium-review.googlesource.com/725341Reviewed-by:
Mike West <mkwst@google.com> Reviewed-by:
Mike West <mkwst@chromium.org> Commit-Queue: Andy Paicu <andypaicu@chromium.org> Commit-Queue: Mike West <mkwst@chromium.org> Cr-Commit-Position: refs/heads/master@{#509729}
-
Marc Treib authored
SearchBox may execute JS in response to IPCs, but running JS in the given frame is illegal before the provisional navigation has committed. Bug: 765101 Change-Id: I7fc01989177b768667b5d64c2198b2da3ec0adb9 Reviewed-on: https://chromium-review.googlesource.com/723079Reviewed-by:
Daniel Cheng <dcheng@chromium.org> Reviewed-by:
Alex Moshchuk <alexmos@chromium.org> Commit-Queue: Marc Treib <treib@chromium.org> Cr-Commit-Position: refs/heads/master@{#509728}
-
Takashi Toyoshima authored
Cache-aware loading is an internal optimization and implementation details that should not be exposed to the fetch event. That makes a hidden request in order to check cache entry existence for legacy sites, but it is useless for modern sites that enables service worker to optimize such resource loading by hands. This patch just stops using the cache-aware loading on service worker enabled pages. It solves the problem that expose the hidden request to the fetch event, and does not cause real world performance impact. Bug: 750612 Change-Id: I8c698ae259274482d35cdbb5abde84c946652cfd Reviewed-on: https://chromium-review.googlesource.com/725026Reviewed-by:
Yutaka Hirano <yhirano@chromium.org> Commit-Queue: Takashi Toyoshima <toyoshim@chromium.org> Cr-Commit-Position: refs/heads/master@{#509727}
-
Erik Luo authored
This adds counters to each sidebar item, revealing the number of messages within. Screenshot: https://imgur.com/a/o4pSF Bug: 685283 Change-Id: Iee551c44506d9949b6a1a143ec6650cb12c9f728 Reviewed-on: https://chromium-review.googlesource.com/719958 Commit-Queue: Erik Luo <luoe@chromium.org> Reviewed-by:
Dmitry Gozman <dgozman@chromium.org> Cr-Commit-Position: refs/heads/master@{#509726}
-
Koji Ishii authored
Following bot results are included. 1673 1674 1676 1677 1678 1680 1681 1685 1687 1688 1691 1696 1697 1699 1700 1701 1702 1709 1712 1720 1722 1723 1725 1731 30 lines were removed by consecutive passes since 1685. TBR=eae@chromium.org NOTRY=true Bug: 591099 Change-Id: Ib64f101f2ef3ce6c10d56fab65fe6b352f76fedc Reviewed-on: https://chromium-review.googlesource.com/720859 Commit-Queue: Koji Ishii <kojii@chromium.org> Reviewed-by:
Koji Ishii <kojii@chromium.org> Cr-Commit-Position: refs/heads/master@{#509725}
-
Stanislav Albreht authored
It seems that offline_pages component participates in desktop builds whereas it is currently Android only. This puts such dependencies under an appropriate GN variable. Change-Id: Ic6b401ba60e40648eafeaf382068342a5d37d311 Reviewed-on: https://chromium-review.googlesource.com/721701Reviewed-by:
Scott Violet <sky@chromium.org> Reviewed-by:
Dmitry Titov <dimich@chromium.org> Commit-Queue: Stanislav Albreht <fatalerr@yandex-team.ru> Cr-Commit-Position: refs/heads/master@{#509724}
-
Renee Wright authored
(where possible) Remove commas from ends of lists Remove stray newlines Clarify logical_side comment I have checked to gen diff to verify no change Bug: Change-Id: Iaade785508cec25741dca08e54ea2294a5293eb1 Reviewed-on: https://chromium-review.googlesource.com/717956 Commit-Queue: Renée Wright <rjwright@chromium.org> Reviewed-by:
meade_UTC10 <meade@chromium.org> Cr-Commit-Position: refs/heads/master@{#509723}
-
Alex Clarke authored
Bug: 775513 Change-Id: I5917eeebcf1e86f0e6dc2db588ca18e34452d0c0 Reviewed-on: https://chromium-review.googlesource.com/723421Reviewed-by:
Eric Seckler <eseckler@chromium.org> Commit-Queue: Eric Seckler <eseckler@chromium.org> Commit-Queue: Alex Clarke <alexclarke@chromium.org> Cr-Commit-Position: refs/heads/master@{#509722}
-
Roman Sorokin authored
Fixes Chrome crash on Chromad devices if DeviceUserWhitelist policy is set Bug: 775834 Test: manual Change-Id: I3264363336d0186d7401ef8733457c1cdbcd1829 Reviewed-on: https://chromium-review.googlesource.com/725421Reviewed-by:
Achuith Bhandarkar <achuith@chromium.org> Commit-Queue: Roman Sorokin <rsorokin@chromium.org> Cr-Commit-Position: refs/heads/master@{#509721}
-
Martin Šrámek authored
This reverts commit 0cece6a4. Reason for revert: The test indeed flakes again. Original change's description: > Re-enable TabCaptureApiTest.ApiTests on Mac. > > I have run the test locally multiple times with iteration count of 100 > and 200 without any failures. Re-enabling to see if it will flake on > the bots or whatever the root cause was is possibly fixed. > > Bug: 764464 > Change-Id: Iedd41ec3b7fa1172766ac32196f3c8dbc2994f9d > Reviewed-on: https://chromium-review.googlesource.com/716797 > Reviewed-by: Yuri Wiitala <miu@chromium.org> > Reviewed-by: Alex Moshchuk <alexmos@chromium.org> > Commit-Queue: Nasko Oskov <nasko@chromium.org> > Cr-Commit-Position: refs/heads/master@{#508486} TBR=nasko@chromium.org,miu@chromium.org,alexmos@chromium.org # Not skipping CQ checks because original CL landed > 1 day ago. Bug: 764464 Change-Id: I770164ccc298fdb33cf082fc3aa6aef307c10f49 Reviewed-on: https://chromium-review.googlesource.com/725399Reviewed-by:
Martin Šrámek <msramek@chromium.org> Commit-Queue: Martin Šrámek <msramek@chromium.org> Cr-Commit-Position: refs/heads/master@{#509720}
-
Daniel Cheng authored
This reverts commit 585cd042. Reason for revert: it's relatively common to load > 2MB data URLs as subresources =( Bug: 774039 Original change's description: > Treat overly long URLs as invalid during canonicalization. > > Bug: 571784 > Cq-Include-Trybots: master.tryserver.chromium.linux:linux_site_isolation > Change-Id: Ied18e9d76e13028b7077a4ed3631f0a64ce8bdf6 > Reviewed-on: https://chromium-review.googlesource.com/678058 > Commit-Queue: Daniel Cheng <dcheng@chromium.org> > Reviewed-by: Asanka Herath <asanka@chromium.org> > Reviewed-by: Charlie Reis <creis@chromium.org> > Reviewed-by: Mike West <mkwst@chromium.org> > Cr-Commit-Position: refs/heads/master@{#506553} TBR=dcheng@chromium.org,creis@chromium.org,asanka@chromium.org,mkwst@chromium.org # Not skipping CQ checks because original CL landed > 1 day ago. Bug: 571784 Change-Id: I011fd8a9bcf5534ec23bcee10cc1de0add21d150 Cq-Include-Trybots: master.tryserver.chromium.linux:linux_site_isolation Reviewed-on: https://chromium-review.googlesource.com/725023Reviewed-by:
-
Fredrik Söderquist authored
Said function would ignore any size keyword or explicit size specification if there was a shape keyword. Adjust the comparisons so that first any explicit size checked, followed by checking the size keyword and the shape value. Also rewrite the if-ladder for the center point (|first_x_| and |first_y_|) to just use two DataEquivalent. Bug: 775201 Change-Id: I44686b54d3e02e1fd4682c4288eec3aca0e3e441 Reviewed-on: https://chromium-review.googlesource.com/723460Reviewed-by:
-
Dariusz Marcinkiewicz authored
BUG=b:67364190 Change-Id: I4ed4699343fa9604ec4d47a033dc30f3427b628e Reviewed-on: https://chromium-review.googlesource.com/718749 Commit-Queue: Dariusz Marcinkiewicz <darekm@chromium.org> Reviewed-by:
Ahmed Fakhry <afakhry@chromium.org> Reviewed-by:
Toni Barzic <tbarzic@chromium.org> Cr-Commit-Position: refs/heads/master@{#509717}
-
Emily Stark authored
- What percentage of connections with EV certificates are CT-compliant? We used to have this histogram but removed it when the EV whitelist was removed. I think it's still useful to have around, as it helps us gauge how often connections that are supposed to be CT-compliant are in fact CT-compliant. For example, a high rate of non-compliant EV certs might indicate that a CA is logging improperly. - What percentage of connections where CT is required are actually CT-compliant? As above, this helps us judge whether servers that are supposed to send CT actually are. The existing count that we have of the ERR_CERTIFICATE_TRANSPARENCY_REQUIRED error code is not enough to tell us this, because it doesn't tell us how many connections were actually supposed to have CT, only how many were supposed to but didn't. The latter histogram requires a modification to the return value of TransportSecurityState::CheckCTRequirements; instead of returning early when a connection is compliant and telling the caller that CT requirements were met, it now differentiates "CT requirements met" from "CT wasn't required". (An alternative approach is to record the histogram inside CheckCTRequirements itself and not modify its return value, but I thought it was preferable to minimize side effects inside that method [even though it does already have other side effects, unfortunately].) Bug: 772534 Change-Id: Ifd3d6f72475e07e1470b0d0b171f9ce0a325807e Reviewed-on: https://chromium-review.googlesource.com/707994 Commit-Queue: Emily Stark <estark@chromium.org> Reviewed-by:
Steven Holte <holte@chromium.org> Reviewed-by:
Matt Mueller <mattm@chromium.org> Cr-Commit-Position: refs/heads/master@{#509716}
-
arthursonzogni authored
This CL adds a test for issue 769645 that was fixed by: https://chromium-review.googlesource.com/c/709057 It checks that when a browser-initiated same-document navigation happens while the current document is loading, the load isn't interrupted. Bug: 769645 Cq-Include-Trybots: master.tryserver.chromium.linux:linux_site_isolation Change-Id: I2e2ae42d970dacc7217143488b5506759d61f22f Reviewed-on: https://chromium-review.googlesource.com/716756 Commit-Queue: Arthur Sonzogni <arthursonzogni@chromium.org> Reviewed-by:
Nasko Oskov <nasko@chromium.org> Cr-Commit-Position: refs/heads/master@{#509715}
-
Yutaka Hirano authored
RequestInit is manually implemented and has some compatibility issues. 1. undefined value is treated as a real value, e.g., (new Request('/', {method: undefined}).method returns "undefined". 2. Exceptions are silently ignored, e.g., new Request('/', {get method() { throw Error(); }}) doesn't throw. This change fixes the issues. Bug: 775318 Change-Id: I3d5b2b49f47cfdf92ba0707f6af0260653e386d9 Reviewed-on: https://chromium-review.googlesource.com/722543 Commit-Queue: Yutaka Hirano <yhirano@chromium.org> Reviewed-by:Adam Rice <ricea@chromium.org> Reviewed-by:
Kouhei Ueno <kouhei@chromium.org> Cr-Commit-Position: refs/heads/master@{#509714}
-
Sheng-Hao Tsao authored
JPEG encode accelerator interface will be implemented with VAAPI and v4l2 solutions for different platforms. JPEG encode acceleration can be used to reduce camera still capture encoding latency. BUG=b:67488323 TEST=It compiles. Change-Id: I3b135f0ad07971e0ec68684015816f5da02cbb28 Reviewed-on: https://chromium-review.googlesource.com/704619 Commit-Queue: Sheng-hao Tsao <shenghao@google.com> Reviewed-by:
Ricky Liang <jcliang@chromium.org> Reviewed-by:
Dale Curtis <dalecurtis@chromium.org> Cr-Commit-Position: refs/heads/master@{#509713}
-
Makoto Shimazu authored
As a step of removing SWReadFromCacheJob, this patch adds a path to get the same installed scripts multiple times. WebServiceWorkerInstalledScriptsManager asks the browser if the script has already been taken, and the browser side counterpart (SWInstalledScriptsSender) sends the script after streaming all of scripts. This is not optimal for sites which importing the same script multiple times, but it simplifies the code. Bug: 756312 Change-Id: I8d8c03fe8488367b5aa1b8d2c02e13a5f25e676e Reviewed-on: https://chromium-review.googlesource.com/697165 Commit-Queue: Makoto Shimazu <shimazu@chromium.org> Reviewed-by:
Matt Falkenhagen <falken@chromium.org> Reviewed-by:
-
Darren Shen authored
Spec: https://drafts.css-houdini.org/css-typed-om-1/#keywordvalue-objects There are no failing tests. Bug: 774887 Change-Id: If78a3e490c406e8743eb591f1e9bdcb28f9fb3a5 Reviewed-on: https://chromium-review.googlesource.com/720582 Commit-Queue: Darren Shen <shend@chromium.org> Reviewed-by:
Renée Wright <rjwright@chromium.org> Cr-Commit-Position: refs/heads/master@{#509711}
-
Rakina Zata Amni authored
In some cases, relatedTarget kept its value after event dispatch has finished even though target is set to null. See test file for the cases. This change adds a check so that when target is set to null in the post-processing phase, the change is also reflected in relatedTarget. Bug: 771580 Change-Id: Idda5d00c1a793b2669be66e76439851d1f66459c Reviewed-on: https://chromium-review.googlesource.com/720897Reviewed-by:
Hayato Ito <hayato@chromium.org> Reviewed-by:
Takayoshi Kochi <kochi@chromium.org> Commit-Queue: Rakina Zata Amni <rakina@chromium.org> Cr-Commit-Position: refs/heads/master@{#509710}
-
Kinuko Yasuda authored
When S13nServiceWorker is enabled: Before this change, subresource requests initiated from client documents are: - intercepted by SWSubresourceLoader in the renderer process, - forwarded to BrowserSideControllerServiceWorker in the browser process, and - dispatched to ServiceWorkerContextClient in the SW's renderer process. After this change, subresource requests are: - intercepted by SWSubresourceLoader in the renderer process, - forwarded to ControllerServiceWorkerImpl in the SW's renderer process, and - dispatched to ServiceWorkerContextClient in the SW's renderer process. Same layout tests and browsertests should keep passing with this change on Linux Mojo bots. Bug: 715640 Change-Id: I1c2be11897df162f9c375049e9b3eb618b81d3e7 Reviewed-on: https://chromium-review.googlesource.com/713114 Commit-Queue: Kinuko Yasuda <kinuko@chromium.org> Reviewed-by:
Tom Sepez <tsepez@chromium.org> Reviewed-by:
Tsuyoshi Horo <horo@chromium.org> Cr-Commit-Position: refs/heads/master@{#509709}
-
Bhagirathi Satpathy authored
This CL removes unused memory inclusion from Document. Bug: Change-Id: I14b5f687b901270301d456b924373355692bf07f Reviewed-on: https://chromium-review.googlesource.com/723239Reviewed-by:
Kentaro Hara <haraken@chromium.org> Commit-Queue: srirama chandra sekhar <srirama.m@samsung.com> Cr-Commit-Position: refs/heads/master@{#509708}
-
Mikhail Pozdnyakov authored
As it follows from https://github.com/w3c/sensors/issues/313 Change-Id: Ic81e118b2a3c8db6d8b30ee61ef8811b88a0f970 Reviewed-on: https://chromium-review.googlesource.com/721742Reviewed-by:
Reilly Grant <reillyg@chromium.org> Reviewed-by:
-
Dariusz Marcinkiewicz authored
BUG=b:67364190 Change-Id: I7cd3816955683342276b585132de5fc37ed936e5 Reviewed-on: https://chromium-review.googlesource.com/718748Reviewed-by:
-
skia-deps-roller@chromium.org authored
https://skia.googlesource.com/skia.git/+log/5546c921e0ca..1f7c31e27e26 $ git log 5546c921e..1f7c31e27 --date=short --no-merges --format='%ad %ae %s' 2017-10-18 angle-deps-roller Roll skia/third_party/externals/angle2/ c514348e2..0a6752e3c (1 commit) 2017-10-18 angle-deps-roller Roll skia/third_party/externals/angle2/ ebf7299e4..c514348e2 (1 commit) Created with: roll-dep src/third_party/skia Documentation for the AutoRoller is here: https://skia.googlesource.com/buildbot/+/master/autoroll/README.md If the roll is causing failures, see: http://www.chromium.org/developers/tree-sheriffs/sheriff-details-chromium#TOC-Failures-due-to-DEPS-rolls 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=benjaminwagner@chromium.org Change-Id: I10e32e98271f2ebd3fc482aaeaf8c4c5fe3a6ae6 Reviewed-on: https://chromium-review.googlesource.com/725219Reviewed-by:
Skia Deps Roller <skia-deps-roller@chromium.org> Commit-Queue: Skia Deps Roller <skia-deps-roller@chromium.org> Cr-Commit-Position: refs/heads/master@{#509705}
-
Raymes Khoury authored
This also changes the log level for these type of messages to WARNING since that seems more appropriate given their nature. Bug: 689802 Change-Id: I1ca15b71caa35ca941572d0fbca864855696bb6e Reviewed-on: https://chromium-review.googlesource.com/722659Reviewed-by:
Timothy Loh <timloh@chromium.org> Commit-Queue: Raymes Khoury <raymes@chromium.org> Cr-Commit-Position: refs/heads/master@{#509704}
-
Aleks Totic authored
This CL implements flag feature from results.html. I've used this feature to rework "Rebaseline" feature. Now it is just another format. Also, fixed a bug where hitting enter in text search would display results twice. With this, I think we've reached feature parity with results.html. Once it is in, I'll make another pitch to replace results.html with test-expectations. Bug: 726520 Change-Id: I811fead7940c7c1e44326ea03c3394ccf06a44bc Reviewed-on: https://chromium-review.googlesource.com/720172 Commit-Queue: Aleks Totic <atotic@chromium.org> Reviewed-by:
Quinten Yearsley <qyearsley@chromium.org> Cr-Commit-Position: refs/heads/master@{#509703}
-
Vlad Tsyrklevich authored
This reverts commits 3d8ae2fd and e3168df0. Reason for revert: e3168df0 broke the Linux CFI bot due to a misplaced blacklist entry and 3d8ae2fd was a follow-up change. Change-Id: I13f8db49d0621cb8e588ae26d261e362faa462a9 Reviewed-on: https://chromium-review.googlesource.com/724921Reviewed-by:
Peter Collingbourne <pcc@chromium.org> Commit-Queue: Peter Collingbourne <pcc@chromium.org> Cr-Commit-Position: refs/heads/master@{#509702}
-
Hajime Hoshi authored
This CL is a preparation for the change that WebTaskRunnerImpl will always be created for each call of WebFrameSchedulerImpl:: *TaskRunner(). This CL replaces WebTaskRunner* with RefPtr<WebTaskRunner>. This is part of the big CL for changing WebFrameSchedulerImpl:: *TaskRunner() to create WebTaskRunnerImpl anytime: https://chromium-review.googlesource.com/c/chromium/src/+/704459 Bug: 762453 Change-Id: Iad2b4ae62bbc73d5c20a90c79c2a1fc947bbd3cc Reviewed-on: https://chromium-review.googlesource.com/722861 Commit-Queue: Hajime Hoshi <hajimehoshi@chromium.org> Reviewed-by:
Alexander Timin <altimin@chromium.org> Reviewed-by:
Takeshi Yoshino <tyoshino@chromium.org> Cr-Commit-Position: refs/heads/master@{#509701}
-
Henrik Grunell authored
Bug: None Change-Id: Iaa2da4e58fe8605cbccbd8d204eafe0689de2387 Reviewed-on: https://chromium-review.googlesource.com/721039Reviewed-by:
Olga Sharonova <olka@chromium.org> Reviewed-by:
Alexei Svitkine <asvitkine@chromium.org> Commit-Queue: Henrik Grunell <grunell@chromium.org> Cr-Commit-Position: refs/heads/master@{#509700}
-
Giovanni Ortuño Urquidi authored
This way we are sure we are not unintentionally changing the behavior for hosted apps. Also changes tests to use a LINK page transition which more closely matches what happens when a user clicks a link. Passing NavigateParams as an argument to NavigateToURL causes it to not block so this patch also adds code to wait for the url to load. Change-Id: I20c57ba8727535a8cc7768fc1fb80847f9d0977a Reviewed-on: https://chromium-review.googlesource.com/720724 Commit-Queue: Giovanni Ortuño Urquidi <ortuno@chromium.org> Reviewed-by:
Ben Wells <benwells@chromium.org> Cr-Commit-Position: refs/heads/master@{#509699}
-
Eddy Mead authored
This allows us to remove some duplicate code, and prepares the way for generating ComputedStyle's Initial* functions elsewhere. This ended up reducing the number of includes in each APO subclass header. Gist diff: https://gist.github.com/wilddamon/5bfa127e71e9c7993104bbfa602b3f97/revisions Bug: 763306 Change-Id: Icaacee6abe218b73a460311eb1ffcbff5c81d4c5 Reviewed-on: https://chromium-review.googlesource.com/716442 Commit-Queue: meade_UTC10 <meade@chromium.org> Reviewed-by:
Darren Shen <shend@chromium.org> Cr-Commit-Position: refs/heads/master@{#509698}
-
Hidehiko Abe authored
Before this CL, the state machine may enter STARTING_INSTANCE twice (for a mini instance starting, and upgrade), and as a result and because of that OnInstanceStarted() needed an extra parameter. With this CL, the state machine enters each state up to once, which simplifies the callback functions. BUG=None TEST=Ran trybot. Change-Id: Ie92abbfd92fbae53c47bba1ed34fd3c71a60f46f Reviewed-on: https://chromium-review.googlesource.com/723679 Commit-Queue: Hidehiko Abe <hidehiko@chromium.org> Reviewed-by:
Yusuke Sato <yusukes@chromium.org> Cr-Commit-Position: refs/heads/master@{#509697}
-
Chris Watkins authored
The docs previously claimed that you don't need to add a Trace() method to an on-heap class if it does not contain on-heap members. However, this is no longer true. Change-Id: I4a8faf9805f940735af4fb722265fe0d7bf0edda Reviewed-on: https://chromium-review.googlesource.com/725100Reviewed-by:
-
Chromium WPT Sync authored
Using wpt-import in Chromium 1e66d75f. Build: https://build.chromium.org/p/chromium.infra.cron/builders/wpt-importer/builds/3387 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 Directory owners for changes in this CL: jsbell@chromium.org: external/wpt/IndexedDB TBR=qyearsley@chromium.org No-Export: true Change-Id: I0774a1cef1818f3b0bdfe54999640f418ada537b Reviewed-on: https://chromium-review.googlesource.com/724899 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@{#509695}
-
skia-deps-roller@chromium.org authored
https://skia.googlesource.com/skia.git/+log/d71c6d1994f6..5546c921e0ca $ git log d71c6d199..5546c921e --date=short --no-merges --format='%ad %ae %s' 2017-10-18 angle-deps-roller Roll skia/third_party/externals/angle2/ b9cb7f60d..ebf7299e4 (1 commit) 2017-10-17 angle-deps-roller Roll skia/third_party/externals/angle2/ 8a9e0fc23..b9cb7f60d (1 commit) 2017-10-17 benjaminwagner Synchronize public.bzl float-cast-overflow omit list with test.py. Created with: roll-dep src/third_party/skia BUG=774309 Documentation for the AutoRoller is here: https://skia.googlesource.com/buildbot/+/master/autoroll/README.md If the roll is causing failures, see: http://www.chromium.org/developers/tree-sheriffs/sheriff-details-chromium#TOC-Failures-due-to-DEPS-rolls 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=benjaminwagner@chromium.org Change-Id: I4f4ad7cf3865ab75b485e46ce77942a42f261637 Reviewed-on: https://chromium-review.googlesource.com/724960Reviewed-by:
-
Jay Civelli authored
When using zip::Zip() with an IPC based FileAccessor, zipping directories with large number of files triggers many IPC calls making the entire operation significantly slower than with direct file access. In order to alleviate this performance hit, this patch groups file reads by modifying the FileAccessor read method so it reads multiple files at once. zip::Zip() can then group these reads when writing the ZIP file. The writing code has been factored out into a new ZipWriter class to make that code more readable. Bug: 773310 Change-Id: I8121980bf05d87a174c63164840ec6bf325c7e52 Reviewed-on: https://chromium-review.googlesource.com/719356 Commit-Queue: Jay Civelli <jcivelli@chromium.org> Reviewed-by:
Ilya Sherman <isherman@chromium.org> Cr-Commit-Position: refs/heads/master@{#509693}
-
Renee Wright authored
- Slightly re-arranged the order of the members in the "parameters" object. - Re-arranges members in the data objects to match the order in which they appear in "parameters" - Deleted a useless comments (-1 LOC) - Added a comment advising we stick to this ordering (+2 LOC) I've checked the gen diff to verify that there is no change in output. It would be nice to add a script to enforce this ordering if it's not too hard. Bug: Change-Id: I9f8b1248c6fd48f0f0e77769558b2770848b8b65 Reviewed-on: https://chromium-review.googlesource.com/714799 Commit-Queue: Renée Wright <rjwright@chromium.org> Reviewed-by:
Bugs Nash <bugsnash@chromium.org> Reviewed-by:
-
Alice Boxhall authored
Bug: 390254 Change-Id: I3fc5b0ac80fe861c04b0bbe72ec93438bbf25145 Reviewed-on: https://chromium-review.googlesource.com/698844 Commit-Queue: Alice Boxhall <aboxhall@chromium.org> Reviewed-by:
Pavel Feldman <pfeldman@chromium.org> Cr-Commit-Position: refs/heads/master@{#509691}
-
