- 07 Jun, 2018 40 commits
-
-
Nico Weber authored
Enables these tests on the linux memory bots: filesystem_service_unittests leveldb_service_unittests traffic_annotation_auditor_unittests They already run on the regular main waterfall linux bots and on the clang tot bots. Bug: 843511 Change-Id: Ie0f1b5ea6433c2f21f87df993ac205f4553ced89 Reviewed-on: https://chromium-review.googlesource.com/1091070 Commit-Queue: Nico Weber <thakis@chromium.org> Reviewed-by:
Kenneth Russell <kbr@chromium.org> Cr-Commit-Position: refs/heads/master@{#565342}
-
Varun Khaneja authored
The tests check whether we can parse the file properly, identify if the .rar file contains other archives, or executables, etc. Bug: 750327 Change-Id: I71f75521a186115d8ba2c7f2983dfb4abdbbbe1a Reviewed-on: https://chromium-review.googlesource.com/1089853Reviewed-by:
Ken Rockot <rockot@chromium.org> Reviewed-by:
Jialiu Lin <jialiul@chromium.org> Commit-Queue: Varun Khaneja <vakh@chromium.org> Cr-Commit-Position: refs/heads/master@{#565341}
-
Nico Weber authored
They run on the main waterfall, so they should run on the clang/tot bots as well. Bug: 843511 Change-Id: I6fe7256c741d224c090fb38074fe98ff07353149 Reviewed-on: https://chromium-review.googlesource.com/1091071 Commit-Queue: Nico Weber <thakis@chromium.org> Reviewed-by:
Kenneth Russell <kbr@chromium.org> Cr-Commit-Position: refs/heads/master@{#565340}
-
glevin authored
Note that there are 3 app dialogs here generated by the same code: one for uninstalling, one for USB list permission, and one for USB access permission. As per crbug.com/788010 Comment 1 (and previous dlgs): - Allow text to run to right edge of dialog (minus margin) - Remove X (Close) button from upper right - Width = 448px Bug: 788010 Test: Try to uninstall an ARC++ app, or use one to access USB devices. OMG Harmony! Change-Id: I6ffe099ba9f4fe3fae7cd3b6f203873000dd62e8 Reviewed-on: https://chromium-review.googlesource.com/1066450 Commit-Queue: Greg Levin <glevin@chromium.org> Reviewed-by:
Elly Fong-Jones <ellyjones@chromium.org> Cr-Commit-Position: refs/heads/master@{#565339}
-
James Cook authored
They seem to pass now in mash_browser_tests (--enable-features=Mash) Bug: 723880 Change-Id: If1695c2b175dd72188c32233ae016aeef1d9636a Reviewed-on: https://chromium-review.googlesource.com/1091115Reviewed-by:
Xiyuan Xia <xiyuan@chromium.org> Commit-Queue: James Cook <jamescook@chromium.org> Cr-Commit-Position: refs/heads/master@{#565338}
-
Daniele Castagna authored
blink::VideoFrameSubmitter produces quads using VideoFrameResourceProvider.VideoFrameSubmitter, that produces at most one quad. This CL DCHECKs that at most one quad is produced, since we want to use this property for optimizing damage tracking in SurfaceAggregator/GLRenderer. Having the guarantee that only one quad will be produced for this CompositorFrame will allow us to mark the video as the only quad producing damage in SurfaceAggregator. In this way GLRenderer will be able to promote to HW overlay the quad for the video and completely skip compositing of the remaining quads. This will result in significant power savings. Bug: 795008, b/80061773 Cq-Include-Trybots: master.tryserver.blink:linux_trusty_blink_rel;master.tryserver.chromium.linux:linux_layout_tests_slimming_paint_v2 Change-Id: Ic4c612c5c221366bd67039d937dc0e42104b4c56 Reviewed-on: https://chromium-review.googlesource.com/1083977Reviewed-by:
Dan Sanders <sandersd@chromium.org> Reviewed-by:
Stephen White <senorblanco@chromium.org> Reviewed-by:
danakj <danakj@chromium.org> Reviewed-by:
CJ DiMeglio <lethalantidote@chromium.org> Commit-Queue: Daniele Castagna <dcastagna@chromium.org> Cr-Commit-Position: refs/heads/master@{#565337}
-
Tiger Oakes authored
D8 is the new default dexer in Android Studio. We're replacing DX with D8. This improves the APK size by removing 30 KB. Bug: 842681 Change-Id: Ic0d310f615d6aad44a55a2ab5f56ef40288df53f Reviewed-on: https://chromium-review.googlesource.com/1079373 Commit-Queue: Tiger Oakes <tigero@google.com> Reviewed-by:
Eric Stevenson <estevenson@chromium.org> Reviewed-by:
agrieve <agrieve@chromium.org> Cr-Commit-Position: refs/heads/master@{#565336}
-
Robbie McElrath authored
I've made several sensor changes since this test was disabled that should fix the flakiness. Bug: 846920 Change-Id: Ibe38cad96acd7d5636f89d0eb8a6d915ab6798a0 Reviewed-on: https://chromium-review.googlesource.com/1089712 Commit-Queue: Robbie McElrath <rmcelrath@chromium.org> Reviewed-by:
Reilly Grant <reillyg@chromium.org> Cr-Commit-Position: refs/heads/master@{#565335}
-
Nico Weber authored
It runs without issues on the clang tot linux bots, and on the main waterfall cros bots already. Bug: 843511 Change-Id: I9b0fa22cbb5adc146d7d6a72e12c177eb92b0179 Reviewed-on: https://chromium-review.googlesource.com/1090821Reviewed-by:
Kenneth Russell <kbr@chromium.org> Commit-Queue: Nico Weber <thakis@chromium.org> Cr-Commit-Position: refs/heads/master@{#565334}
-
Theresa authored
BUG=841928 Change-Id: I3ca67c780818d360d93830c0507734edba401a6e Reviewed-on: https://chromium-review.googlesource.com/1089993Reviewed-by:
Becky Zhou <huayinz@chromium.org> Commit-Queue: Theresa <twellington@chromium.org> Cr-Commit-Position: refs/heads/master@{#565333}
-
Chris Hamilton authored
These helpers are intended for use with WebContents hosted in tabs, but are occasionally used outside of that context. This simple fix stops the helper from exploding in that case. BUG=849796 Change-Id: I85caef9cd397eec5fd77b8ad16bdcc9d6f3adbbd Reviewed-on: https://chromium-review.googlesource.com/1089294Reviewed-by:
John Abd-El-Malek <jam@chromium.org> Commit-Queue: Chris Hamilton <chrisha@chromium.org> Cr-Commit-Position: refs/heads/master@{#565332}
-
Chris Nardi authored
word-wrap was previously defined as its own property in CSSProperties.json5, but with the same behavior as overflow-wrap. However, this meant that word-wrap showed up alongside word-wrap in cssText. This patch converts word-wrap to be an alias of overflow-wrap, as this will prevent the issue of both properties showing up in cssText and matches the resolution in https://github.com/w3c/csswg-drafts/issues/866. Bug: 679068 Change-Id: I3d415eb1cde9a92dffa2a045ba5fa8b015eeb2d7 Reviewed-on: https://chromium-review.googlesource.com/1073677Reviewed-by:
Philip Jägenstedt <foolip@chromium.org> Reviewed-by:
Rune Lillesveen <futhark@chromium.org> Commit-Queue: Chris Nardi <cnardi@chromium.org> Cr-Commit-Position: refs/heads/master@{#565331}
-
catapult-chromium-autoroll authored
https://chromium.googlesource.com/catapult.git/+log/a02c34c..1331d4b git log a02c34c..1331d4b --date=short --no-merges --format='%ad %ae %s' 2018-06-07 perezju@chromium.org [soundwave] Make RequestError pickleable Created with: gclient setdep -r src/third_party/catapult@1331d4b 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 TBR=sullivan@chromium.org Change-Id: Ib3e40358a70293053b294c6e588657c83b6ba1e9 Reviewed-on: https://chromium-review.googlesource.com/1091030Reviewed-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@{#565330}
-
David 'Digit' Turner authored
Nothin too sexy here, but making small improvements to the code base for long term maintainer happiness, and upcoming CLs that will perform more deep cleanup work. - Adding missing const qualifiers to some LibraryView methods. Note that making LookupSymbol() also const requires more changes that are better left to another CL. Noted with TODO. - Make crazy::SearchResult declaration public by moving it to crazy_linker_util.h - Use a VectorBase non-template class that all Vector<> templates inherit from, in order to reduce the amount of generated code for vectors. - Replace Vector::IndexOf() with Vector::Find() which returns a SearchResult value. Gets rid of pesky 'int' indices for InsertAt / RemoveAt method. - Support ranged-based for loops with begin() / end() declarations. And adjust all callers to use them when that makes sense. Cleaner code. Yeah. BUG=NONE R=lizeb@chromium.org, agrieve@chromium.org, pasko@chromium.org, rmcilroy@chromium.org android: crazy-linker: Small Vector<> optimization. Change-Id: If4d22b85fdd44885159a2bb27626f2942ccb437e Reviewed-on: https://chromium-review.googlesource.com/1090840 Commit-Queue: David Turner <digit@chromium.org> Reviewed-by:
Egor Pasko <pasko@chromium.org> Cr-Commit-Position: refs/heads/master@{#565329}
-
Chris Blume authored
Recently, Skia changed its Vulkan code. This ended up breaking Chrome when building with Vulkan support. This patch fixes the Chrome build with Vulkan. BUG=850342 Change-Id: Ibd12791bca6b06d1124e6014766ed2958bea59c1 Reviewed-on: https://chromium-review.googlesource.com/1089817Reviewed-by:
Brian Salomon <bsalomon@chromium.org> Commit-Queue: Chris Blume <cblume@chromium.org> Cr-Commit-Position: refs/heads/master@{#565328}
-
John Z Wu authored
Cards from Google Pay will require special handling. Cq-Include-Trybots: luci.chromium.try:ios-simulator-full-configs;master.tryserver.chromium.mac:ios-simulator-cronet Change-Id: Ia05ff0f31518272a6aaafc6392e730123d805d4a Reviewed-on: https://chromium-review.googlesource.com/1089581 Commit-Queue: John Wu <jzw@chromium.org> Reviewed-by:
Hiroshi Ichikawa <ichikawa@chromium.org> Reviewed-by:
Eugene But <eugenebut@chromium.org> Cr-Commit-Position: refs/heads/master@{#565327}
-
Koji Ishii authored
This patch changes RunInfo::CreateSubRun() to copy the entire array, then modify character_index. LayoutNG uses CreateSubRun heavily, with much longer ShapeResult than the current layout engine, and this function turned out to be hot. perf_tests/layout/word-break-break-all.html gets ~10% faster, ~700ms to ~630ms. Bug: 636993 Cq-Include-Trybots: master.tryserver.chromium.linux:linux_layout_tests_layout_ng Change-Id: I869d1b90b1afe5cd84ad3b809ce8453f1f8feba4 Reviewed-on: https://chromium-review.googlesource.com/1088527 Commit-Queue: Emil A Eklund <eae@chromium.org> Reviewed-by:
Emil A Eklund <eae@chromium.org> Cr-Commit-Position: refs/heads/master@{#565326}
-
Olga Sharonova authored
To run Audio service out of process on bots. Bug: 832666 Change-Id: I226ee0c9ce8e6adccb11c02dc2c3e13fead9886d Reviewed-on: https://chromium-review.googlesource.com/1065911Reviewed-by:
Robert Kaplow <rkaplow@chromium.org> Commit-Queue: Olga Sharonova <olka@chromium.org> Cr-Commit-Position: refs/heads/master@{#565325}
-
Chromium WPT Sync authored
Using wpt-import in Chromium 2b719243. With Chromium commits locally applied on WPT: e8832df8 "webrtc wpt: remove test_state_change_event" fbdd17fb "Improve RTCPeerConnection-setRemoteDescription-tracks.https.html tests." Build: https://ci.chromium.org/buildbot/chromium.infra.cron/wpt-importer/19162 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, mek@chromium.org: external/wpt/FileAPI TBR=foolip No-Export: true Change-Id: I2c9b4ca003b6a0044ac8f5d4aee7c10411ae59c8 Reviewed-on: https://chromium-review.googlesource.com/1090796 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@{#565324}
-
Hayato Ito authored
If we don't clear slot's assigned nodes when detaching a slot, if the slot assignment recalc flag is dirty (the actual condition is more complex, see the code), *cycle* would happen in detaching (after attaching a slot again) because a series of DOM mutations can create a tree of trees as such: - one of the slot's formerly assigned node, call it |A|, is a shadow-including ancestor of the slot. - |A| is already attached. In this case, detaching the slot will try to lazy-reattach |A|, and lazy-reattaching |A| doesn't return early (because |A| doesn't need Attach). As a result, detaching the slot *cycles* and causes an infinite loop (... -> |A| -> .. -> host -> shadow root -> .. -> slot -> |A| -> ...). See the test (or bug 849599) for a concrete example. Bug: 849599 Change-Id: I1c1ddb06ca9777af0052260aa721c2438da3c62b Reviewed-on: https://chromium-review.googlesource.com/1090420Reviewed-by:
Takayoshi Kochi <kochi@chromium.org> Reviewed-by:
Rune Lillesveen <futhark@chromium.org> Commit-Queue: Hayato Ito <hayato@chromium.org> Cr-Commit-Position: refs/heads/master@{#565323}
-
Keishi Hattori authored
r546092 accidentally removed the ability for GCs to wait until lazy sweeping finishes. This CL refactor idle gc scheduling so that behavior is official. If a GC is scheduled during lazy sweeping, CompleteSweep is called immediately. Also adds GCSchedulingTest tests that test this behavior. Change-Id: I63d947b21ae7d6ba0d8e3e81632cf54eb5baae98 Reviewed-on: https://chromium-review.googlesource.com/1086010 Commit-Queue: Keishi Hattori <keishi@chromium.org> Reviewed-by:
Kentaro Hara <haraken@chromium.org> Reviewed-by:
Michael Lippautz <mlippautz@chromium.org> Cr-Commit-Position: refs/heads/master@{#565322}
-
Mark Pilgrim authored
Bug: 844966 Cq-Include-Trybots: luci.chromium.try:ios-simulator-full-configs;master.tryserver.chromium.mac:ios-simulator-cronet Change-Id: I3c198d95735770668eed4a1342a3979224239529 Reviewed-on: https://chromium-review.googlesource.com/1089132 Commit-Queue: Mark Pilgrim <pilgrim@chromium.org> Reviewed-by:
Mikel Astiz <mastiz@chromium.org> Reviewed-by:
Scott Violet <sky@chromium.org> Reviewed-by:
Matt Menke <mmenke@chromium.org> Cr-Commit-Position: refs/heads/master@{#565321}
-
Kyle Milka authored
When the user selects a background image actually update the page background. The choice of background is stored in Preferences. A new preference is added for this purpose: kNTPCustomBackgroundURL. This is exposed to the NTP javascript through ThemeBackgroundInfo. Add a NewTabPageBinding to update these preferences when an image is selected by propagating the change to the InstantService where the browser profile is available. The InstantService then notifies all New Tab Pages of the change. Bug: 839152 Cq-Include-Trybots: master.tryserver.chromium.linux:closure_compilation Change-Id: Ib6f74f1f3e9dda6a3352833662db373ec1358769 Reviewed-on: https://chromium-review.googlesource.com/1081651 Commit-Queue: Kyle Milka <kmilka@chromium.org> Reviewed-by:
Emily Stark <estark@chromium.org> Reviewed-by:
Bernhard Bauer <bauerb@chromium.org> Reviewed-by:
Marc Treib <treib@chromium.org> Cr-Commit-Position: refs/heads/master@{#565320}
-
Filip Gorski authored
* Adding ability to parameterize fetch delay time at a feature level. * Udpating and adding tests. Bug: 838748 Change-Id: I87e2a33a9ee94111615b80566d6d611600271c29 Reviewed-on: https://chromium-review.googlesource.com/1089881 Commit-Queue: Filip Gorski <fgorski@chromium.org> Reviewed-by:
Theresa <twellington@chromium.org> Cr-Commit-Position: refs/heads/master@{#565319}
-
Devlin Cronin authored
Introduce a set of runtime-granted permissions to keep track of all permissions that were granted at runtime (rather than at install-time). This includes optional permissions, as well as permissions granted with the runtime host permissions feature. Add support in ExtensionPrefs, and add/remove permissions through PermissionsUpdater::AddPermissions and PermissionsUpdater::RemovePermissions, which are only used for runtime permissions. Add unittests for storing the permissions in prefs, and updating them via the PermissionsUpdater and the ScriptingPermissionsModifier (used with the runtime host permissions feature). Bug: 848448 Change-Id: I57377d367e28c3738825e794dd7847137eae004c Reviewed-on: https://chromium-review.googlesource.com/1086067 Commit-Queue: Devlin <rdevlin.cronin@chromium.org> Reviewed-by:
Karan Bhatia <karandeepb@chromium.org> Cr-Commit-Position: refs/heads/master@{#565318}
-
Dominic Farolino authored
Before this CL, ResourceRequest-based logic for all <link>s that were not rel=manifest or rel=import resided in LinkStyle. This CL pushes that logic down to LinkLoader for a better separation of concerns. R=yoav@yoav.ws Bug: n/a Change-Id: If1cc664b7de2ad54cf73d63a2e0d14943e698d04 Reviewed-on: https://chromium-review.googlesource.com/1087351 Commit-Queue: Dominic Farolino <domfarolino@gmail.com> Reviewed-by:
Yoav Weiss <yoav@yoav.ws> Cr-Commit-Position: refs/heads/master@{#565317}
-
Christian Dullweber authored
Deletion of CONTENT_SETTINGS_TYPE_PASSWORD_PROTECTION wasn't covered by ChromeBrowsingDataRemoverDelegate::AllTypesAreGettingDeleted yet. This is fixed by creating a PasswordProtectionService during the test and removing the type from the whitelist. Bug: 710873 Change-Id: I4b2dd8ffd0eb2996f89e616b3c94626fc04b3982 Reviewed-on: https://chromium-review.googlesource.com/1069090 Commit-Queue: Christian Dullweber <dullweber@chromium.org> Reviewed-by:
Martin Šrámek <msramek@chromium.org> Reviewed-by:
Jialiu Lin <jialiul@chromium.org> Cr-Commit-Position: refs/heads/master@{#565316}
-
Bo Liu authored
Layout depends on tab visibility now, so need to update layout. Switching between incognito and normal tabs on the tablet interface changes tab visibility without layout, which was broken previously. Bug: 849191 Change-Id: If3ab0df7a1316cd20930b9e9a25d64f2beb3e6b5 Reviewed-on: https://chromium-review.googlesource.com/1090312Reviewed-by:
Matthew Jones <mdjones@chromium.org> Commit-Queue: Bo <boliu@chromium.org> Cr-Commit-Position: refs/heads/master@{#565315}
-
Ted Choc authored
Ran locally 50 times w/o flake. This also cleans up the logging and test prep to make it easier to detect systemic failures. BUG=673930 Change-Id: I86349f9256ddc554c8de12d9dca80fc06458aa20 Reviewed-on: https://chromium-review.googlesource.com/1089649Reviewed-by:
David Trainor <dtrainor@chromium.org> Commit-Queue: Ted Choc <tedchoc@chromium.org> Cr-Commit-Position: refs/heads/master@{#565314}
-
Alexis Hetu authored
When running chromium unit tests using SwiftShader, we need a way to enable high precision filtering. In order to do that, the GL command decoder must allow the GL_TEXTURE_FILTERING_HINT_CHROMIUM hint to be used. This cl allows access to this hint inside unit tests. Bug: chromium:848035 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: I0a2f90b1cff4767f82175cd20f87c83e900f3e20 Reviewed-on: https://chromium-review.googlesource.com/1080020 Commit-Queue: Alexis Hétu <sugoi@chromium.org> Reviewed-by:
Antoine Labour <piman@chromium.org> Cr-Commit-Position: refs/heads/master@{#565313}
-
Steven Bennetts authored
This CL: * Shows the settings > details UI for non connectable networks for secondary users. * Disables toggles for secondary users. Bug: 843888,827096 Change-Id: Ie3b0554211e1ad42f910d921b55d6553e3a55836 Reviewed-on: https://chromium-review.googlesource.com/1089585 Commit-Queue: Steven Bennetts <stevenjb@chromium.org> Reviewed-by:
Kyle Horimoto <khorimoto@chromium.org> Cr-Commit-Position: refs/heads/master@{#565312}
-
Ulan Degenbaev authored
This is a partial revert of 269fa691. Bug: 847511 Change-Id: Ie8e5e1620e3748903b8c8dcdb6d190a7b03d1fe7 Reviewed-on: https://chromium-review.googlesource.com/1090925Reviewed-by:
Kentaro Hara <haraken@chromium.org> Commit-Queue: Ulan Degenbaev <ulan@chromium.org> Cr-Commit-Position: refs/heads/master@{#565311}
-
John Abd-El-Malek authored
It's too slow on Windows, and on Windows we shard it. Change-Id: I2747e7b5bc3b1fa4adf7e574f0c3a875dc3655e9 Reviewed-on: https://chromium-review.googlesource.com/1090314Reviewed-by:
Chong Zhang <chongz@chromium.org> Commit-Queue: John Abd-El-Malek <jam@chromium.org> Cr-Commit-Position: refs/heads/master@{#565310}
-
Xing Liu authored
The storage summary string under download home title currently uses SpannableString that can expand to multiple lines, and can only apply simple styles defined by Spannale. This CL adds a view for the summary string. Bug: 849973 Change-Id: If01d323022e0cc743540a02ba58801c664957aa4 Reviewed-on: https://chromium-review.googlesource.com/1089815Reviewed-by:
David Trainor <dtrainor@chromium.org> Commit-Queue: Xing Liu <xingliu@chromium.org> Cr-Commit-Position: refs/heads/master@{#565309}
-
angle-chromium-autoroll authored
https://chromium.googlesource.com/angle/angle.git/+log/2bd1fab..ea17575 git log 2bd1fab..ea17575 --date=short --no-merges --format='%ad %ae %s' 2018-06-07 lucferron@chromium.org Vulkan: enable invariance dEQP tests Created with: gclient setdep -r src/third_party/angle@ea17575 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=geofflang@chromium.org Change-Id: Ie1bfb05d8d242c36d09dd57972cf34e3898719dd Reviewed-on: https://chromium-review.googlesource.com/1090970Reviewed-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@{#565308}
-
Elad Alon authored
When browsing data is cleared, in addition to deleting pending logs, also cancel ongoing uploads of WebRTC event log files which fit the user profile and time range that is being cleared. To facilitate this change, it was necessary to change the way WebRtcEventLogUploaderImpl gets hold of a URLRequestContextGetter. Instead of thread hopping to get one, it is now given one in the constructor. Since g_browser_process->system_request_context() is not available when WebRtcEventLogManager is constructed, setting the URLRequestContextGetter is deferred to the time when the first user profile is loaded. Bug: 775415 Change-Id: I2b19b95d7acba44c84d0fb72b014b9e4c6e2cdf7 Reviewed-on: https://chromium-review.googlesource.com/1078737 Commit-Queue: Elad Alon <eladalon@chromium.org> Reviewed-by:
Guido Urdaneta <guidou@chromium.org> Cr-Commit-Position: refs/heads/master@{#565307}
-
pdfium-chromium-autoroll authored
https://pdfium.googlesource.com/pdfium.git/+log/d5c4ab1..5ebfd64 git log d5c4ab1..5ebfd64 --date=short --no-merges --format='%ad %ae %s' 2018-06-07 hnakashima@chromium.org Test to show editing a PageObject and saving the .pdf duplicates it. Created with: gclient setdep -r src/third_party/pdfium@5ebfd64 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: I11c01fcce6baf33e4e8a5cb49c6e61beb0c669e3 Reviewed-on: https://chromium-review.googlesource.com/1090763Reviewed-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@{#565306}
-
Eric Stevenson authored
Removing the reference to BuildHooksAndroidImpl from BuildHooksAndroid shrinks the main dex by ~2000 methods for downstream targets. Bug: 820570 Change-Id: If4051f7a85e387129c65a7a5b927183b5b36bf65 Reviewed-on: https://chromium-review.googlesource.com/1089625 Commit-Queue: Eric Stevenson <estevenson@chromium.org> Reviewed-by:
agrieve <agrieve@chromium.org> Cr-Commit-Position: refs/heads/master@{#565305}
-
Emily Hanley authored
NOTRY=true # CQ flake Bug: 713357 Change-Id: Ib0a4a63c02c78ac3015cef4e03eff07d3a2e9197 Reviewed-on: https://chromium-review.googlesource.com/1090757 Commit-Queue: Ned Nguyen <nednguyen@google.com> Reviewed-by:
Ned Nguyen <nednguyen@google.com> Cr-Commit-Position: refs/heads/master@{#565304}
-
Evan Stade authored
Only support native notifications, i.e. those passed over mojo. Some tests are permanently removed on Chrome OS. Some tests are migrated to message_center_unittests. This relands commit 4da4d1e4 originally reviewed on: https://chromium-review.googlesource.com/1083937 Bug: 578868 Change-Id: I87834ba69bcb37db599e3e046225b793d09b9396 Reviewed-on: https://chromium-review.googlesource.com/1089917Reviewed-by:
Steven Bennetts <stevenjb@chromium.org> Reviewed-by:
Scott Violet <sky@chromium.org> Commit-Queue: Evan Stade <estade@chromium.org> Cr-Commit-Position: refs/heads/master@{#565303}
-