- 13 May, 2018 5 commits
-
-
chrome-release-bot authored
# This is an automated release commit. # Do not revert without consulting chrome-pmo@google.com. NOAUTOREVERT=true TBR=mmoss@chromium.org Change-Id: I73e84b64b6842ff34c0f13530c9a21d7988ed8fe Reviewed-on: https://chromium-review.googlesource.com/1056021Reviewed-by:
chrome-release-bot@chromium.org <chrome-release-bot@chromium.org> Cr-Commit-Position: refs/heads/master@{#558151}
-
Dominic Farolino authored
Initial infrastructure changes to support Priority Hints. This includes adding a FetchImportanceMode enum to fetch_api_request.mojom, some supporting getters/setters to FetchParameters as well as ResourceRequest, and priority-modifying logic to ResourceFetcher. R=kinuko@chromium.org, yhirano@chromium.org, yoav@yoav.ws Bug: 821464 Change-Id: I5ab7f70f44e80d01b7d47ad4c00ae3f4817815ee Reviewed-on: https://chromium-review.googlesource.com/1053221 Commit-Queue: Dominic Farolino <domfarolino@gmail.com> Reviewed-by:
Kinuko Yasuda <kinuko@chromium.org> Reviewed-by:
Yoav Weiss <yoav@yoav.ws> Reviewed-by:
Yutaka Hirano <yhirano@chromium.org> Cr-Commit-Position: refs/heads/master@{#558150}
-
Jinsuk Kim authored
ByteArrayGenerator, CipherFactory are used by Chrome only, and don't have to be in content layer. This CL moves it to chrome directory, and deletes the corresponding DEPS exception rules. Bug: 617324 Change-Id: Iee5d8effe2148ef3e50375e8848048a58a0cc046 Reviewed-on: https://chromium-review.googlesource.com/1049769 Commit-Queue: Jinsuk Kim <jinsukkim@chromium.org> Reviewed-by:
Chris Palmer <palmer@chromium.org> Reviewed-by:
Ted Choc <tedchoc@chromium.org> Reviewed-by:
Bo <boliu@chromium.org> Cr-Commit-Position: refs/heads/master@{#558149}
-
skia-chromium-autoroll authored
https://skia.googlesource.com/skia.git/+log/96b0b46f2334..ec48812c5a54 $ git log 96b0b46f2..ec48812c5 --date=short --no-merges --format='%ad %ae %s' 2018-05-12 angle-skia-autoroll Roll third_party/externals/angle2/ 18e323ab2..66aafcb46 (1 commit) 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;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=egdaniel@chromium.org Change-Id: Icc36391577d1663b3802986e38d96850f9a14e87 Reviewed-on: https://chromium-review.googlesource.com/1056063 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@{#558148}
-
Dmitry Gozman authored
This is a reland of 500de904 Original change's description: > Merge WebClipboardImpl and Pasteboard into Clipboard > > Bug: 775830 > Cq-Include-Trybots: master.tryserver.blink:linux_trusty_blink_rel;master.tryserver.chromium.linux:linux_layout_tests_slimming_paint_v2 > Change-Id: I3da04418bac37e0e03b63df4a8d2407296948e77 > Reviewed-on: https://chromium-review.googlesource.com/1045492 > Commit-Queue: Dmitry Gozman <dgozman@chromium.org> > Reviewed-by: Kentaro Hara <haraken@chromium.org> > Reviewed-by: Daniel Cheng <dcheng@chromium.org> > Cr-Commit-Position: refs/heads/master@{#557739} Bug: 775830 Change-Id: I921c9c3becf2acf3b169049656e898be6e4b8e1e Cq-Include-Trybots: master.tryserver.blink:linux_trusty_blink_rel;master.tryserver.chromium.linux:linux_layout_tests_slimming_paint_v2 Reviewed-on: https://chromium-review.googlesource.com/1056427 Commit-Queue: Dmitry Gozman <dgozman@chromium.org> Reviewed-by:
Kentaro Hara <haraken@chromium.org> Cr-Commit-Position: refs/heads/master@{#558147}
-
- 12 May, 2018 35 commits
-
-
Max Moroz authored
The test has been removed in https://chromium-review.googlesource.com/c/chromium/src/+/1053765 Bug: 789981 TBR=inferno@chromium.org,liaoyuke@chromium.org Change-Id: Ic27246e0e839557f5dab5a7238c166b16ab1dba2 Reviewed-on: https://chromium-review.googlesource.com/1056065 Commit-Queue: Max Moroz <mmoroz@chromium.org> Reviewed-by:
Max Moroz <mmoroz@chromium.org> Cr-Commit-Position: refs/heads/master@{#558146}
-
https://chromium.googlesource.com/angle/angle.git/+log/18e323ab2cff..66aafcb4641c $ git log 18e323ab2..66aafcb46 --date=short --no-merges --format='%ad %ae %s' 2018-05-12 fjhenigman Revert "GLES1: Renderer (minimal)" Created with: roll-dep src/third_party/angle 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=fjhenigman@chromium.org Change-Id: Ie5ec39db3634c878528158de50d635d80c62bd91 Reviewed-on: https://chromium-review.googlesource.com/1056062Reviewed-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@{#558145}
-
Wez authored
Bug: 838386 Change-Id: I614e823499c927704ed95d76103bf1e1c2c216f0 Reviewed-on: https://chromium-review.googlesource.com/1036445 Commit-Queue: Wez <wez@chromium.org> Reviewed-by:
Daniel Murphy <dmurph@chromium.org> Cr-Commit-Position: refs/heads/master@{#558144}
-
Chris Harrelson authored
This is behind a new runtime-enabled flag called IntersectionObserverGeometryMapper. Bug:831762 Change-Id: I022e988d3abee45e767c04487e8e9662bfc109f2 Reviewed-on: https://chromium-review.googlesource.com/1055783 Commit-Queue: Stefan Zager <szager@chromium.org> Reviewed-by:
Stefan Zager <szager@chromium.org> Cr-Commit-Position: refs/heads/master@{#558143}
-
Daniele Castagna authored
crbug.com/789292 has been fixed, we can get rid of the log used to debug it. Additionally, the DCHECK on AddFramebuffer2 has been removed since it can fail for buffers imported via exo (crbug.com/779356) Bug: 779356, 789292 Change-Id: If25ed181dbbaabe8f1b3297c2cad3edf2bc689f5 Reviewed-on: https://chromium-review.googlesource.com/1056117Reviewed-by:
David Reveman <reveman@chromium.org> Commit-Queue: Daniele Castagna <dcastagna@chromium.org> Cr-Commit-Position: refs/heads/master@{#558142}
-
fuchsia-sdk-chromium-autoroll authored
The AutoRoll server is located here: https://fuchsia-sdk-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:fuchsia_arm64_cast_audio;luci.chromium.try:fuchsia_x64_cast_audio TBR=cr-fuchsia+bot@chromium.org Change-Id: I4b682f120ac9aef13f63a1cb172e27fa676a4c34 Reviewed-on: https://chromium-review.googlesource.com/1056061Reviewed-by:
Fuchsia SDK Autoroller <fuchsia-sdk-chromium-autoroll@skia-buildbots.google.com.iam.gserviceaccount.com> Commit-Queue: Fuchsia SDK Autoroller <fuchsia-sdk-chromium-autoroll@skia-buildbots.google.com.iam.gserviceaccount.com> Cr-Commit-Position: refs/heads/master@{#558141}
-
Daniele Castagna authored
AddFramebuffer2 can fail for buffers coming via exo. We should not use an invalid fb when pageflipping in those cases otherwise the GPU process will crash. Bug: 779356 Test: wayland_simple_client with crrev.com/c/1056018 Change-Id: I179ce2ddbe1fb5080f9d00989ccf96ddf3f94d68 Reviewed-on: https://chromium-review.googlesource.com/1056017Reviewed-by:
David Reveman <reveman@chromium.org> Commit-Queue: Daniele Castagna <dcastagna@chromium.org> Cr-Commit-Position: refs/heads/master@{#558140}
-
Nicolas Pena authored
This CL does the following: * Use TimeTicks instead of double in WebURLLoaderClient::DidFinishLoading and propagate. * Remove unused double from ThreadableLoaderClient::DidFinishLoading and from WebAssociatedURLLoaderClient::DidFinishLoading. Change-Id: If1eed191efe6c39d376f6d7c1a636bc9dcb79251 Reviewed-on: https://chromium-review.googlesource.com/1048137Reviewed-by:
Dmitry Gozman <dgozman@chromium.org> Reviewed-by:
Dale Curtis <dalecurtis@chromium.org> Reviewed-by:
Kentaro Hara <haraken@chromium.org> Reviewed-by:
Fady Samuel <fsamuel@chromium.org> Reviewed-by:
Bill Budge <bbudge@chromium.org> Commit-Queue: Nicolás Peña Moreno <npm@chromium.org> Cr-Commit-Position: refs/heads/master@{#558139}
-
Alexey Kozyatinskiy authored
With V8 side CL [1] we will start using proper break location instead of stack check location almost all the time, e.g. for async step-into and for a lot of DOM breakpoints. [1] https://chromium-review.googlesource.com/c/v8/v8/+/1054155 R=dgozman@chromium.org Bug: v8:7753 Change-Id: I550d4d43494df7abe8c5fcd0fbb1d08ca88364f6 Reviewed-on: https://chromium-review.googlesource.com/1055791Reviewed-by:
Dmitry Gozman <dgozman@chromium.org> Commit-Queue: Aleksey Kozyatinskiy <kozyatinskiy@chromium.org> Cr-Commit-Position: refs/heads/master@{#558138}
-
Lei Zhang authored
Currently, it only synthesizes a key char event, which is insufficient for the PDF plugin to produce the proper response. With the additional key up event, the PDF plugin will react correctly to keep the selected text state synchronized between the plugin and the renderer. BUG=841196 Change-Id: I331562d2f39facc9affdeac98d85306c09d56130 Reviewed-on: https://chromium-review.googlesource.com/1053164Reviewed-by:
Bill Budge <bbudge@chromium.org> Commit-Queue: Lei Zhang <thestig@chromium.org> Cr-Commit-Position: refs/heads/master@{#558137}
-
Alexandros Frantzis authored
Add a parameter for an unowned GpuFence to the NativePixmap::ScheduleOverlayPlane() method. The parameter is unused at the moment, but will be used in upcoming commits to wire the overlay plane GpuFences in Ozone-DRM. Bug: 828393 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: I4697764c88a307b307af0ca5c1305e2d83cceec4 Reviewed-on: https://chromium-review.googlesource.com/1021737 Commit-Queue: Daniele Castagna <dcastagna@chromium.org> Reviewed-by:
David Reveman <reveman@chromium.org> Reviewed-by:
Sadrul Chowdhury <sadrul@chromium.org> Reviewed-by:
Antoine Labour <piman@chromium.org> Cr-Commit-Position: refs/heads/master@{#558136}
-
Chromium WPT Sync authored
Using wpt-import in Chromium cfd0d899. Build: https://ci.chromium.org/buildbot/chromium.infra.cron/wpt-importer/17392 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=qyearsley No-Export: true Change-Id: I0da577389513a1182cb6ec57555845e300c60d94 Reviewed-on: https://chromium-review.googlesource.com/1056016 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@{#558135}
-
Erik Luo authored
This CL dismantles the side effect constructor blacklist, so Blink constructors (default, named, custom) all throw when evaluating with throwOnSideEffect. [Affects=Everything] is no longer a valid extended attribute. Constructors can still be invoked without side effects in one case: creating a DOM wrapper for the V8 object. Any whitelisted attribute/operation that returns an HTMLDivElement, for example, can call the constructor with ConstructorMode::kWrapExistingObject the first time it returns. Bug: 829571 Change-Id: I3ed2166da80d38a7835dc57a1ccaec23e22734e9 Reviewed-on: https://chromium-review.googlesource.com/1043655Reviewed-by:
Yuki Shiino <yukishiino@chromium.org> Reviewed-by:
Kentaro Hara <haraken@chromium.org> Commit-Queue: Erik Luo <luoe@chromium.org> Cr-Commit-Position: refs/heads/master@{#558134}
-
Stefan Zager authored
The viewport intersection shouldn't be updated for a force layout or other non-BeginMainFrame lifecycle update. BUG=842368 R=kenrb@chromium.org Change-Id: Ifd656b495e60e8b1413df8126c7b6ff95933abaf Reviewed-on: https://chromium-review.googlesource.com/1056491Reviewed-by:
Ken Buchanan <kenrb@chromium.org> Commit-Queue: Stefan Zager <szager@chromium.org> Cr-Commit-Position: refs/heads/master@{#558133}
-
Reid Kleckner authored
This reverts commit 646faed8. Reason for revert: These tests fail in official build configurations: https://uberchromegw.corp.google.com/i/official.desktop.continuous/builders/win%20trunk/builds/84233 Original change's description: > Check that Channel ID is in use before providing Channel IDs to extensions > > Change-Id: Iabe9977651ec3e38f43d2495e790af508a43e0e5 > Reviewed-on: https://chromium-review.googlesource.com/1055676 > Commit-Queue: Nick Harper <nharper@chromium.org> > Reviewed-by: Devlin <rdevlin.cronin@chromium.org> > Cr-Commit-Position: refs/heads/master@{#558096} TBR=rdevlin.cronin@chromium.org,nharper@chromium.org Change-Id: I4f0f2b194a709dd8b1a6cd690de60e24dacb7653 No-Presubmit: true No-Tree-Checks: true No-Try: true Reviewed-on: https://chromium-review.googlesource.com/1056567Reviewed-by:
Reid Kleckner <rnk@chromium.org> Commit-Queue: Reid Kleckner <rnk@chromium.org> Cr-Commit-Position: refs/heads/master@{#558132}
-
erikchen authored
Change-Id: Icbfecced7e68f62c7f9eca0623ddf35a9a241c24 Reviewed-on: https://chromium-review.googlesource.com/1053960Reviewed-by:
Siddhartha S <ssid@chromium.org> Commit-Queue: Erik Chen <erikchen@chromium.org> Cr-Commit-Position: refs/heads/master@{#558131}
-
Antonio Gomes authored
Manual test: <out>/chrome --extensions-install-verification="enforce_strict" TEST=covered by existing tests. BUG=773295 Change-Id: I116d5c826935c6b5c6c9d062cf55422b7b1915f6 Reviewed-on: https://chromium-review.googlesource.com/1052867 Commit-Queue: Antonio Gomes <tonikitoo@igalia.com> Reviewed-by:
Devlin <rdevlin.cronin@chromium.org> Reviewed-by:
Matt Menke <mmenke@chromium.org> Cr-Commit-Position: refs/heads/master@{#558130}
-
spqchan authored
Bug: 838303 Change-Id: Iba9ae89b36f711aec3e00b61fafd4c2e2672cc0d Reviewed-on: https://chromium-review.googlesource.com/1056403 Commit-Queue: Mathieu Perreault <mathp@chromium.org> Reviewed-by:
Mathieu Perreault <mathp@chromium.org> Cr-Commit-Position: refs/heads/master@{#558129}
-
https://chromium.googlesource.com/chromiumos/chromite.git/+log/50c42dd35113..193c9144b170 $ git log 50c42dd35..193c9144b --date=short --no-merges --format='%ad %ae %s' 2018-05-09 jwerner fwgdb: Remove unified_depthcharge, make reboot flow more permissive Created with: roll-dep src/third_party/chromite The AutoRoll server is located here: https://chromite-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. TBR=chrome-os-gardeners@chromium.org Change-Id: I8ddedcaf94e27807e0e58e87bfba9770679883ea Reviewed-on: https://chromium-review.googlesource.com/1056477Reviewed-by:
Chromite Chromium Autoroll <chromite-chromium-autoroll@skia-buildbots.google.com.iam.gserviceaccount.com> Commit-Queue: Chromite Chromium Autoroll <chromite-chromium-autoroll@skia-buildbots.google.com.iam.gserviceaccount.com> Cr-Commit-Position: refs/heads/master@{#558128}
-
catapult-chromium-autoroll authored
https://chromium.googlesource.com/catapult.git/+log/1d2e7600b0b4..b424d15a6070 $ git log 1d2e7600b..b424d15a6 --date=short --no-merges --format='%ad %ae %s' 2018-05-11 sammiequon Disable cache_temperature tests on chromeos. 2018-05-11 sullivan Update bug assigns to autorollers to assign to sheriff. 2018-05-11 simonhatch Dashboard - Remove some old logging from graph_data. 2018-05-11 dtu [pinpoint] Don't CC reviewers on bug comments. 2018-05-10 simonhatch Dashboard - Add some logging around /add_histograms. 2018-05-11 simonhatch Pinpoint - Update results2 generator for job format. Created with: roll-dep src/third_party/catapult BUG=chromium:830667,chromium:842248 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. TBR=sullivan@chromium.org Change-Id: I6cfe1c7f4f7b7be60b505b4cae00b27cef8a66fc Reviewed-on: https://chromium-review.googlesource.com/1056471Reviewed-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@{#558127}
-
https://pdfium.googlesource.com/pdfium.git/+log/47a4f418a86b..82cb80962bc6 $ git log 47a4f418a..82cb80962 --date=short --no-merges --format='%ad %ae %s' 2018-05-12 hnakashima Prepare image comparison in safetynet_compare.py to work in nightly run. Created with: roll-dep src/third_party/pdfium 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: I68419bab49c625e7fc1f68b0465aae961052d91b Reviewed-on: https://chromium-review.googlesource.com/1056493Reviewed-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@{#558126}
-
https://chromium.googlesource.com/chromiumos/chromite.git/+log/348b95b14ab7..50c42dd35113 $ git log 348b95b14..50c42dd35 --date=short --no-merges --format='%ad %ae %s' 2018-05-11 khmel arc: Fix ACL for archive with SDK tools 2018-05-09 yunlian enable thinlto on elm and veryon_jaq release builders. Created with: roll-dep src/third_party/chromite BUG=chromium:827217 The AutoRoll server is located here: https://chromite-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. TBR=chrome-os-gardeners@chromium.org Change-Id: Id75b97be2c132d6fcdfa128b050efc5d1d55fc35 Reviewed-on: https://chromium-review.googlesource.com/1056474Reviewed-by:
Chromite Chromium Autoroll <chromite-chromium-autoroll@skia-buildbots.google.com.iam.gserviceaccount.com> Commit-Queue: Chromite Chromium Autoroll <chromite-chromium-autoroll@skia-buildbots.google.com.iam.gserviceaccount.com> Cr-Commit-Position: refs/heads/master@{#558125}
-
skia-chromium-autoroll authored
https://skia.googlesource.com/skia.git/+log/3202ac4d24de..96b0b46f2334 $ git log 3202ac4d2..96b0b46f2 --date=short --no-merges --format='%ad %ae %s' 2018-05-12 angle-skia-autoroll Roll third_party/externals/angle2/ a69099212..18e323ab2 (3 commits) 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;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=egdaniel@chromium.org Change-Id: I6cf28500f14c483bdf28699ac6d50f1dae0c5181 Reviewed-on: https://chromium-review.googlesource.com/1056473Reviewed-by:
skia-chromium-autoroll <skia-chromium-autoroll@skia-buildbots.google.com.iam.gserviceaccount.com> Commit-Queue: skia-chromium-autoroll <skia-chromium-autoroll@skia-buildbots.google.com.iam.gserviceaccount.com> Cr-Commit-Position: refs/heads/master@{#558124}
-
Noel Gordon authored
Code is using std::move, defined in <utility>. Maybe clang-tidy has a feature for this, but anyho. Tbr: slangley Bug: 841659 Change-Id: I01d81203a77e96bc0115980eda43a442944abb31 Reviewed-on: https://chromium-review.googlesource.com/1056508Reviewed-by:
Noel Gordon <noel@chromium.org> Commit-Queue: Noel Gordon <noel@chromium.org> Cr-Commit-Position: refs/heads/master@{#558123}
-
Alexei Filippov authored
Profiler functions used to lock the mutex first and then update the entered_ TLS variable. That caused the following sequence leading to a deadlock: 1. In DoRecordAlloc it locks the mutex and then calls entered_.Set(true) 2. If this happens to be the very first access to TLS, it causes internal TLS vector to be allocated. 3. It then reenters DoRecordAlloc, the entered_.Get() is still false since #1 is not yet finished. 4. Thus it proceed to locking the mutex again. The fix is basically swapping the order of mutex locking and entered_.Set(true). BUG=842396 Change-Id: I926c05755321c082215ca84bfc312348c89ecdca Reviewed-on: https://chromium-review.googlesource.com/1056412 Commit-Queue: Alexei Filippov <alph@chromium.org> Reviewed-by:
Dmitry Gozman <dgozman@chromium.org> Cr-Commit-Position: refs/heads/master@{#558122}
-
chromeos-commit-bot@chromium.org authored
BUG=762641 TBR=sammiequon@google.com Change-Id: Ie84183c812ac70c3f3c64de3340c11d37c25e04a Reviewed-on: https://chromium-review.googlesource.com/1056116 Commit-Queue: ChromeOS Commit Bot <chromeos-commit-bot@chromium.org> Reviewed-by:
ChromeOS Commit Bot <chromeos-commit-bot@chromium.org> Cr-Commit-Position: refs/heads/master@{#558121}
-
Jose Dapena Paz authored
AXNode accessors to AXNodeData have been added and declared constexpr. But the methods in AXNodeData are not constexpr (and they cannot be such). GCC complains about that. So remove the constexpr declarations accordingly. Change-Id: Ifaaaa44921e85076031f0d73a9260e074efbef07 Reviewed-on: https://chromium-review.googlesource.com/1053778 Commit-Queue: Dominic Mazzoni <dmazzoni@chromium.org> Reviewed-by:
Dominic Mazzoni <dmazzoni@chromium.org> Cr-Commit-Position: refs/heads/master@{#558120}
-
Chris Nardi authored
In https://drafts.csswg.org/css-values-4/#resolution, 'x' is defined as a synonym for dppx. Add support for parsing 'x' as dots per px. Intent to Implement and Ship: https://groups.google.com/a/chromium.org/d/topic/blink-dev/FsD97U77tss Bug: 630597, 807653 Change-Id: I70c440c89e42cd005a9c58fd89745a5603e0dee6 Reviewed-on: https://chromium-review.googlesource.com/892467 Commit-Queue: Chris Nardi <cnardi@chromium.org> Reviewed-by:
Emil A Eklund <eae@chromium.org> Cr-Commit-Position: refs/heads/master@{#558119}
-
afdo-chromium-autoroll authored
This CL may cause a small binary size increase, roughly proportional to how long it's been since our last AFDO profile roll. For larger increases (around or exceeding 100KB), please file a bug against gbiv@chromium.org. Additional context: https://crbug.com/805539 The AutoRoll server is located here: https://afdo-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. TBR=gbiv@chromium.org Change-Id: Ie205999387e07ec7632a0c3a0f1bb6f31ebc9095 Reviewed-on: https://chromium-review.googlesource.com/1056492 Commit-Queue: afdo-chromium-autoroll <afdo-chromium-autoroll@skia-buildbots.google.com.iam.gserviceaccount.com> Reviewed-by:
afdo-chromium-autoroll <afdo-chromium-autoroll@skia-buildbots.google.com.iam.gserviceaccount.com> Cr-Commit-Position: refs/heads/master@{#558118}
-
https://pdfium.googlesource.com/pdfium.git/+log/626a093f5fde..47a4f418a86b $ git log 626a093f5..47a4f418a --date=short --no-merges --format='%ad %ae %s' 2018-05-12 hnakashima Remove unused CFWL_FormProxy. Created with: roll-dep src/third_party/pdfium 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: I31a2625cc0b33758c1d51b5cfa2e2633dd830cb3 Reviewed-on: https://chromium-review.googlesource.com/1056470Reviewed-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@{#558117}
-
Noel Gordon authored
Enabled in RELEASE, now re-enable this test in DEBUG/MSAN/ASAN. Tbr: fukino Bug: 836254 Change-Id: Ife1c1ccc84b3d25f469da93d69bd334145b8f4b8 Reviewed-on: https://chromium-review.googlesource.com/1055370 Commit-Queue: Noel Gordon <noel@chromium.org> Reviewed-by:
Noel Gordon <noel@chromium.org> Cr-Commit-Position: refs/heads/master@{#558116}
-
Noel Gordon authored
Enabled in RELEASE, now re-enable this test in DEBUG/MSAN/ASAN. Tbr: fukino Bug: 836254 Change-Id: I7b6553cf460ee330e50b8dc8a4fa666369affeab Reviewed-on: https://chromium-review.googlesource.com/1055036Reviewed-by:
Noel Gordon <noel@chromium.org> Commit-Queue: Noel Gordon <noel@chromium.org> Cr-Commit-Position: refs/heads/master@{#558115}
-
sczs authored
- Deletes BookmarkContextBar files and usage from BookmarkHomeVC. - Uses BookmarkHomeVC navigationController UIToolbar instead. - Creates accessibilityIdentifiers for toolbar buttons. - Changes the BookmarkTableCell margins since these were giving the effect of shrinking row height. Screenshots Old: https://drive.google.com/open?id=1v3GQRiiPiIzJNZ7Lxd-pgpaaqP15b0PR New: https://drive.google.com/open?id=1z5mOT3p1NBj62EaMKfR2P5WSD2irQmyX Bug: 839435 Cq-Include-Trybots: master.tryserver.chromium.mac:ios-simulator-cronet;master.tryserver.chromium.mac:ios-simulator-full-configs Change-Id: I155365b6f9520010153ce97187ef1f6a2a8514f7 Reviewed-on: https://chromium-review.googlesource.com/1054123 Commit-Queue: Sergio Collazos <sczs@chromium.org> Reviewed-by:
Rohit Rao <rohitrao@chromium.org> Cr-Commit-Position: refs/heads/master@{#558114}
-
David Jacobo authored
Automatically close tabs after the user is authenticated and redirected to an ARC app. ArcWebContentsData can be removed via ArcExternalProtocolDialog as well as in AppsNavigationThrottle, with this we try to reduce the scope of the flag as much as possible while still allowing the user to bypass the UI and automatically close tabs left behind in the process. Bug: 838294 Test: Build, unit_tests still works Change-Id: Ifb812a3fab2fe7f49f07b4fbaee754ac96ea158f Reviewed-on: https://chromium-review.googlesource.com/1026870 Commit-Queue: David Jacobo <djacobo@chromium.org> Reviewed-by:
Yusuke Sato <yusukes@chromium.org> Cr-Commit-Position: refs/heads/master@{#558113}
-
Han Leon authored
Some code pathes aquire a service worker object info but do not send it immediately, the service worker object info holds a SWObjectAssociatedRequest whose corresponding SWObjectAssociatedPtr is held by a SWHandle, but during the period the SWHandle may call StateChanged() on this SWObjectAssociatedPtr, because the SWObjectAssociatedRequest has not been sent over Mojo yet, this will lead to a crash. This CL fixes such scenarios by enabling those code pathes to create/handle the (SWObjectAssociatedPtrInfo,SWObjectAssociatedRequest) pair on their own terms whenever they need to send the object info over Mojo, they will send SWObjectAssociatedRequest over Mojo first and then pass SWObjectAssociatedPtrInfo back to SWHandle to make calls safely there. Details: SWHandle::CreateIncompleteObjectInfo() does not bind the SWObjectAssociatedPtr and just returns an object info without a valid SWObjectAssociatedRequest. Callers will create (SWObjectAssociatedPtrInfo,SWObjectAssociatedRequest) by themselves later at any time when they want to send the object info over Mojo, after the SWObjectAssociatedRequest was sent they can call SWHandle::AddRemoteObjectPtr() to pass the SWObjectAssociatedPtrInfo back to SWHandle. BUG=838397 Change-Id: If2ed45cef1fe2a25bb9760a66a1ecc4c432ad8d3 Reviewed-on: https://chromium-review.googlesource.com/1036498 Commit-Queue: Han Leon <leon.han@intel.com> Reviewed-by:
Kinuko Yasuda <kinuko@chromium.org> Reviewed-by:
Matt Falkenhagen <falken@chromium.org> Reviewed-by:
Makoto Shimazu <shimazu@chromium.org> Cr-Commit-Position: refs/heads/master@{#558112}
-