- 12 Feb, 2018 23 commits
-
-
Noel Gordon authored
Run git cl format over BUILD.gn, and add a line space here and there to increase readability. Tbr: cavalcanti@chromium.org Bug: None Change-Id: Id46ff314c0fe820880fd441858cbaca6d7754b46 Reviewed-on: https://chromium-review.googlesource.com/912209Reviewed-by:
Noel Gordon <noel@chromium.org> Commit-Queue: Noel Gordon <noel@chromium.org> Cr-Commit-Position: refs/heads/master@{#536036}
-
https://skia.googlesource.com/skia.git/+log/3a2b8031331c..a76f6161f1c2 $ git log 3a2b80313..a76f6161f --date=short --no-merges --format='%ad %ae %s' 2018-02-12 update-docs Update markdown files Created with: roll-dep src/third_party/skia The AutoRoll server is located here: https://autoroll.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.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 TBR=caryclark@chromium.org Change-Id: Id16619608021c9df2134631ebfb89a6b06091290 Reviewed-on: https://chromium-review.googlesource.com/912662 Commit-Queue: skia-chromium-autoroll <skia-chromium-autoroll@skia-buildbots.google.com.iam.gserviceaccount.com> Reviewed-by:
skia-chromium-autoroll <skia-chromium-autoroll@skia-buildbots.google.com.iam.gserviceaccount.com> Cr-Commit-Position: refs/heads/master@{#536035}
-
Ramin Halavati authored
Update annotations.xml policy fields to match field numbers in traffic_annotation.proto. Bug:656607 Change-Id: Iba28b3090b9a65f06f505551812a77429801b43c TBR=msramek@chromium.org Change-Id: Iba28b3090b9a65f06f505551812a77429801b43c Reviewed-on: https://chromium-review.googlesource.com/911549Reviewed-by:
Ramin Halavati <rhalavati@chromium.org> Commit-Queue: Ramin Halavati <rhalavati@chromium.org> Cr-Commit-Position: refs/heads/master@{#536034}
-
Han Leon authored
This CL mojofies the legacy IPC ServiceWorkerHostMsg_TerminateWorker. BUG=772713 Change-Id: If91640688b0c7e7d08b7d375a7776d7c19c17781 Reviewed-on: https://chromium-review.googlesource.com/897582 Commit-Queue: Han Leon <leon.han@intel.com> Reviewed-by:
Matt Falkenhagen <falken@chromium.org> Reviewed-by:
Kinuko Yasuda <kinuko@chromium.org> Reviewed-by:
Makoto Shimazu <shimazu@chromium.org> Cr-Commit-Position: refs/heads/master@{#536033}
-
Filip Gorski authored
Bug: 805177 Change-Id: I81bc5c2b54304f60123f464b155c3a1ee8182c72 Reviewed-on: https://chromium-review.googlesource.com/910148 Commit-Queue: Filip Gorski <fgorski@chromium.org> Reviewed-by:
Marc Treib <treib@chromium.org> Cr-Commit-Position: refs/heads/master@{#536032}
-
Meredith Lane authored
After multiple problems with running async_tests and them timing out, we have decided to move all tests to just synchronous test runners. BUG=792783 Change-Id: I300c94795ea0c9a657561916eccc86bbed0be0a9 Reviewed-on: https://chromium-review.googlesource.com/913051Reviewed-by:
Alice Boxhall <aboxhall@chromium.org> Commit-Queue: Meredith Lane <meredithl@google.com> Cr-Commit-Position: refs/heads/master@{#536031}
-
chrome-release-bot authored
# This is an automated release commit. # Do not revert without consulting chrome-pmo@google.com. NOAUTOREVERT=true TBR=govind@chromium.org Change-Id: I37ac0069eba0216f798a8d05d2adcbf4947bcdfe Reviewed-on: https://chromium-review.googlesource.com/912034Reviewed-by:
chrome-release-bot@chromium.org <chrome-release-bot@chromium.org> Cr-Commit-Position: refs/heads/master@{#536030}
-
Han Leon authored
In the implementations of ServiceWorkerRegistrationObjectHost.{SetNavigationPreloadHeader,EnableNavigationPreload}, if ServiceWorkerStorage::{UpdateNavigationPreloadHeader,UpdateNavigationPreloadEnabled} failed, a ServiceWorkerErrorType::kUnknown error code is sent back to Blink side, over there it's mapped as an ExceptionCode::kUnknownError to construct a DOMException to be exported to JavaScript context. This is an acceptable behavior, as all of other core DOMException codes are not appropriate to indicate such an storage access failure, and we're already putting an error message "Failed to access storage." into the JavaScript DOMException mentioned above. Based on above explanations, this CL removes an unnecessary DCHECK against the kUnKnownError in Blink side. BUG=807239 Change-Id: Ic2222e264139abc000f5a8894bb0b33932b5335d Reviewed-on: https://chromium-review.googlesource.com/908059Reviewed-by: -
Darren Shen authored
Currently, clicking on a dropdown when the VK is active will cause the VK to hide, which sends out a resize message to RenderViewImpl, which then forces the newly opened dropdown to hide again. The resize message sent by the VK doesn't actually change the size, so we can prevent the popup from hiding if the new requested size is the same as the old. The hope is that this doesn't affect any other existing behaviour. Bug: 761908 Change-Id: I0d2d7e876b42cced2be867ebc5ad3432a4d045c7 Reviewed-on: https://chromium-review.googlesource.com/909869Reviewed-by:
Yuichiro Hanada <yhanada@chromium.org> Reviewed-by:
Kentaro Hara <haraken@chromium.org> Commit-Queue: Darren Shen <shend@chromium.org> Cr-Commit-Position: refs/heads/master@{#536028}
-
Sasha Morrissey authored
Break down 'other' in FileBrowser.QuickView.FileType UMA to 'directory', 'no extension' and 'unknown extension'. extension and a file with a .gn extension. Then check values 59 through 62 inclusive Bug: 772599 Test: Quick View and Open on a folder, a .zip file, a file without an chrome: //histograms has 4 entries for FileBrowser.QuickView.FileType, Cq-Include-Trybots: master.tryserver.chromium.linux:closure_compilation Change-Id: Id13b8734e96b238bfb98b98b0062d1f0721180c7 Reviewed-on: https://chromium-review.googlesource.com/907992Reviewed-by:
Naoki Fukino <fukino@chromium.org> Commit-Queue: Sasha Morrissey <sashab@chromium.org> Cr-Commit-Position: refs/heads/master@{#536027}
-
https://skia.googlesource.com/skia.git/+log/78c110e2f0fa..3a2b8031331c $ git log 78c110e2f..3a2b80313 --date=short --no-merges --format='%ad %ae %s' 2018-02-12 liyuqian Remove SK_SUPPORT_LEGACY_AAA_SMOOTH Created with: roll-dep src/third_party/skia The AutoRoll server is located here: https://autoroll.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.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 TBR=caryclark@chromium.org Change-Id: I079db80d8c7b4eb3051102b22cc9828eea306e05 Reviewed-on: https://chromium-review.googlesource.com/911979Reviewed-by:
-
Zhuoyu Qian authored
This is part of the gradual process of putting all types that derive from CSSValue into blink::cssvalue:: namespace. Bug: 667961 Signed-off-by:
Zhuoyu Qian <zhuoyu.qian@samsung.com> Change-Id: I755e7f48ea7f7f558e9ed6cc386af8e87b20ec66 Reviewed-on: https://chromium-review.googlesource.com/910473Reviewed-by:
Stuart Langley <slangley@chromium.org> Reviewed-by:
nainar <nainar@chromium.org> Cr-Commit-Position: refs/heads/master@{#536025}
-
Nico Weber authored
Found by PVS-Studio. Bug: 805881 Change-Id: I96cd4b37494514ae784a2583d31738e8d3ef616a Reviewed-on: https://chromium-review.googlesource.com/912252 Commit-Queue: Nico Weber <thakis@chromium.org> Reviewed-by:
Avi Drissman <avi@chromium.org> Cr-Commit-Position: refs/heads/master@{#536024}
-
Darren Shen authored
There are some layout tests in typedcssom that are now covered by other tests (either WPT or IDLHarness). We can remove them. Bug: 774887 Change-Id: I39277868e2a656d66795753c984755ac8a5d330d Reviewed-on: https://chromium-review.googlesource.com/912850Reviewed-by:
-
Darren Shen authored
This patch implements CSSTransformComponent.toMatrix(), which was previously implemented as AsMatrix() but just not webexposed. We also split up the tests since it makes more sense. Bug: 807877 Change-Id: Icee742fdde033d63cea85a27250e91a71ad67e2a Reviewed-on: https://chromium-review.googlesource.com/910529 Commit-Queue: Darren Shen <shend@chromium.org> Reviewed-by:
-
Darren Shen authored
Currently, our per-property tests only test that styleMap.set accept valid values. This patch refactors the per-property tests a bit to also test that we reject invalid values. Bug: 774887 Change-Id: I41b5c4c1a6dfe6e766ff37fdc1b1f890dfb81aac Reviewed-on: https://chromium-review.googlesource.com/905523 Commit-Queue: Darren Shen <shend@chromium.org> Reviewed-by:
-
Noel Gordon authored
This reverts commit b07c92a1. Reason for revert: the litmus test of running this code on linux x86 bots worked. However, performance-wise we're not so sure. Back this change out to restrict it to Intel x64 for now. Original change's description: > Add x86 bots to the read64le party > > thestig@ reminds me we do have linux x86 bots on the waterfall. Add them > to the read64le party. Minor: run git cl format over BUILD.gn, and add a > line space here and there for readability. > > Bug: 760853 > Change-Id: Ia47296a26bff77f9be699e31053d8b94aac583f4 > Reviewed-on: https://chromium-review.googlesource.com/910328 > Reviewed-by: Mike Klein <mtklein@chromium.org> > Commit-Queue: Noel Gordon <noel@chromium.org> > Cr-Commit-Position: refs/heads/master@{#535612} TBR=noel@chromium.org,mtklein@chromium.org,nigeltao@chromium.org # Not skipping CQ checks because original CL landed > 1 day ago. Bug: 760853 Change-Id: If3d24a2cdc6614ff82c781019623108a52dbd16a Reviewed-on: https://chromium-review.googlesource.com/913029 Commit-Queue: Noel Gordon <noel@chromium.org> Reviewed-by:
-
Yuqian Li authored
TBR: fmalita@chromium.org Bug: skia:7573 Change-Id: I7c7fb884003038a016db36bafe6031c89757d84a Reviewed-on: https://chromium-review.googlesource.com/912909Reviewed-by:
Florin Malita <fmalita@chromium.org> Commit-Queue: Yuqian Li <liyuqian@chromium.org> Cr-Commit-Position: refs/heads/master@{#536019}
-
Chromium WPT Sync authored
Using wpt-import in Chromium 121b6cfa. Build: https://ci.chromium.org/buildbot/chromium.infra.cron/wpt-importer/12452 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=leon.han@intel.com No-Export: true Change-Id: I17d74a4ba963d0150744fd14a30472fe7c9941a0 Reviewed-on: https://chromium-review.googlesource.com/912251 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@{#536018}
-
Alan Cutter authored
frames() is no longer being shipped in M60 [0], as a result we cannot refer to it as a replacement for step-middle. [0] https://groups.google.com/a/chromium.org/forum/#!msg/blink-dev/j7l8_DjMsVE/0HKZgZXNAQAJ Bug: 646265 Change-Id: I8f828fabd7c6c8ccc5e72d393ef260de31fcb23c Reviewed-on: https://chromium-review.googlesource.com/557481 Commit-Queue: Alan Cutter <alancutter@chromium.org> Reviewed-by:
Suzy Howlett <suzyh@chromium.org> Cr-Commit-Position: refs/heads/master@{#536017}
-
Jinsuk Kim authored
WebContentsUserData-backed objects requires its WebContents instance from CVC be non-null, which causes numerous null checking statments. This CL examines callsites to the methods that contains accesses to those objects, adds null-checking statements if missing, and removes it if redundant. Some observations the changes are based on: - private native method doesn't need null check; WebContents is certainly valid at that point. - CVC.destroy() itself or calls from it (before it nulls out WebContents) doesn't need null check since WebContent is valid up to that point. Added null check: ... since clusterfuzz results warn that these need it: - setTouchScrollInProgress - destroyPastePopup ... by moving it over from the 2 callsites: - updateTextSelectionUI (onAttachedToWindow, onDetachedFromWindow) Removed null check as callsites already have null checking/assert, or assure WebContents is not null: - hidePopupsAndClearSelection (WebContentObserver.resetPopupsAndInput, onFocusChanged, showSelectPopup) - hidePopupsAndPreserveSelection (destroy, onHide, updateTextSelectionUI, onFocusChanged, onRotationChanged) - hidePopups (above 2) - onAttachedToWindow, onDetachedFromWindow: moved to updateTextSelectionUI - onRotationChanged: consolidated null checks Inlined one-liner restoreSelectionPopupsIfNecessary into callsites to eliminate the concern on null check on it (much clearer when inlined). Bug: 810179 Change-Id: I55aab2b1f7917a9af4522a1bcf8d3e7d9b172f2a Reviewed-on: https://chromium-review.googlesource.com/907770 Commit-Queue: Jinsuk Kim <jinsukkim@chromium.org> Reviewed-by:Bo <boliu@chromium.org> Cr-Commit-Position: refs/heads/master@{#536016}
-
Patti authored
Update the "clear all text" button's icon and show it while editing in touch mode. Screenshots - https://drive.google.com/file/d/1EFdKPFsvo70ZD0FfK0mfKLS-avijyCtN/view?usp=sharing Bug: 801583 Change-Id: I9b5dca22654d8f24c467e3f9c21d278a7c63005e Reviewed-on: https://chromium-review.googlesource.com/910828 Commit-Queue: Patti <patricialor@chromium.org> Reviewed-by:
Dominick Ng <dominickn@chromium.org> Reviewed-by:
Peter Kasting <pkasting@chromium.org> Cr-Commit-Position: refs/heads/master@{#536015}
-
Alan Cutter authored
Bookmark apps don't get special Chrome app permissions and it doesn't make sense to display the permissions allocated to them when it will always be none. This CL changes the "App info" context menu entry to go to "Site settings" for the bookmark app's URL. This is inline with our UX mocks for PWAs. Bug: 800205 Change-Id: I8e0135836c44bd7c88a22ea87b7bf1526dba14eb Reviewed-on: https://chromium-review.googlesource.com/910329 Commit-Queue: Alan Cutter <alancutter@chromium.org> Reviewed-by:
Trent Apted <tapted@chromium.org> Cr-Commit-Position: refs/heads/master@{#536014}
-
- 11 Feb, 2018 17 commits
-
-
Giovanni Ortuño Urquidi authored
When a website calls window.open() with a URL for which there is a PWA installed, we cancel the navigation in the newly opened tab and start a new navigation in a new app window. This patch plumbs the opener that would have existed in the new tab to the new app window, so that the new app window can access it. This only takes care of the child window. The parent window is not yet able to access the newly opened app window. Bug: 804881 Change-Id: Ie08c3425b13b4940435e6cf3614a015d61ed10a3 Reviewed-on: https://chromium-review.googlesource.com/906042Reviewed-by:
Matt Giuca <mgiuca@chromium.org> Reviewed-by:
Ben Wells <benwells@chromium.org> Commit-Queue: Giovanni Ortuño Urquidi <ortuno@chromium.org> Cr-Commit-Position: refs/heads/master@{#536013}
-
Noel Gordon authored
Patch files are no longer needed per the patch/README guidelines. Bug: 697280 Change-Id: I975a99ac0de50eb7c61a037c458853dc6406ff69 Reviewed-on: https://chromium-review.googlesource.com/912908Reviewed-by:
Adenilson Cavalcanti <cavalcantii@chromium.org> Commit-Queue: Noel Gordon <noel@chromium.org> Cr-Commit-Position: refs/heads/master@{#536012}
-
wutao authored
Mipmap generation is not guaranteed to be GPU accelerated on all devices that Chrome OS ships on, and without acceleration it is too slow to use trilinear filtering. In addition, some device drivers may not support correct mipmap generation. We can use this flag as a workaround until driver bugs have been fixed. This cl creates an ash switch so that we can turn off trilinear filtering. Bug: 799501, b/72298538 Test: Can create the flag and turn off the trilinear filtering. Change-Id: Ia711fce0bc037a391f4bb610e73af32301de7a89 Reviewed-on: https://chromium-review.googlesource.com/908251 Commit-Queue: Tao Wu <wutao@chromium.org> Reviewed-by:
Mitsuru Oshima <oshima@chromium.org> Reviewed-by:
David Reveman <reveman@chromium.org> Cr-Commit-Position: refs/heads/master@{#536011}
-
Eric Willigers authored
The CSS translate property no longer resolves percentages in getComputedStyle results. CSS WG Resolution: RESOLVED: % values of translate are serialized as percent for computed values. Add note making the behavior explicit. https://github.com/w3c/csswg-drafts/issues/2124 BUG=811027 Change-Id: I6f9b60c285d1eccc2e13ff0e806794df5cce5151 Reviewed-on: https://chromium-review.googlesource.com/912052 Commit-Queue: Eric Willigers <ericwilligers@chromium.org> Reviewed-by:
Fredrik Söderquist <fs@opera.com> Cr-Commit-Position: refs/heads/master@{#536010}
-
Mike Klein authored
Change-Id: I2693be780057c1ba715fce3ae3c398c536f147ea Reviewed-on: https://chromium-review.googlesource.com/911734Reviewed-by:
-
David 'Digit' Turner authored
This CL adds a small unit-test for the ZIP parsing code. To achieve this, the FileDescriptor class is also slightly improved: - It now always wraps a single integer value, instead of using void* when unit-testing. - The Read() method now returns an ssize_t value. - The SeekTo() method now returns an off_t value. - The ReadFull() method was added. - The Map() method now returns nullptr on failure, instead of MAP_FAILED, and the mock implementation was fixed. - More importantly, the GetFileSize() method was added to avoid calling stat() or fstat() directly in the zip parser, making it unit-testable!! + A new script is added to generate manually a set of data tables corresponding to various test zip archives embedded directly into the unit-test source code (see generate_zip_test_tables.sh) BUG=NONE R=pasko@chromium.org,agrieve@chromium.org,rmcilroy@chromium.org Change-Id: I095aa21ba26d924f7b95cf826d606875bd7d4591 Reviewed-on: https://chromium-review.googlesource.com/911408Reviewed-by:
agrieve <agrieve@chromium.org> Commit-Queue: David Turner <digit@chromium.org> Cr-Commit-Position: refs/heads/master@{#536008}
-
Rohit Rao authored
BUG=811075 Cq-Include-Trybots: master.tryserver.chromium.mac:ios-simulator-cronet;master.tryserver.chromium.mac:ios-simulator-full-configs Change-Id: I4c6ab0393b1051807d3e9886fd08556e6fdb67a3 Reviewed-on: https://chromium-review.googlesource.com/911736Reviewed-by:
Sergio Collazos <sczs@chromium.org> Commit-Queue: Rohit Rao <rohitrao@chromium.org> Cr-Commit-Position: refs/heads/master@{#536007}
-
https://skia.googlesource.com/skia.git/+log/c5cecdcaf8ab..78c110e2f0fa $ git log c5cecdcaf..78c110e2f --date=short --no-merges --format='%ad %ae %s' 2018-02-09 caryclark resolve bookmaker refs fixes Created with: roll-dep src/third_party/skia The AutoRoll server is located here: https://autoroll.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.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 TBR=caryclark@chromium.org Change-Id: If19828715f27fedc1bdd488fd65fe4218db29bb7 Reviewed-on: https://chromium-review.googlesource.com/913148 Commit-Queue: skia-chromium-autoroll <skia-chromium-autoroll@skia-buildbots.google.com.iam.gserviceaccount.com> Reviewed-by:
-
Kush Sinha authored
Enable Mirror account reconciliation on Chrome OS if the preference |kAccountConsistencyMirrorRequired| is set. This is a Chrome OS specific preference and was being used to stop sign in from secondary Google accounts and sign out from the primary account. Enabling Mirror here will have the effect of users' cookies getting re-minted if they clear their cookies from the Settings page. Bug: 805519 Test: out/ChromeOS/unit_tests --gtest_filter="*AccountConsistencyModeManagerTest*" Change-Id: Ie8fe9c4be9868d1301f6063dc84764a1c9dea028 Reviewed-on: https://chromium-review.googlesource.com/906430 Commit-Queue: Kush Sinha <sinhak@chromium.org> Reviewed-by:
David Roger <droger@chromium.org> Cr-Commit-Position: refs/heads/master@{#536005}
-
https://skia.googlesource.com/skia.git/+log/954295bc700d..c5cecdcaf8ab $ git log 954295bc7..c5cecdcaf --date=short --no-merges --format='%ad %ae %s' 2018-02-09 brianosman Remove image functions from GrGLInterface, part one Created with: roll-dep src/third_party/skia The AutoRoll server is located here: https://autoroll.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.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 TBR=caryclark@chromium.org Change-Id: Ie300196e258f5c344d6d8aa921a23d77e33751a7 Reviewed-on: https://chromium-review.googlesource.com/911978 Commit-Queue: skia-chromium-autoroll <skia-chromium-autoroll@skia-buildbots.google.com.iam.gserviceaccount.com> Reviewed-by:
-
Ken Rockot authored
TBR=dcheng@chromium.org TBR=jam@chromium.org Bug: 806965 Cq-Include-Trybots: master.tryserver.chromium.mac:ios-simulator-cronet;master.tryserver.chromium.mac:ios-simulator-full-configs Change-Id: Ifdb2041f20f777b06180bc981786d1c6777eb778 Reviewed-on: https://chromium-review.googlesource.com/911999 Commit-Queue: Ken Rockot <rockot@chromium.org> Reviewed-by:
Ken Rockot <rockot@chromium.org> Cr-Commit-Position: refs/heads/master@{#536003}
-
Ken Rockot authored
TBR=dcheng@chromium.org TBR=jam@chromium.org Bug: 806965 Cq-Include-Trybots: master.tryserver.chromium.mac:mac_optional_gpu_tests_rel;master.tryserver.chromium.win:win10_chromium_x64_rel_ng Change-Id: I301857ed92c18e80943ac4b6abab062c839b332f Reviewed-on: https://chromium-review.googlesource.com/911998 Commit-Queue: Ken Rockot <rockot@chromium.org> Reviewed-by:
-
yiyix authored
In this patch, I added some metrics to measure effectiveness of draw occlusion so that we can get more meaningful data from finch trials. Metrics added includes measuring the number of quads that are removed or resized, total area of drawing that are skipped, and total number draw quads that need to go through complex transforms. Bug:672929 Cq-Include-Trybots: master.tryserver.chromium.android:android_optional_gpu_tests_rel Change-Id: I354c95a3ac5525ab81694ad67f4f1b183c86f0cb Reviewed-on: https://chromium-review.googlesource.com/898342 Commit-Queue: Yi Xu <yiyix@chromium.org> Reviewed-by:
Robert Kaplow <rkaplow@chromium.org> Reviewed-by:
danakj <danakj@chromium.org> Cr-Commit-Position: refs/heads/master@{#536001}
-
https://skia.googlesource.com/skia.git/+log/193d3a051400..954295bc700d $ git log 193d3a051..954295bc7 --date=short --no-merges --format='%ad %ae %s' 2018-02-11 update-docs Update markdown files Created with: roll-dep src/third_party/skia The AutoRoll server is located here: https://autoroll.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.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 TBR=caryclark@chromium.org Change-Id: Ie262a2ca81f4047c12fbbf7168a7e3089c35234e Reviewed-on: https://chromium-review.googlesource.com/912779Reviewed-by:
-
Sylvain Defresne authored
Use browsing_data::RecordDeletionForPeriod() to record the action instead of re-implementing it and use the function browsing_data::CalculateEndDeleteTime to correctly compute the end of the time period (fix the "older than 30 days" time period). Remove unnecessary code to clean up delete_end_ as the corresponding issue is long fixed. Bug: none Cq-Include-Trybots: master.tryserver.chromium.mac:ios-simulator-cronet;master.tryserver.chromium.mac:ios-simulator-full-configs Change-Id: Ib1cf0287daad9f4228a1d2c146e0f8f2028e16cd Reviewed-on: https://chromium-review.googlesource.com/911885Reviewed-by:
Olivier Robin <olivierrobin@chromium.org> Commit-Queue: Sylvain Defresne <sdefresne@chromium.org> Cr-Commit-Position: refs/heads/master@{#535999}
-
Ken Rockot authored
and struct_traits => mojom_traits TBR=dcheng@chromium.org TBR=asvitkine@chromium.org TBR=jam@chromium.org Bug: 806965 Change-Id: Ic201838785b6613659bea1dab6a3f59b070e5e9a Reviewed-on: https://chromium-review.googlesource.com/912733 Commit-Queue: Ken Rockot <rockot@chromium.org> Reviewed-by:
-
Bence Béky authored
HttpBasicState::Initialize() always succeeds, so remove callback argument and change return type to void. Why convert CompletionCallback to CompletionOnceCallback if one can safely remove it instead? :) Bug: 807724 Change-Id: I16244d0600a278139ba2e112373f95933245a1b2 Reviewed-on: https://chromium-review.googlesource.com/908635Reviewed-by:
Eric Roman <eroman@chromium.org> Commit-Queue: Bence Béky <bnc@chromium.org> Cr-Commit-Position: refs/heads/master@{#535997}
-
