- 24 Sep, 2019 40 commits
-
-
Eugene But authored
This change is made because all subdirectories of ios/web/public represent features, while web_state subdirectory is class name. The move proposal was discussed here: https://docs.google.com/document/d/1C2YIU0rDbH5wz2VI6nk0COrktwY3DbF-e7QbMIAKEJc/edit Bug: 942244 Change-Id: Ia3c5cf1a6e0a0a58ba3f549e3619a6ada707d1a6 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1819355 Commit-Queue: Yi Su <mrsuyi@chromium.org> Auto-Submit: Eugene But <eugenebut@chromium.org> Reviewed-by:
Yi Su <mrsuyi@chromium.org> Cr-Commit-Position: refs/heads/master@{#699232}
-
Ken Rockot authored
This is a reland of 749dcc3b PS1->PS4 shows the exclusive delta from the original patch, fixing a subtle lifetime issue in FileService. This turns out to have always been wrong: LevelDBServiceImpl can be destroyed before its created LevelDBDatabaseImpl objects have received all messages, and the service impl owns the shared message pipe for the associated database interfaces; thus messages can be dropped and data lost. This never mattered before because FileService instances effectively lived forever anyway. The fix here makes LevelDBServiceImpl lifetime independent of FileService lifetime, tying it instead to client lifetime. Original change's description: > Move File Service off Service Manager > > This is a first step towards eliminating the File Service and migrating > DOM Storage code into the Storage Service. > > File Service and its only clients (DOM storage objects) are moved off of > Service Manager APIs. LocalStorageContextMojo and > SessionStorageContextMojo are changed to own their own FileService > instance, now constructed directly over a directory path. > > No functional changes intended in this CL. > > Bug: 994911,977637 > Change-Id: I2ffd0bef670304a0dc3c103981139603521a14ac > Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1778801 > Commit-Queue: Ken Rockot <rockot@google.com> > Reviewed-by: Avi Drissman <avi@chromium.org> > Reviewed-by: Robert Sesek <rsesek@chromium.org> > Reviewed-by: Marijn Kruisselbrink <mek@chromium.org> > Cr-Commit-Position: refs/heads/master@{#698487} NOPRESUBMIT=true Bug: 994911, 977637 Change-Id: I12658247e428f2e5eee32a2e3eabe3eb35381238 Tbr: avi@chromium.org Tbr: mek@chromium.org Tbr: rsesek@chromium.org Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1816582 Commit-Queue: Ken Rockot <rockot@google.com> Reviewed-by:
Ken Rockot <rockot@google.com> Cr-Commit-Position: refs/heads/master@{#699231}
-
Mario Sanchez Prada authored
Convert both the implementation and clients in the browser and renderer processes for the media_router.mojom.MediaController interface, and adapt unit tests. Bug: 955171 Change-Id: I59ab6af047fe6b87ae470e0379f7f607e12a427c Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1810998Reviewed-by:
Brandon Tolsch <btolsch@chromium.org> Reviewed-by:
Oksana Zhuravlova <oksamyt@chromium.org> Reviewed-by:
Dominick Ng <dominickn@chromium.org> Commit-Queue: Mario Sanchez Prada <mario@igalia.com> Cr-Commit-Position: refs/heads/master@{#699230}
-
Tomasz Moniuszko authored
Bug: 1006749 Change-Id: I74334514acd6fd195e6cc435e37e5b46731ea4a2 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1819306Reviewed-by:
Hector Carmona <hcarmona@chromium.org> Commit-Queue: Tomasz Moniuszko <tmoniuszko@opera.com> Cr-Commit-Position: refs/heads/master@{#699229}
-
Alan Cutter authored
Mark virtual/omt-service-worker-startup/external/wpt/service-workers/service-worker/update-bytecheck.https.html as timing out https://test-results.appspot.com/dashboards/flakiness_dashboard.html#testType=webkit_layout_tests&tests=virtual%2Fomt-service-worker-startup%2Fexternal%2Fwpt%2Fservice-workers%2Fservice-worker%2Fupdate-bytecheck.https.html This test recently started timing out frequently on Windows 7. TBR=knollr@chromium.org Bug: 1002377 Change-Id: I5c54b1c0b683b214f11beaddd2259508dd71ba8c Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1820405Reviewed-by:
Alan Cutter <alancutter@chromium.org> Commit-Queue: Alan Cutter <alancutter@chromium.org> Cr-Commit-Position: refs/heads/master@{#699228}
-
chromium-internal-autoroll authored
https://chrome-internal.googlesource.com/chrome/src-internal.git/+log/e712395b0e52..ca2ef4313b40 Created with: gclient setdep -r src-internal@ca2ef4313b40 If this roll has caused a breakage, revert this CL and stop the roller using the controls here: https://skia-autoroll.corp.goog/r/src-internal-chromium-autoroll Please CC jbudorick@google.com on the revert to ensure that a human is aware of the problem. To report a problem with the AutoRoller itself, please file a bug: https://bugs.chromium.org/p/skia/issues/entry?template=Autoroller+Bug Documentation for the AutoRoller is here: https://skia.googlesource.com/buildbot/+/master/autoroll/README.md CQ_INCLUDE_TRYBOTS=luci.chrome.try:linux-chromeos-chrome TBR=jbudorick@google.com Bug: chromium:None Change-Id: I6d62654e2e88426b048b9ccb323a9906b3bcb9d0 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1819854Reviewed-by:
chromium-internal-autoroll <chromium-internal-autoroll@skia-corp.google.com.iam.gserviceaccount.com> Commit-Queue: chromium-internal-autoroll <chromium-internal-autoroll@skia-corp.google.com.iam.gserviceaccount.com> Cr-Commit-Position: refs/heads/master@{#699227}
-
Yuta Kitamura authored
Originally WindowAgent's constructor's comment suggested it should not be used, but nevertheless Document was using it. This CL updates the comment and adds information about when it should be used. Change-Id: Ia960af496bdea3fe7dbb0e07dc1f6a97cf64b760 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1809035Reviewed-by:
Dave Tapuska <dtapuska@chromium.org> Reviewed-by:
Kentaro Hara <haraken@chromium.org> Commit-Queue: Yuta Kitamura <yutak@chromium.org> Auto-Submit: Yuta Kitamura <yutak@chromium.org> Cr-Commit-Position: refs/heads/master@{#699226}
-
v8-ci-autoroll-builder authored
Summary of changes available at: https://chromium.googlesource.com/v8/v8/+log/017e8357..778af2fd 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: I8d6e0ec38b34b1e956c861e77606313aa469d0fe Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1820443Reviewed-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@{#699225}
-
Jeevan Shikaram authored
This CL corrects a regression which hides the more settings and permission linkout. Bug: 979531 Change-Id: I468734a201836b23c1d7943f139ff9090af45390 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1820401Reviewed-by:
-
John Z Wu authored
This makes it easier to output debug menus. I had to change the private property into a method because C++ things do not seem to work well with valueForKey: Change-Id: I62b08be230b968449a6a002a1c7e62a4c0026a03 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1816288Reviewed-by:
Hiroshi Ichikawa <ichikawa@chromium.org> Commit-Queue: John Wu <jzw@chromium.org> Cr-Commit-Position: refs/heads/master@{#699223}
-
Jeevan Shikaram authored
This CL deletes the Google Play Store section and adds it's children into the new Apps settings section. Before: https://bugs.chromium.org/p/chromium/issues/attachment?aid=413184&signed_aid=VDO2a7j5Bu_Via8i0H_BzA==&inline=1 After: https://bugs.chromium.org/p/chromium/issues/attachment?aid=413183&signed_aid=sQOsHwyPDGWKiaC2zZlgQQ==&inline=1 Bug: 1004583 Change-Id: If9d307de4240ab8246f2b7ebb61e38b2530f792a Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1815957 Commit-Queue: Jeevan Shikaram <jshikaram@chromium.org> Reviewed-by:
James Cook <jamescook@chromium.org> Reviewed-by:
Steven Bennetts <stevenjb@chromium.org> Cr-Commit-Position: refs/heads/master@{#699222}
-
Peter Kasting authored
Bug: 82078 Change-Id: Id6acded56ab6c6e4133a274875db985e963fcf8a Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1820539 Commit-Queue: Peter Kasting <pkasting@chromium.org> Commit-Queue: Jonathan Metzman <metzman@chromium.org> Auto-Submit: Peter Kasting <pkasting@chromium.org> Reviewed-by:
Jonathan Metzman <metzman@chromium.org> Cr-Commit-Position: refs/heads/master@{#699221}
-
François Beaufort authored
This CL adds missing attribute error to GPUUncapturedErrorEvent. It was forgotten in http://crrev.com/1787504 Bug: 877147 Change-Id: Ia135b7b79a4215c0c90a4704516c3b70f3658072 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1814827 Auto-Submit: François Beaufort <beaufort.francois@gmail.com> Reviewed-by:
Corentin Wallez <cwallez@chromium.org> Reviewed-by:
Kai Ninomiya <kainino@chromium.org> Commit-Queue: François Beaufort <beaufort.francois@gmail.com> Cr-Commit-Position: refs/heads/master@{#699220}
-
chromium-autoroll authored
https://chromium.googlesource.com/catapult.git/+log/984536bb45f4..e18bf0acc0be git log 984536bb45f4..e18bf0acc0be --date=short --no-merges --format='%ad %ae %s' 2019-09-24 dberris@chromium.org PinpointScheduler: A Validator Implementation 2019-09-24 dberris@chromium.org Pinpoint: Allow for partial job instance migration Created with: gclient setdep -r src/third_party/catapult@e18bf0acc0be If this roll has caused a breakage, revert this CL and stop the roller using the controls here: https://autoroll.skia.org/r/catapult-autoroll Please CC simonhatch@google.com on the revert to ensure that a human is aware of the problem. To report a problem with the AutoRoller itself, please file a bug: https://bugs.chromium.org/p/skia/issues/entry?template=Autoroller+Bug Documentation for the AutoRoller is here: https://skia.googlesource.com/buildbot/+/master/autoroll/README.md 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=simonhatch@google.com Bug: chromium:988912,chromium:1006658 Change-Id: I6bc3f39f8d0124e31726b702551b7a6142468a89 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1819851Reviewed-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@{#699219}
-
Mario Sanchez Prada authored
This CL migrates code to the new BrowserInterfaceBroker class, allowing to retrieve the remote implementation from the renderer process without relying on FrameClient's InterfaceProvider. Additionally, this CL also migrates the few usages of the old Mojo types that were still around in the codebase. TBR=clamy@chromium.org Bug: 955171, 978694, 936482 Change-Id: I95d2d9ef39838a09748542c847440b060190e2f5 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1819238 Commit-Queue: Mario Sanchez Prada <mario@igalia.com> Reviewed-by:
Arthur Sonzogni <arthursonzogni@chromium.org> Reviewed-by:
-
chromium-autoroll authored
https://chromium.googlesource.com/chromium/src/third_party/freetype2.git/+log/5327092bb28b..4881f75b7ca0 git log 5327092bb28b..4881f75b7ca0 --date=short --no-merges --format='%ad %ae %s' 2019-09-24 apodtele@gmail.com * src/base/ftstroke.c (ft_stroke_border_arcto): Speed up calculations. Created with: gclient setdep -r src/third_party/freetype/src@4881f75b7ca0 If this roll has caused a breakage, revert this CL and stop the roller using the controls here: https://autoroll.skia.org/r/freetype-chromium Please CC bungeman@google.com,drott@google.com,thestig@google.com on the revert to ensure that a human is aware of the problem. To report a problem with the AutoRoller itself, please file a bug: https://bugs.chromium.org/p/skia/issues/entry?template=Autoroller+Bug Documentation for the AutoRoller is here: https://skia.googlesource.com/buildbot/+/master/autoroll/README.md CQ_INCLUDE_TRYBOTS=luci.chromium.try:linux_chromium_msan_rel_ng;luci.chromium.try:linux-blink-rel;luci.chromium.try:mac10.12-blink-rel;luci.chromium.try:mac10.13_retina-blink-rel;luci.chromium.try:win10-blink-rel;luci.chromium.try:win7-blink-rel TBR=bungeman@google.com,drott@google.com,thestig@google.com Bug: None Change-Id: I010c59395b4b818f5ecd45d26cf5230010a5f727 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1816426Reviewed-by:
-
Mario Sanchez Prada authored
This CL migrates code to the new BrowserInterfaceBroker class. This allows retrieving the remote implementation from the renderer process without relying on FrameClient's InterfaceProvider. TBR=clamy@chromium.org Bug: 955171, 978694, 936482 Change-Id: I38672e3a40864aef9a179afc5463ab241491d096 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1816552 Commit-Queue: Mario Sanchez Prada <mario@igalia.com> Reviewed-by:
Eric Stevenson <estevenson@chromium.org> Reviewed-by:
-
Mario Sanchez Prada authored
Convert the remaining bits in both the implementation and clients from the browser and renderer processes for the blink::mojom::DevToolsAgent[Host] and blink::mojom::DevToolsSession[Host] interfaces. Also, rename blink::DevToolsAgent's BindRequest() methods to BindReceiver, to make them consistent with the usual pattern followed on this migration. Bug: 955171, 978694 Change-Id: Ide73670eb7d3d3fafe30f09a4d94a0e98d63b552 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1819309Reviewed-by:
-
Mario Sanchez Prada authored
Convert both the implementation and clients in the browser and renderer processes for the media_router.mojom.MediaRouterTraitsTestService interface, and adapt unit tests. Bug: 955171 Change-Id: I4856427d46032c66a0dec395cd0711e93ba93700 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1816539Reviewed-by:
-
Julie Jeongeun Kim authored
This CL converts SpellCheckHost and SpellChecker to new Mojo types using Remote, PendingRemote, PendingReceiver, Receiver, and ReceiverSet. It introduces a new template for GetInterface from LocalInterfaceProvider. Bug: 955171 Change-Id: I459d2ffc50242f8f2159c31212fd2346e571544f Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1813125 Commit-Queue: Julie Kim <jkim@igalia.com> Reviewed-by:
John Abd-El-Malek <jam@chromium.org> Reviewed-by:
Tim Volodine <timvolodine@chromium.org> Cr-Commit-Position: refs/heads/master@{#699213}
-
Julie Jeongeun Kim authored
This CL converts apps::mojom::PublisherPtr to new Mojo types using PendingRemote or Remote, PendingReceiver, ReceiverSet, and Receiver. It also updates RegisterPublisher from app_service.mojom Bug: 955171 Change-Id: Ic4001997764a9872a0df5f4931368312c263d27f Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1818886 Commit-Queue: Julie Kim <jkim@igalia.com> Reviewed-by:
-
Roman Sorokin authored
I've left it a separate polymer element to make the CL as simple as possible Before: https://screenshot.googleplex.com/O4r365FLMzJ.png After: https://screenshot.googleplex.com/2uf0mVpH5ck.png Bug: 999090 Test: manual Change-Id: If815a9b5e45334cde63b615d60d72be32f9d9021 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1819299Reviewed-by:
Denis Kuznetsov <antrim@chromium.org> Reviewed-by:
Alexander Alekseev <alemate@chromium.org> Commit-Queue: Roman Sorokin [CET] <rsorokin@chromium.org> Cr-Commit-Position: refs/heads/master@{#699211}
-
Marina Sakai authored
V8_PRIVATE_PROPERTY_GETTER_NAME: a macro-generated function can be cleaned by using the new API which was created in CL1810511. Two macros variable can be removed with this CL. Bug: 715418, 1005601 Change-Id: I1da3aa583d95700ce305c237e4a4ee4ce2aa01ee Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1813117Reviewed-by:
Hitoshi Yoshida <peria@chromium.org> Reviewed-by:
-
Jeevan Shikaram authored
This CL follows up crrev.com/c/1784238 with few naming and console error fixes. Bug: 999896 Change-Id: Ic053db94e250aebd38105ace7e895c468d3d30c7 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1798255Reviewed-by:
-
Ricky Liang authored
In crrev.com/c/1687018 we added a local call site to RegisterClient, so we can no longer assume the method is always called through Mojo IPC. enabled. Bug: 1000842 Test: Make sure Chrome debug build starts on Nocturne with ARC++ Change-Id: Ibe546396646bab01c7caec02ff72f8122ac80181 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1817875Reviewed-by:Wei Lee <wtlee@chromium.org> Reviewed-by:
Shik Chen <shik@chromium.org> Commit-Queue: Ricky Liang <jcliang@chromium.org> Cr-Commit-Position: refs/heads/master@{#699208}
-
Alex Keng authored
This change updates forced_color.css to enable <select multiple> to render high contrast color. Bug: 994219 Change-Id: If9814a11544232cfe8ee4de219f0ddaa83f70204 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1816448 Commit-Queue: Alex Keng <shihken@microsoft.com> Reviewed-by:
Kent Tamura <tkent@chromium.org> Reviewed-by:
Mason Freed <masonfreed@chromium.org> Cr-Commit-Position: refs/heads/master@{#699207}
-
chromium-autoroll authored
https://skia.googlesource.com/skia.git/+log/b4b1005d485f..c096654fa7c6 git log b4b1005d485f..c096654fa7c6 --date=short --no-merges --format='%ad %ae %s' 2019-09-24 csmartdalton@google.com ccpr: Don't assign inverse fill types to fan tessellations Created with: gclient setdep -r src/third_party/skia@c096654fa7c6 If this roll has caused a breakage, revert this CL and stop the roller using the controls here: https://autoroll.skia.org/r/skia-autoroll Please CC bungeman@google.com on the revert to ensure that a human is aware of the problem. To report a problem with the AutoRoller itself, please file a bug: https://bugs.chromium.org/p/skia/issues/entry?template=Autoroller+Bug Documentation for the AutoRoller is here: https://skia.googlesource.com/buildbot/+/master/autoroll/README.md 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=bungeman@google.com Bug: None Change-Id: Ida0d7f3e7aa6a2e18927c488f476f1e705b06bd4 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1820012Reviewed-by:
-
Xiaocheng Hu authored
The current parser allows `/ <bg-size>` as long as there's a <bg-position> in the current or previous layers, which is wrong. This patch ensures that it's allowed only after a <bg-position> in the same layer as per spec: https://drafts.csswg.org/css-backgrounds-3/#the-background Bug: 1002568 Change-Id: I79630a1331d880248d27205788a65cd5b4032a0e Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1819562 Commit-Queue: Emil A Eklund <eae@chromium.org> Reviewed-by:
Emil A Eklund <eae@chromium.org> Cr-Commit-Position: refs/heads/master@{#699205}
-
Chih-Yu Huang authored
Originally we assume that VD::CreateSurface() will be called after AVD::SetStream() is called, and the returned surface is used for the current bitstream buffer. However, the description of AVD doesn't guarantee this fact. This CL changes to set the timestamp that is mapped from bitstream id passed from SurfaceReady(). This timestamp is actually reliable value. BUG=b:140842667 TEST=run video_decode_accelerator_tests on Kevin and Eve Change-Id: I255870c6220a4bd771e1f23b54c309fe8ca42246 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1810452Reviewed-by:
Hirokazu Honda <hiroh@chromium.org> Commit-Queue: Chih-Yu Huang <akahuang@chromium.org> Cr-Commit-Position: refs/heads/master@{#699204}
-
Gyuyoung Kim authored
As a step to remove all uses of WebLocalizedString::kFooBar, this CL replaces below ones with matched IDS_FOO. - WebLocalizedString::kAXMillisecondFieldText - WebLocalizedString::kAXMinuteFieldText - WebLocalizedString::kAXMonthFieldText - WebLocalizedString::kAXSecondFieldText - WebLocalizedString::kAXWeekOfYearFieldText - WebLocalizedString::kAXYearFieldText Bug: 995644 Change-Id: I09495c8f2737a398e1b553b2b006f44e424f6686 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1820398Reviewed-by:
-
Raul Tambre authored
Initial conversion performed using '2to3 -f print .'. Imports added and duplicate parentheses removed manually. Manually converted files, comments and inline code that 2to3 missed. Presubmit disabled due to an unrelated error in find_runtime_symbols/find_runtime_symbols.py. Afterwards ran "git cl format --python" and cherry-picked the formatting changes. There are no intended behavioural changes. NOPRESUBMIT=true Bug: 941669 Change-Id: I3174ed0eb7005d493c6bc44751353b8fae18a1f1 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1818478 Commit-Queue: Raul Tambre <raul@tambre.ee> Reviewed-by:
Nico Weber <thakis@chromium.org> Auto-Submit: Raul Tambre <raul@tambre.ee> Cr-Commit-Position: refs/heads/master@{#699202}
-
Kenichi Ishibashi authored
We should terminate a service worker thread when the top-level script fetch fails. Otherwise the thread would stay running and leak a V8 isolate. WorkerGlobalScope::close() schedules thread termination. I manually confirmed that this CL fixes the leak. I'll add a test for this fix in a follow-up CL. Bug: 1005650 Change-Id: If3fd2f7f0f746c4e240b9fdebd4cf242d87a0943 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1819282 Commit-Queue: Kenichi Ishibashi <bashi@chromium.org> Reviewed-by:
Hiroki Nakagawa <nhiroki@chromium.org> Cr-Commit-Position: refs/heads/master@{#699201}
-
Wei Lee authored
This CL changes the behavior to let CCA open back camera as the default one when the device is under tablet mode. Bug: b/140273786 Test: Tested on Soraka and it open front camera as default under clamshell mode and open back camera as default under tablet mode. Change-Id: I1fa8f16265e9021ab0093cba506e0b652cb27af1 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1815927Reviewed-by:
Kuo Jen Wei <inker@chromium.org> Commit-Queue: Wei Lee <wtlee@chromium.org> Cr-Commit-Position: refs/heads/master@{#699200}
-
Alexis Savery authored
Since the host is reading/writing in power of 2 units to the device, and lifetime_writes is originally reported in KiB, the amount of writes reported in the histogram should be in GiB. Implementation CL: https://chromium-review.googlesource.com/c/chromiumos/platform2/+/1779603 Bug: None Test: ran validate_format.py Change-Id: I481c31730e66fe1f677ff52280fa8bfef97ca7c4 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1818607Reviewed-by:
Ilya Sherman <isherman@chromium.org> Commit-Queue: Ilya Sherman <isherman@chromium.org> Cr-Commit-Position: refs/heads/master@{#699199}
-
chromium-internal-autoroll authored
https://chrome-internal.googlesource.com/chrome/src-internal.git/+log/3da5033009ec..e712395b0e52 Created with: gclient setdep -r src-internal@e712395b0e52 If this roll has caused a breakage, revert this CL and stop the roller using the controls here: https://skia-autoroll.corp.goog/r/src-internal-chromium-autoroll Please CC jbudorick@google.com on the revert to ensure that a human is aware of the problem. To report a problem with the AutoRoller itself, please file a bug: https://bugs.chromium.org/p/skia/issues/entry?template=Autoroller+Bug Documentation for the AutoRoller is here: https://skia.googlesource.com/buildbot/+/master/autoroll/README.md CQ_INCLUDE_TRYBOTS=luci.chrome.try:linux-chromeos-chrome TBR=jbudorick@google.com Bug: chromium:None Change-Id: I4bd645f7b2ab42668d6044b70d3d41b1af57b21d Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1819846Reviewed-by:
-
Alan Cutter authored
This test is failing on just Windows 7 after getting deflaked. https://test-results.appspot.com/dashboards/flakiness_dashboard.html#testType=webkit_layout_tests&tests=virtual%2Fthreaded%2Fexternal%2Fwpt%2Fcss%2Fcss-scroll-snap%2Fscroll-margin.html TBR=alsan@google.com Bug: 964239 Change-Id: I9e63ab319955f70138f529585ac8ce51efd58bf8 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1820399Reviewed-by:
-
chromium-autoroll authored
https://chromium.googlesource.com/catapult.git/+log/b9650d22d16e..984536bb45f4 git log b9650d22d16e..984536bb45f4 --date=short --no-merges --format='%ad %ae %s' 2019-09-24 ntfschr@chromium.org Docs: add devil to navbar.md Created with: gclient setdep -r src/third_party/catapult@984536bb45f4 If this roll has caused a breakage, revert this CL and stop the roller using the controls here: https://autoroll.skia.org/r/catapult-autoroll Please CC simonhatch@google.com on the revert to ensure that a human is aware of the problem. To report a problem with the AutoRoller itself, please file a bug: https://bugs.chromium.org/p/skia/issues/entry?template=Autoroller+Bug Documentation for the AutoRoller is here: https://skia.googlesource.com/buildbot/+/master/autoroll/README.md 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=simonhatch@google.com Bug: None Change-Id: I1e16eb8a35d1289fa26bb0f262f8369ffe71c834 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1820294Reviewed-by:
-
Jay Harris authored
Bug: 1007063 Change-Id: Ife944e1e1784037c442e260df7d02e65454606cb Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1818890 Commit-Queue: Jay Harris <harrisjay@chromium.org> Reviewed-by:
-
Maggie Cai authored
This CL updates the App Service intent filter condition value to an array and create a intent_filter_util file to handle the creation of the condition values. Bug: 853604 Change-Id: Ib43f1efd4cde468379f25d6aaf9dd3ae54a6a142 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1816000 Commit-Queue: Maggie Cai <mxcai@chromium.org> Reviewed-by:
-
v8-ci-autoroll-builder authored
Summary of changes available at: https://chromium.googlesource.com/v8/v8/+log/d0581808..017e8357 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: I7d91dde62095dffb25df97bd6a6b656a36ebbb64 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1820307Reviewed-by:
-
