- 31 Jul, 2019 30 commits
-
-
chromium-autoroll authored
https://skia.googlesource.com/skia.git/+log/245c002bc098..6980c4e0aef8 git log 245c002bc098..6980c4e0aef8 --date=short --no-merges --format='%ad %ae %s' 2019-07-30 egdaniel@google.com Remove isConfigCopyable from GrCaps. 2019-07-30 fmalita@chromium.org [skottie] Safe div-by-zero in LevelsEffectAdapter::apply Created with: gclient setdep -r src/third_party/skia@6980c4e0aef8 The AutoRoll server is located here: https://autoroll.skia.org/r/skia-autoroll 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-blink-rel;luci.chromium.try:linux-chromeos-compile-dbg;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=benjaminwagner@google.com Bug: None Change-Id: I9ca1df013b4fd2e21f6be5c5bb71fb6f00953264 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1726645Reviewed-by:
chromium-autoroll <chromium-autoroll@skia-public.iam.gserviceaccount.com> Commit-Queue: chromium-autoroll <chromium-autoroll@skia-public.iam.gserviceaccount.com> Cr-Commit-Position: refs/heads/master@{#682549}
-
Jeremy Chinsen authored
Currently the logic in PDFiumEngine::FillPageSides() for drawing the empty space below a page only handles single-up view. In two-up view, a page may be shorter than the one adjacent to it. In this case, the bottom gap must have height as tall as the difference in the adjacent pages' height. This CL changes FillPageSides() such that when |two_up_view_| is true, it calls draw_utils::GetBottomGapBetweenRects() with the current page and |dirty_in_screen|, computing the space of |dirty_in_screen| that is below the page, and finally drawing it as empty space. This allows FillPageSides() to properly draw the bottom empty space for a page in two-up view. Bug: 51472 Change-Id: Idd01fd959419fd8aebc2f70f514eb83ba61c160c Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1720792Reviewed-by:
Lei Zhang <thestig@chromium.org> Commit-Queue: Jeremy Chinsen <chinsenj@google.com> Cr-Commit-Position: refs/heads/master@{#682548}
-
chromium-autoroll authored
https://swiftshader.googlesource.com/SwiftShader.git/+log/c39e7c7cc884..324bdfed0692 git log c39e7c7cc884..324bdfed0692 --date=short --no-merges --format='%ad %ae %s' 2019-07-30 capn@google.com Use a finite maxSamplerLodBias limit Created with: gclient setdep -r src/third_party/swiftshader@324bdfed0692 The AutoRoll server is located here: https://autoroll.skia.org/r/swiftshader-chromium-autoroll 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=swiftshader-team+autoroll@google.com Bug: chromium:b/138670766 Change-Id: Ia4a409c84d7cb4d204d514900226db05aab85dca Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1726643Reviewed-by:
chromium-autoroll <chromium-autoroll@skia-public.iam.gserviceaccount.com> Commit-Queue: chromium-autoroll <chromium-autoroll@skia-public.iam.gserviceaccount.com> Cr-Commit-Position: refs/heads/master@{#682547}
-
Miyoung Shin authored
This CL renames mojo to mojom in //media/capture/video/chromeos. media/capture/video/chromeos/mojo => media/capture/video/chromeos/mojom Bug: 806965 Change-Id: I2b739ea05a99a02e9654659ec2b04a309b8dc26b Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1724901Reviewed-by:
Guido Urdaneta <guidou@chromium.org> Reviewed-by:
Colin Blundell <blundell@chromium.org> Reviewed-by:
Kinuko Yasuda <kinuko@chromium.org> Reviewed-by:
Ken Rockot <rockot@google.com> Commit-Queue: Miyoung Shin <myid.shin@igalia.com> Cr-Commit-Position: refs/heads/master@{#682546}
-
tby authored
This CL moves Roselle out of the search providers and into our ranking infrastructure. The logic for scoring apps is kept the same, but now happens in two different places: its time-based relevance score is still set in the AppSearchProvider but, if it is present in the ranker, the relevance is overridden by the ranker's score. Roselle is also used in the ArcAppShortcutsSearchProvider. Since we've finished the experiments ranking Arc app shortcuts and don't have plans to in the near future, this logic has been removed. Bug: 989350 Change-Id: I5c1e891508b669c5b91e7164e94af9fa7a6fbbaf Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1694561 Commit-Queue: Tony Yeoman <tby@chromium.org> Reviewed-by:
Jenny Zhang <jennyz@chromium.org> Reviewed-by:
Charles . <charleszhao@chromium.org> Cr-Commit-Position: refs/heads/master@{#682545}
-
Theresa authored
We still need these to collect data for various experiments. Extends the following histograms to M82. - Settings.ShowHomeButtonPreferenceState - Settings.HomePageIsCustomized - Settings.ShowHomeButtonPreferenceStateChanged - Settings.GivenShowHomeButton_HomePageIsNewTabPage - Settings.ShowHomeButton BUG=986514,986515 Change-Id: Ie1e4156010a783e310997c2dded55816b8cde83d Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1727514Reviewed-by:
Mark Pearson <mpearson@chromium.org> Commit-Queue: Theresa <twellington@chromium.org> Cr-Commit-Position: refs/heads/master@{#682544}
-
Miguel Casas authored
This CL adds a histogram for Media.MojoVideoDecoder.Decode; this is already tracked as an async trace, for the purpose of CrOs Tast, a histogram is needed.(This is basically the same case as crrev.com/c/1680764) Bug: 953253 Change-Id: I4cab76dd739f7b8196b29ec5870e141f04b1076c Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1725313Reviewed-by:
Dan Sanders <sandersd@chromium.org> Reviewed-by:
Stephen Chenney <schenney@chromium.org> Commit-Queue: Miguel Casas <mcasas@chromium.org> Cr-Commit-Position: refs/heads/master@{#682543}
-
Julie Jeongeun Kim authored
This CL is a part of moving access token management to OAuth2AccessTokenManager. It is specifically a step toward folding O2TS into PO2TS now that O2TS is only one subclass of PO2TS. OAuth2TokenService has been folded completely into ProfileOAuth2TokenService on https://crrev.com/c/1712560. So, it's not valid to comment OAuth2TokenService on the current codebase. This CL updates comments for OAuth2TOkenService with IdentityManager, the access token fetcher or OAuth2AccessTokenManager. Bug: 967598 Change-Id: I1aba6c57d8cc65063f12be512cece3404d602fcf Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1725222 Commit-Queue: Julie Jeongeun Kim <jkim@igalia.com> Reviewed-by:
Sam McNally <sammc@chromium.org> Reviewed-by:
Maksim Ivanov <emaxx@chromium.org> Reviewed-by:
Colin Blundell <blundell@chromium.org> Cr-Commit-Position: refs/heads/master@{#682542}
-
v8-ci-autoroll-builder authored
Summary of changes available at: https://chromium.googlesource.com/v8/v8/+log/2ddb2cc3..79a5354d Please follow these instructions for assigning/CC'ing issues: https://v8.dev/docs/triage-issues Please close rolling in case of a roll revert: https://v8-roll.appspot.com/ This only works with a Google account. CQ_INCLUDE_TRYBOTS=luci.chromium.try:linux-blink-rel CQ_INCLUDE_TRYBOTS=luci.chromium.try:linux_optional_gpu_tests_rel CQ_INCLUDE_TRYBOTS=luci.chromium.try:mac_optional_gpu_tests_rel CQ_INCLUDE_TRYBOTS=luci.chromium.try:win_optional_gpu_tests_rel CQ_INCLUDE_TRYBOTS=luci.chromium.try:android_optional_gpu_tests_rel TBR=hablich@chromium.org,v8-waterfall-sheriff@grotations.appspotmail.com Change-Id: Ic9ad500efd1806b331d56b5b2f71493d7975a8a9 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1727170Reviewed-by:
v8-ci-autoroll-builder <v8-ci-autoroll-builder@chops-service-accounts.iam.gserviceaccount.com> Commit-Queue: v8-ci-autoroll-builder <v8-ci-autoroll-builder@chops-service-accounts.iam.gserviceaccount.com> Cr-Commit-Position: refs/heads/master@{#682541}
-
Tina Wang authored
Previously the omnibox icon will show and hide according to whether the user focuses on the omnibox. Now it won't hide when the user unfocuses the omnibox. There is a updated summary of how send tab to self omnibox icon shows and hides: - If showing: keep showing til the user start to edit the url. - If hidden: show if send tab to self feature is offered, omnibox is focused, and the url has not been edited. Bug: 978520 Change-Id: Id3f2a368a283c43f35dbe83847d593eba2498e66 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1715511 Commit-Queue: Tina Wang <tinazwang@chromium.org> Reviewed-by:
Avi Drissman <avi@chromium.org> Cr-Commit-Position: refs/heads/master@{#682540}
-
Xing Liu authored
This CL catches an exception in PackageMetrics.getPackageStatsForAndroidO. The crash happens if (Binder.getCallingUid() != appInfo.uid). Currently we don't know the reason. Something tries to launch Chrome process in a remote process. Bug: 984761 Change-Id: If6186d9218fdde99be95c4ad8ec4da896abbba4d Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1725473Reviewed-by:
David Trainor <dtrainor@chromium.org> Commit-Queue: Xing Liu <xingliu@chromium.org> Cr-Commit-Position: refs/heads/master@{#682539}
-
Alex Cooper authored
This addresses feedback submitted to crrev.com/c/1725319 after it was submitted. Mainly around using exact equality for whole numbers and the math around a length of zero. Bug: 977703 Change-Id: Ia43f204a9f754077cda6bd74b2e901f724c7ee9b Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1727571 Commit-Queue: Alexander Cooper <alcooper@chromium.org> Commit-Queue: Klaus Weidner <klausw@chromium.org> Auto-Submit: Alexander Cooper <alcooper@chromium.org> Reviewed-by:
Klaus Weidner <klausw@chromium.org> Cr-Commit-Position: refs/heads/master@{#682538}
-
Dominic Battre authored
This CL improves the readability of votes uploads: Before the CL, you would see "autofill_type: [51]". After the CL, you now see "autofill_type: [51/CREDIT_CARD_NAME_FULL]". If std::vector<std::string> elements are piped into the LogBuffer, these are automatically represented as a comma separated list in the output. R=kolos@chromium.org Bug: 928595 Change-Id: I68c917be57c5dbf32a7932235b8c52dc8916cf97 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1724521 Commit-Queue: Dominic Battré <battre@chromium.org> Reviewed-by:
Maxim Kolosovskiy <kolos@chromium.org> Cr-Commit-Position: refs/heads/master@{#682537}
-
Yuly Novikov authored
Start running tests on Win10 x64 Intel experimental bot. TBR=kbr Bug: 988833 Change-Id: Iadea428fd44ec07477c574d06d1cd8dc7d52890f Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1726758Reviewed-by:
Yuly Novikov <ynovikov@chromium.org> Reviewed-by:
Takuto Ikuta <tikuta@chromium.org> Commit-Queue: Yuly Novikov <ynovikov@chromium.org> Cr-Commit-Position: refs/heads/master@{#682536}
-
Ian Wells authored
When sharing an offline page after opening it from Downloads Home, share the original URL if a content URI can't be generated. Also, on Android Q+, if the page's file path is already a content URI, share that directly. Bug: 985699 Change-Id: I53c2e6effb929a00c4e083206bce01338610dc02 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1713765Reviewed-by:
David Trainor <dtrainor@chromium.org> Reviewed-by:
Dan H <harringtond@chromium.org> Commit-Queue: Ian Wells <iwells@chromium.org> Cr-Commit-Position: refs/heads/master@{#682535}
-
Angel Alvarez authored
Tweaks UI of Explore Sites Pages to conform with initial UI review feedback, including modification of the icon corner radius to conform with Android system UI guidelines. Bug: 977713, 989214 Change-Id: I4593587451fb2dd53ecfc7af367aa06f8bf7012c Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1717920Reviewed-by:
Cathy Li <chili@chromium.org> Reviewed-by:
Theresa <twellington@chromium.org> Commit-Queue: Angel Alvarez <angelii@google.com> Cr-Commit-Position: refs/heads/master@{#682534}
-
Chris Harrelson authored
If the first parameter is nullptr, the second must be also, or else the code will try to dereference |ancestor|. Bug: 988960 Change-Id: I74ada6169a77c6dce08695abf5d94d3df3bc5176 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1726722Reviewed-by:
Mason Freed <masonfreed@chromium.org> Commit-Queue: Chris Harrelson <chrishtr@chromium.org> Cr-Commit-Position: refs/heads/master@{#682533}
-
Michael Spang authored
Without flushing after calling XMapWindow, the window may never appear. Bug: 988915 Test: gn gen out/chromecast_rel --args='use_ozone=true is_chromecast=true' ninja -C out/chromecast_rel cast_shell ./out/chromecast_rel/cast_shell --ozone-platform=x11 Change-Id: I6f6f08d4baa981cdee78adfa0056fc51b83e6dc5 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1725243 Auto-Submit: Michael Spang <spang@chromium.org> Reviewed-by:
Thomas Anderson <thomasanderson@chromium.org> Commit-Queue: Michael Spang <spang@chromium.org> Cr-Commit-Position: refs/heads/master@{#682532}
-
Robert Liao authored
This allows the HatsService to have multiple site IDs and specify what site ID to use. BUG=979530 Change-Id: I6b255a2ebe9cc1cb5e4b7ec10ab98f3f3b50c6dc Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1711055Reviewed-by:
Scott Violet <sky@chromium.org> Reviewed-by:
Wei Li <weili@chromium.org> Commit-Queue: Robert Liao <robliao@chromium.org> Auto-Submit: Robert Liao <robliao@chromium.org> Cr-Commit-Position: refs/heads/master@{#682531}
-
Joe DeBlasio authored
This CL adds an advanced section to the lookalike URL interstitial, updates the strings, and adds support for different descriptions for top site and engagement matches. The advanced section is hidden behind a feature param for now, so this CL has no immediately-user-visible change. Bug: 987359 Change-Id: Ic4ffb3610db7608de0adeb020b253caf8755e2ed Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1717191Reviewed-by:
Mustafa Emre Acer <meacer@chromium.org> Commit-Queue: Joe DeBlasio <jdeblasio@chromium.org> Auto-Submit: Joe DeBlasio <jdeblasio@chromium.org> Cr-Commit-Position: refs/heads/master@{#682530}
-
Tom Anderson authored
This is necessary for switching GlobalMenuBarX11 from gio to //dbus. GlobalMenuBarX11 lazily creates the history menu when it is opened, so we cannot call SetModel() because that would remove the menu and replace it with a new one. We need to update the menu in-place, which is now possible with MenuLayoutUpdated(). Also provided is MenuItemsPropertiesUpdated(), which avoids having to rebuild the entire menu tree if a single item becomes checked/unchecked, disabled/enabled, etc. BUG=981201 R=thestig Change-Id: I36397810ba38f16ef44206282203db16edfdd0c0 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1720796 Commit-Queue: Thomas Anderson <thomasanderson@chromium.org> Reviewed-by:
Lei Zhang <thestig@chromium.org> Reviewed-by:
Nico Weber <thakis@chromium.org> Cr-Commit-Position: refs/heads/master@{#682529}
-
Jenna Himawan authored
Bug: 988044, 160194 Change-Id: Ieecb98d113d7a769aebf6be1a3694f1a2c1a5141 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1710773 Commit-Queue: Jenna Himawan <jhimawan@google.com> Reviewed-by:
Jesse Doherty <jwd@chromium.org> Reviewed-by:
Theresa <twellington@chromium.org> Cr-Commit-Position: refs/heads/master@{#682528}
-
Patrick Noland authored
Also, add a flag that disables the InstantExtended limit on search suggestions. Bug: 974852 Change-Id: I9f3a443b89c83cc4fa958756e25448688d9e0370 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1725172 Commit-Queue: Patrick Noland <pnoland@chromium.org> Reviewed-by:
Justin Donnelly <jdonnelly@chromium.org> Cr-Commit-Position: refs/heads/master@{#682527}
-
Kristi Park authored
Like placeholder text in input fields, this should not be selectable by users. Bug: 987948 Change-Id: I56d4586878bcd09fab1ba6186379305d3163dfae Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1726936 Commit-Queue: Kristi Park <kristipark@chromium.org> Commit-Queue: Gayane Petrosyan <gayane@chromium.org> Auto-Submit: Kristi Park <kristipark@chromium.org> Reviewed-by:
Gayane Petrosyan <gayane@chromium.org> Cr-Commit-Position: refs/heads/master@{#682526}
-
Eric Stevenson authored
If a class is marked @MainDex and needed outside of the browser process then it's possible that a native method may be referenced outside of the browser process. Currently, this can cause problems for NativeMethods JNI interfaces on KitKat since the generated wrapper classes do not have the @MainDex annotation. This CL changes the JNI annotation processor to propagate @MainDex to generated native wrapper classes where necessary. Bug: 929661 Change-Id: I2f6e09475032550768ec3b03cc3b52519ba0dc3c Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1726815 Commit-Queue: Andrew Grieve <agrieve@chromium.org> Reviewed-by:
Andrew Grieve <agrieve@chromium.org> Auto-Submit: Eric Stevenson <estevenson@chromium.org> Cr-Commit-Position: refs/heads/master@{#682525}
-
Avery Musbach authored
The present CL is just some simple code cleanup. Change-Id: Ica26c2b3f9bdfdd18a93debe1350db76f08760ea Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1727369 Commit-Queue: Avery Musbach <amusbach@chromium.org> Reviewed-by:
Mitsuru Oshima <oshima@chromium.org> Cr-Commit-Position: refs/heads/master@{#682524}
-
Ken Rockot authored
The only services used in production ios code are the FilePatcher and Unzipper services, and these have been moved off of Service Manager. Identity service has no clients on ios so its support does not need to be maintained, and all other Service Manager usage was either only for test coverage, or nominal use of the InterfaceProvider API. The latter has been replaced with a new WebState::InterfaceBinder API which is roughly equivalent. The net result of this CL is the complete removal of all services/service_manager dependencies from ios/. Bug: 977637 Change-Id: I88ee321cf4fe58c550ac235de828ff93aa940f00 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1725482 Commit-Queue: Ken Rockot <rockot@google.com> Reviewed-by:
Eugene But <eugenebut@chromium.org> Reviewed-by:
Robert Sesek <rsesek@chromium.org> Cr-Commit-Position: refs/heads/master@{#682523}
-
Rune Lillesveen authored
This is necessary for having a per element color scheme for form controls in web pages using the color-scheme CSS property. Bug: 929098 Change-Id: I45310baec7e19fe8597d8a2359a77595909da7d8 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1650341Reviewed-by:
Elly Fong-Jones <ellyjones@chromium.org> Reviewed-by:
Chris Harrelson <chrishtr@chromium.org> Reviewed-by:
Kinuko Yasuda <kinuko@chromium.org> Reviewed-by:
Robert Liao <robliao@chromium.org> Reviewed-by:
Mason Freed <masonfreed@chromium.org> Commit-Queue: Rune Lillesveen <futhark@chromium.org> Cr-Commit-Position: refs/heads/master@{#682522}
-
chromium-autoroll authored
https://pdfium.googlesource.com/pdfium.git/+log/12bc1c4dae87..ffff6c5fcc92 git log 12bc1c4dae87..ffff6c5fcc92 --date=short --no-merges --format='%ad %ae %s' 2019-07-30 tsepez@chromium.org Make CPDF_Font a retainable type. Created with: gclient setdep -r src/third_party/pdfium@ffff6c5fcc92 The AutoRoll server is located here: https://autoroll.skia.org/r/pdfium-autoroll 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=pdfium-deps-rolls@chromium.org Bug: None Change-Id: Ie38918188e3ba1245a17014d1dca5c93b731e212 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1726836Reviewed-by:
chromium-autoroll <chromium-autoroll@skia-public.iam.gserviceaccount.com> Commit-Queue: chromium-autoroll <chromium-autoroll@skia-public.iam.gserviceaccount.com> Cr-Commit-Position: refs/heads/master@{#682521}
-
chromium-autoroll authored
https://chromium.googlesource.com/chromiumos/chromite.git/+log/019109290b69..67d90245bad4 git log 019109290b69..67d90245bad4 --date=short --no-merges --format='%ad %ae %s' 2019-07-30 vapier@chromium.org convert next to __next__ (and .next() to next()) Created with: gclient setdep -r src/third_party/chromite@67d90245bad4 The AutoRoll server is located here: https://autoroll.skia.org/r/chromite-chromium-autoroll 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:chromeos-kevin-rel TBR=chrome-os-gardeners@google.com Bug: None Change-Id: I34b2e8becf6e79873e52b4ad19c8a343d9a1a9ca Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1727394Reviewed-by:
chromium-autoroll <chromium-autoroll@skia-public.iam.gserviceaccount.com> Commit-Queue: chromium-autoroll <chromium-autoroll@skia-public.iam.gserviceaccount.com> Cr-Commit-Position: refs/heads/master@{#682520}
-
- 30 Jul, 2019 10 commits
-
-
David Pursell authored
An extra "options" field was added to async_wait_t, zero it out by default. Change-Id: Ib6427879872450f1a530475fddf971c55e74a28a Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1719464Reviewed-by:
Wez <wez@chromium.org> Commit-Queue: David Pursell <dpursell@chromium.org> Cr-Commit-Position: refs/heads/master@{#682519}
-
Mitsuru Oshima authored
* Fixed data dependency so that it builds necessary pak files. * Quirks manager isn't available in this test. * Do not run tasks in NetworkStateTestHelper setup, as it can run the tasks that should be run after setup. * Set the host window bounds in AshContentTest::SetUp before initializing ash. AshTestHelper will set its own otherwise. * Compute the test window size based on the display size. The current code uses fixed size and smaller than one used in interactive_ui_tests. Change-Id: I20725e1e5ae6ca9b93af6538b11bebf02bd2485b Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1722205 Commit-Queue: Mitsuru Oshima <oshima@chromium.org> Reviewed-by:
Steven Bennetts <stevenjb@chromium.org> Reviewed-by:
Xiyuan Xia <xiyuan@chromium.org> Cr-Commit-Position: refs/heads/master@{#682518}
-
Kyle Milka authored
Use a different std::string ctor to avoid the overflow. Also return false in the pointer is null. Bug: 988896 Change-Id: I5986937faf3bcc169b329e58efd2438db2d50f0d Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1727469Reviewed-by:
Gayane Petrosyan <gayane@chromium.org> Commit-Queue: Kyle Milka <kmilka@chromium.org> Cr-Commit-Position: refs/heads/master@{#682517}
-
chromium-autoroll authored
https://skia.googlesource.com/skia.git/+log/6e86c1b05eee..245c002bc098 git log 6e86c1b05eee..245c002bc098 --date=short --no-merges --format='%ad %ae %s' 2019-07-30 michaelludwig@google.com Remove use of SkDropShadowImageFilter::ShadowMode internally 2019-07-30 bungeman@google.com Fix potential use of uninitialized data in with GDI. Created with: gclient setdep -r src/third_party/skia@245c002bc098 The AutoRoll server is located here: https://autoroll.skia.org/r/skia-autoroll 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-blink-rel;luci.chromium.try:linux-chromeos-compile-dbg;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=benjaminwagner@google.com Bug: None Change-Id: I862b15e213018a5ea70b3f2a4ee13d7a0e837408 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1726656Reviewed-by:
chromium-autoroll <chromium-autoroll@skia-public.iam.gserviceaccount.com> Commit-Queue: chromium-autoroll <chromium-autoroll@skia-public.iam.gserviceaccount.com> Cr-Commit-Position: refs/heads/master@{#682516}
-
Changwan Ryu authored
Bug: 921655 Test: browser_tests --gtest_filter=ClientHintsBrowserTest.ClientHintsLifetimeNotAttachedJavaScriptBlocked* Change-Id: I19bf9d0c0c42b8f763906544bdb133d65d13bf93 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1688362Reviewed-by:
Bo <boliu@chromium.org> Reviewed-by:
Arthur Sonzogni <arthursonzogni@chromium.org> Reviewed-by:
Tarun Bansal <tbansal@chromium.org> Commit-Queue: Changwan Ryu <changwan@chromium.org> Cr-Commit-Position: refs/heads/master@{#682515}
-
Darren Shen authored
InputMethodEngineBase seems to keep track of a local cached composition but it's not actually needed. We delete this because it is error-prone: every operation needs to update the cache correctly. Bug: 980437 Change-Id: I786317a7b67813ab4a76de9bebbb3f3aa5030503 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1688751 Auto-Submit: Darren Shen <shend@chromium.org> Reviewed-by:
Shu Chen <shuchen@chromium.org> Commit-Queue: Darren Shen <shend@chromium.org> Cr-Commit-Position: refs/heads/master@{#682514}
-
chromium-autoroll authored
https://swiftshader.googlesource.com/SwiftShader.git/+log/fa0175c0988d..c39e7c7cc884 git log fa0175c0988d..c39e7c7cc884 --date=short --no-merges --format='%ad %ae %s' 2019-07-30 capn@google.com Retry mmap() without MAP_JIT on older macOS 2019-07-30 capn@google.com Specify the MAP_JIT flag when allocating executable memory on macOS Created with: gclient setdep -r src/third_party/swiftshader@c39e7c7cc884 The AutoRoll server is located here: https://autoroll.skia.org/r/swiftshader-chromium-autoroll 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=swiftshader-team+autoroll@google.com Bug: chromium:985816,chromium:985816 Change-Id: Iea2cf84af474440160c8c7e5b9ac5c4ec1554ea2 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1727395Reviewed-by:
chromium-autoroll <chromium-autoroll@skia-public.iam.gserviceaccount.com> Commit-Queue: chromium-autoroll <chromium-autoroll@skia-public.iam.gserviceaccount.com> Cr-Commit-Position: refs/heads/master@{#682513}
-
Ian Wells authored
Prevent calling GetCurrentArticleSuggestions() when prefetch_prefs::IsEnabled() is false, but still check whether client is forbidden by the server from prefetching. Bug: 982468 Change-Id: I2acbbdff440a81fd1f6419c81641a27a6bc2af06 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1726657Reviewed-by:
Dan H <harringtond@chromium.org> Commit-Queue: Ian Wells <iwells@chromium.org> Cr-Commit-Position: refs/heads/master@{#682512}
-
Clark DuVall authored
Follow ups to http://crrev.com/c/1721933. The deleted_frame_data_map_ was always getting empty FrameData because the render frame is no longer live by the time OnRenderFrameDeleted is called. Bug: 980774 Change-Id: I1be29171c25e3ff2f8aae74be408833c5b4ebc85 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1726812Reviewed-by:
Karan Bhatia <karandeepb@chromium.org> Commit-Queue: Clark DuVall <cduvall@chromium.org> Cr-Commit-Position: refs/heads/master@{#682511}
-
Olivier Yiptong authored
Using the Native File System API to write files, users will create a configurable FileWriter which defaults to writing files "atomically". Atomic writes are writes that materialize after the successful completion of an explicit `Close()`operation. This is to ensure that the target file is not left in an incomplete state. This is useful for a wide variety of applications like file editors and more. This is implemented by using a `move` or `rename` filesystem operation, which in most filesystems is atomic. Additionally, this CL implements the option to retain the original file data when writing atomically. Bug: 968550 Change-Id: Ia61372e6f52378730388987f5c0749f8ec15f03b Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1692325Reviewed-by:
Daniel Cheng <dcheng@chromium.org> Reviewed-by:
Marijn Kruisselbrink <mek@chromium.org> Commit-Queue: Olivier Yiptong <oyiptong@chromium.org> Cr-Commit-Position: refs/heads/master@{#682510}
-