- 28 Nov, 2018 40 commits
-
-
Dave Tapuska authored
Underlying data structures don't handle more than 4GB frames to process so stick with using an unsigned value to avoid 64 bit truncation errors. BUG=879657 TBR=haraken@chromium.org Change-Id: I353a187cde9885b3642d1d2aaa4019afce40867e Reviewed-on: https://chromium-review.googlesource.com/c/1354124Reviewed-by:
Dave Tapuska <dtapuska@chromium.org> Reviewed-by:
Hongchan Choi <hongchan@chromium.org> Commit-Queue: Dave Tapuska <dtapuska@chromium.org> Cr-Commit-Position: refs/heads/master@{#611829}
-
chromium-autoroll authored
https://chromium.googlesource.com/external/github.com/KhronosGroup/SPIRV-Headers.git/+log/7cb43009d543..bbf63435c37a git log 7cb43009d543..bbf63435c37a --date=short --no-merges --format='%ad %ae %s' 2018-11-28 johnkslang@users.noreply.github.com Merge pull request #84 from grorg/whlsl 2018-11-26 johnkslang@users.noreply.github.com Merge pull request #83 from dgkoch/SPV_EXT_fragment_invocation_density Created with: gclient setdep -r src/third_party/spirv-headers/src@bbf63435c37a The AutoRoll server is located here: https://autoroll.skia.org/r/spirv-headers-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. TBR=dsinclair@chromium.org Change-Id: Ief7d43f768b17d8c5edbe70dbace7ddad1023656 Reviewed-on: https://chromium-review.googlesource.com/c/1354178Reviewed-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@{#611828}
-
Nick Harper authored
The token binding feature was removed in crrev.com/584532 (M70) - this is overdue for being removed. Bug: 875046 Change-Id: I121028692ba02602c54077401dc15eb176c8e67b Reviewed-on: https://chromium-review.googlesource.com/c/1351712Reviewed-by:
Brian White <bcwhite@chromium.org> Commit-Queue: Nick Harper <nharper@chromium.org> Cr-Commit-Position: refs/heads/master@{#611827}
-
Jonathan Metzman authored
Bug: 906214 Change-Id: Iad3053bf54046f3c0c7a59001164dac75d449582 Reviewed-on: https://chromium-review.googlesource.com/c/1352357Reviewed-by:
Ben Pastene <bpastene@chromium.org> Commit-Queue: Jonathan Metzman <metzman@chromium.org> Cr-Commit-Position: refs/heads/master@{#611826}
-
Dana Fried authored
Will be used in a followup CL to simplify making examples for a new layout manager without having to duplicate a bunch of code. Change-Id: I3a37dc1656f70015170434c9da80ccbc20c1db88 Reviewed-on: https://chromium-review.googlesource.com/c/1351968Reviewed-by:
Scott Violet <sky@chromium.org> Commit-Queue: Dana Fried <dfried@chromium.org> Cr-Commit-Position: refs/heads/master@{#611825}
-
Carlos Caballero authored
MessageLoop will go away, eventually. BUG=891670 Change-Id: Id60e6bc512e2e9d1b12088b7131fd2a74ebedc9d Reviewed-on: https://chromium-review.googlesource.com/c/1353881Reviewed-by:
Shakti Sahu <shaktisahu@chromium.org> Commit-Queue: Carlos Caballero <carlscab@google.com> Cr-Commit-Position: refs/heads/master@{#611824}
-
Ben Pastene authored
But not its CI bot (a la every other CI/CQ pair) R=dpranke, jbudorick Change-Id: Ieae0c25ca1841fa21e59fe942ef5f04354ed2297 Reviewed-on: https://chromium-review.googlesource.com/c/1352610Reviewed-by:
Steven Bennetts <stevenjb@chromium.org> Reviewed-by:
Dirk Pranke <dpranke@chromium.org> Commit-Queue: Ben Pastene <bpastene@chromium.org> Cr-Commit-Position: refs/heads/master@{#611823}
-
chromium-autoroll authored
https://skia.googlesource.com/skia.git/+log/6616efbdebd2..8061053b73e2 git log 6616efbdebd2..8061053b73e2 --date=short --no-merges --format='%ad %ae %s' 2018-11-28 borenet@google.com [infra] Flutter bot should sync to origin/skia-master 2018-11-28 reed@google.com add bench for computing convexity 2018-11-28 fmalita@chromium.org [skottie] Add a custom property manager util class 2018-11-28 skia-autoroll@skia-public.iam.gserviceaccount.com Roll third_party/externals/angle2 83b2f0e8a2e0..c4e0a1e511fc (1 commits) 2018-11-28 skia-autoroll@skia-public.iam.gserviceaccount.com Roll third_party/externals/swiftshader 67c93d206431..3b003a51bc9a (1 commits) 2018-11-28 reed@google.com Force clients to specify font when requesting strikecache Created with: gclient setdep -r src/third_party/skia@8061053b73e2 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=borenet@chromium.org Change-Id: Id88a0d085dbe59de4ec85f2af90a67a5e5a45f23 Reviewed-on: https://chromium-review.googlesource.com/c/1352867Reviewed-by:
-
Sergio Villar Senin authored
Replaced the usage of ProfileOAuth2TokenService and SigninManager by identity API calls. This CL adds a new function to the identity_test_utils.h set to set the auth error of a given account with a refresh token. Bug: 903859 Change-Id: I29e1ae3bee6ba7f69404f5f2bb7152fa0db9101d Reviewed-on: https://chromium-review.googlesource.com/c/1352250Reviewed-by:
Sylvain Defresne <sdefresne@chromium.org> Reviewed-by:
Joshua Bell <jsbell@chromium.org> Commit-Queue: Sergio Villar <svillar@igalia.com> Cr-Commit-Position: refs/heads/master@{#611821}
-
ssid authored
Adds categories useful for debugging Android traces. Change-Id: I75cc0ed406f3e3b40fd21106c1e54f7c5a4b0897 Reviewed-on: https://chromium-review.googlesource.com/c/1352229Reviewed-by:
Etienne Bergeron <etienneb@chromium.org> Commit-Queue: ssid <ssid@chromium.org> Cr-Commit-Position: refs/heads/master@{#611820}
-
Alexander Timin authored
TaskQueueImpl::SweepCancelledTasks doesn't have to know about SequenceManager. R=skyostil@chromium.org,alexclarke@chromium.org Change-Id: Ifad12c6fe652a323b776066a49aeb6e11f14e515 Reviewed-on: https://chromium-review.googlesource.com/c/1349265Reviewed-by:
Sami Kyöstilä <skyostil@chromium.org> Commit-Queue: Alexander Timin <altimin@chromium.org> Cr-Commit-Position: refs/heads/master@{#611819}
-
Hector Carmona authored
Bug: 882586 Change-Id: Id18e17c72aa3646a52a074e536fafa5cf789d85c Reviewed-on: https://chromium-review.googlesource.com/c/1343246Reviewed-by:
Scott Chen <scottchen@chromium.org> Commit-Queue: Hector Carmona <hcarmona@chromium.org> Cr-Commit-Position: refs/heads/master@{#611818}
-
David 'Digit' Turner authored
This is a re-land of [1] that has been fixed to not trigger the Clang crash when coverage instrumentation is enabled. [1] https://chromium-review.googlesource.com/c/chromium/src/+/1346305 ---------------- original commit message ---------------------- This is a small cleanup of the TraceEvent class performed which is part of a larget bug allowing cleaning up and reducing the generated machine code for TRACE_EVENTXXX() macro calls (see related bug). A first CL to perform this refactor was submitted as [1], but later reverted because it made some tests fail mysteriously (see http://crbug.com/899813). So the original CL was split into several independent ones. A first CL was submitted as [2], which actually fixed some potential dangling pointer issues that were created from the Initialize() and MoveFrom() methods. This second CL removes this methods by making TraceEvent a proper C++11 movable type, which should prevent (or at least make it more difficult) introducing invalid states for its instances. The goal is to see if this introduces new unexpected test failures (which would indicate that there are still invalid instance states used in the code base). The third CL is [3] and re-introduces the TraceArguments helper class on top of this one. [1] https://chromium-review.googlesource.com/c/chromium/src/+/1318919 [2] https://chromium-review.googlesource.com/c/chromium/src/+/1340308 [3] https://chromium-review.googlesource.com/c/chromium/src/+/1318919 ------------------ end of original commit message ------------ BUG=898794 R=primiano@chromium.org,oysteine@chromium.org,alexilin@chromium.org,chiniforooshan@chromium.org Change-Id: Icb332f91dd9724a2acae9cb1f99a580541e6cc42 Reviewed-on: https://chromium-review.googlesource.com/c/1353929Reviewed-by:
Ehsan Chiniforooshan <chiniforooshan@chromium.org> Commit-Queue: David Turner <digit@chromium.org> Cr-Commit-Position: refs/heads/master@{#611817}
-
Victor Hsieh authored
* Remove callers to SetArcAlwaysStartForTesting(true) * Change the remaining SetArcAlwaysStartForTesting(false) to SetArcAlwaysStartWithoutPlayStoreForTesting() TEST=out/Cleanup/unit_tests --gtest_filter=ChromeLauncherController*: \ Arc*:NoteTakingHelperTest* BUG=b:69960761 R=khmel@chromium.org, yusukes@chromium.org Change-Id: I0bbb0774c005db62c346fcfd40f57d897ec86877 Reviewed-on: https://chromium-review.googlesource.com/c/1352495 Commit-Queue: Victor Hsieh <victorhsieh@chromium.org> Reviewed-by:Alexander Alekseev <alemate@chromium.org> Reviewed-by:
Yusuke Sato <yusukes@chromium.org> Reviewed-by:
Yury Khmel <khmel@chromium.org> Cr-Commit-Position: refs/heads/master@{#611816}
-
Sergey Ulanov authored
Previously when starting subprocesses test launcher would clone some namespace dirs, but not all of them. Update test launcher to use fdio_ns_export_root() to get namespace of the current process so the child process can get complete namespace clone, except for /data that needs to be overridden. Change-Id: I0e49aa12a4a98e036acdf6549252478e6c771936 Reviewed-on: https://chromium-review.googlesource.com/c/1351888 Commit-Queue: Sergey Ulanov <sergeyu@chromium.org> Reviewed-by:
Wez <wez@chromium.org> Reviewed-by:
Nico Weber <thakis@chromium.org> Cr-Commit-Position: refs/heads/master@{#611815}
-
Dave Tapuska authored
SafeCast a sequence to int in bindings. Add explicit cast in heap_page.h BUG=879657 Change-Id: I1c6cf9b1748b84795ed81d903e3e19684bd6d0e3 Reviewed-on: https://chromium-review.googlesource.com/c/1354062Reviewed-by:
Kentaro Hara <haraken@chromium.org> Commit-Queue: Dave Tapuska <dtapuska@chromium.org> Cr-Commit-Position: refs/heads/master@{#611814}
-
Ken Rockot authored
This reverts commit 5fd55b4b. Reason for revert: I am fairly certain this is causing static initializer check failures on the official builder, because it changes BookmarkItem to be a non-POD type, and there are static BookmarkItem constants in the build: https://crbug.com/909828 Original change's description: > Navi: Localize Google Apps strings. > > Bug: 894499 > Change-Id: I698b51eac47cdb7af63e4bb2ace338f87c5e7493 > Reviewed-on: https://chromium-review.googlesource.com/c/1351779 > Reviewed-by: Scott Chen <scottchen@chromium.org> > Commit-Queue: Hector Carmona <hcarmona@chromium.org> > Cr-Commit-Position: refs/heads/master@{#611759} TBR=hcarmona@chromium.org,scottchen@chromium.org Change-Id: I72aff79d75c2c578c509bf4525d120c0e402cc17 No-Presubmit: true No-Tree-Checks: true No-Try: true Bug: 894499 Reviewed-on: https://chromium-review.googlesource.com/c/1352682Reviewed-by:
Ken Rockot <rockot@google.com> Commit-Queue: Ken Rockot <rockot@google.com> Cr-Commit-Position: refs/heads/master@{#611813}
-
Weidong Guo authored
Bug: 462296 Change-Id: I14fade7b9324f869192c4131d2d41d282278d85a Reviewed-on: https://chromium-review.googlesource.com/c/1352536Reviewed-by:
Alex Newcomer <newcomer@chromium.org> Commit-Queue: Weidong Guo <weidongg@chromium.org> Cr-Commit-Position: refs/heads/master@{#611812}
-
chromium-internal-autoroll authored
https://chrome-internal.googlesource.com/chrome/src-internal.git/+log/da575ec6b7fc..dbc542d5f0f3 Created with: gclient setdep -r src-internal@dbc542d5f0f3 The AutoRoll server is located here: https://autoroll-internal.skia.org/r/src-internal-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. TBR=mmoss@chromium.org Change-Id: Ic4699b49a79f91b99ff25da701ab450b90184505 Reviewed-on: https://chromium-review.googlesource.com/c/1352807Reviewed-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@{#611811}
-
Frank Kastenholz authored
This commit is an intermediate commit on the path to adding support for IETF QUIC's STREAM_ID_BLOCKED frame and MAX_STREAM_ID frame. This is an intermediate/safety commit while working on a test issue. R=rch@chromium.org Change-Id: Ie2c24114cb62b640beb31252ee4c216f08b59533 Reviewed-on: https://chromium-review.googlesource.com/c/1344197Reviewed-by:
Ryan Hamilton <rch@chromium.org> Commit-Queue: Frank Kastenholz <fkastenholz@chromium.org> Cr-Commit-Position: refs/heads/master@{#611810}
-
Robert Ogden authored
Adds flags to clear and add to the local blacklist. These flags are evaluated on startup. These flags are needed for Chrome Driver testing. Bug: 908491 Change-Id: Id67f9393d826c9e6bd267eb8ac39dc1118a4bce7 Reviewed-on: https://chromium-review.googlesource.com/c/1351880Reviewed-by:
Ryan Sturm <ryansturm@chromium.org> Commit-Queue: Robert Ogden <robertogden@chromium.org> Cr-Commit-Position: refs/heads/master@{#611809}
-
Simeon Anfinrud authored
It's possible for a new Intent with the same WebContents, but different values for mutable parameters like enableTouchInput. When this happens, the WebContents would be reinitialized, which breaks an assertion in WebContentsImpl. By storing the WebContents in a separate Controller, we allow it to persist across a reactivation of mStartParamsState where the new WebContents is equal to the previous instance. Bug: internal b/120089917 Test: cast_shell_junit_tests Change-Id: Id9d58b577a887ffff4002ec6fcde578a71752b1f Reviewed-on: https://chromium-review.googlesource.com/c/1352077 Commit-Queue: Simeon Anfinrud <sanfin@chromium.org> Reviewed-by:
Luke Halliwell <halliwell@chromium.org> Cr-Commit-Position: refs/heads/master@{#611808}
-
Charlene Yan authored
Changing InkDropGestureHandler to InkDropEventHandler to unconditionally handle mouse events in addition to gesture events for ink drops. This makes the ink drop less error prone as previously InkDropHostView depended on parents to call InkDropHostView::OnMouseEvent after handling it themselves Bug: 901183 Change-Id: Iee9c259b44d1057b2ba5d0c81b29602c9f6b77a4 Reviewed-on: https://chromium-review.googlesource.com/c/1344262 Commit-Queue: Charlene Yan <cyan@chromium.org> Reviewed-by:
Michael Wasserman <msw@chromium.org> Reviewed-by:
Peter Boström <pbos@chromium.org> Cr-Commit-Position: refs/heads/master@{#611807}
-
Ken Rockot authored
This reverts commit 65453a44. Reason for revert: broke the build: https://logs.chromium.org/logs/chromium/buildbucket/cr-buildbucket.appspot.com/8928590708886404784/+/steps/compile/0/stdout Original change's description: > [MacOS Host] Fixing a memory leak in the alert dialog > > Follow-up CL fixing a leak identified post check-in. > > Bug: 902041 > Change-Id: Ic900c107f13185fa8c823163b2278990ed543ff2 > Reviewed-on: https://chromium-review.googlesource.com/c/1352030 > Reviewed-by: Robert Sesek <rsesek@chromium.org> > Commit-Queue: Joe Downing <joedow@chromium.org> > Cr-Commit-Position: refs/heads/master@{#611777} TBR=joedow@chromium.org,rsesek@chromium.org Change-Id: I346b4a43995959ce2c289d5f2ff31feb722b4233 No-Presubmit: true No-Tree-Checks: true No-Try: true Bug: 902041 Reviewed-on: https://chromium-review.googlesource.com/c/1354231Reviewed-by:
-
Becca Hughes authored
Add grouped audio focus. This means that a media session can provide an unguessable token when requesting audio focus. This means that the media session service will treat all media sessions with the same group id as the same session when it comes to audio focus. BUG=906285 Change-Id: I184f8de58cfe3cec67db0bb5877f3b352c794f13 Reviewed-on: https://chromium-review.googlesource.com/c/1342939 Commit-Queue: Becca Hughes <beccahughes@chromium.org> Reviewed-by:
Chrome Cunningham <chcunningham@chromium.org> Reviewed-by:
Tommy Steimel <steimel@chromium.org> Reviewed-by:
Daniel Cheng <dcheng@chromium.org> Cr-Commit-Position: refs/heads/master@{#611805}
-
Pavel Shmakov authored
TabObserverRegistrar adds observers to a static ObserverList in PageLoadMetrics, and removes them when receiving callbacks about the tab: didCloseTab or tabRemoved. These callbacks don't get called if the activity is killed in background, thus a memory leak. It doesn't happen though when activity is killed via back button, the reasons for that boil down to different values of mIsClosing boolean in CustomTabActivity. But we shouldn't rely on such details of other classes and do explicit cleanup in destroy() method of each class. Change-Id: I46138b02a3cbb81a5b82e5dc6499ebbef75bb817 Reviewed-on: https://chromium-review.googlesource.com/c/1353931Reviewed-by:
Peter Conn <peconn@chromium.org> Commit-Queue: Pavel Shmakov <pshmakov@chromium.org> Cr-Commit-Position: refs/heads/master@{#611804}
-
Peng Huang authored
known issues: * skia doesn't support GL_RG8 format uv texture. * gpu::GetGrBackendTexture() doesn't support GL_R8 and GL_RG8 formats. Bug: 908842 Change-Id: Ice3ffa1135a5b9fdbbb69bcb8175c5f9e196e778 Reviewed-on: https://chromium-review.googlesource.com/c/1347108Reviewed-by:
weiliangc <weiliangc@chromium.org> Commit-Queue: Peng Huang <penghuang@chromium.org> Cr-Commit-Position: refs/heads/master@{#611803}
-
v8-ci-autoroll-builder authored
Summary of changes available at: https://chromium.googlesource.com/v8/v8/+log/03ba418f..dad75d09 Please follow these instructions for assigning/CC'ing issues: https://github.com/v8/v8/wiki/Triaging%20issues 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;luci.chromium.try:linux_optional_gpu_tests_rel;luci.chromium.try:mac_optional_gpu_tests_rel;luci.chromium.try:win_optional_gpu_tests_rel;luci.chromium.try:android_optional_gpu_tests_rel TBR=hablich@chromium.org,v8-waterfall-sheriff@grotations.appspotmail.com Change-Id: I011f908680dac90227732ea061759bea04c77200 Reviewed-on: https://chromium-review.googlesource.com/c/1354122Reviewed-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@{#611802}
-
Etienne Bergeron authored
NOTE TO CHROMIUM SHERIFF: If Tracing unittests are flaky, please revert this CL and ping me. Change-Id: Ia4a6a57e190afd82eec4b133ce8e1f33bc6c1f0f Reviewed-on: https://chromium-review.googlesource.com/c/1349495 Commit-Queue: Etienne Bergeron <etienneb@chromium.org> Reviewed-by:
oysteine <oysteine@chromium.org> Cr-Commit-Position: refs/heads/master@{#611801}
-
Abhishek Arya authored
R=ochang@google.com Bug: 907103 Change-Id: I6a6f7f6eb33574e415d71ee72c35a2eecfa22c87 Reviewed-on: https://chromium-review.googlesource.com/c/1352827Reviewed-by:
Eric Roman <eroman@chromium.org> Reviewed-by:
enne <enne@chromium.org> Reviewed-by:
Albert J. Wong <ajwong@chromium.org> Reviewed-by:
Kent Tamura <tkent@chromium.org> Reviewed-by:
Oliver Chang <ochang@chromium.org> Reviewed-by:
Noel Gordon <noel@chromium.org> Reviewed-by:
Max Moroz <mmoroz@chromium.org> Commit-Queue: Abhishek Arya <inferno@chromium.org> Cr-Commit-Position: refs/heads/master@{#611800}
-
Azeem Arshad authored
This CL fixes crash in notifications settings page. This is reproduced on devices where multidevice features are disabled by policy. In this case MultiDeviceSetupClient is not instantiated, causing the look up for android messages info in the notifications settings page to crash. This was fixed by guarding the look up by a loadtime value that indicates whether multidevice features are allowed policy or not. Bug: 905908 Change-Id: Ic637768802e4e15b3643fe20dcc40d1bb3cea9da Reviewed-on: https://chromium-review.googlesource.com/c/1345729 Commit-Queue: Azeem Arshad <azeemarshad@chromium.org> Reviewed-by:
Demetrios Papadopoulos <dpapad@chromium.org> Reviewed-by:
Jeremy Klein <jlklein@chromium.org> Reviewed-by:
Kyle Horimoto <khorimoto@chromium.org> Cr-Commit-Position: refs/heads/master@{#611799}
-
Adam Langley authored
https://boringssl.googlesource.com/boringssl/+log/6965d25602754bc419c5f757d008ba1f4da49ae4..0f5ecd3a854546d943104e1f7421e489b7f4d5aa Bug: none Change-Id: I047aef10bbd47bf50c567d2d22c250aae5fabf85 Reviewed-on: https://chromium-review.googlesource.com/c/1351138 Commit-Queue: David Benjamin <davidben@chromium.org> Reviewed-by:
David Benjamin <davidben@chromium.org> Cr-Commit-Position: refs/heads/master@{#611798}
-
Xiaoqian Dai authored
When overview and splitview are both active, tab dragging might cause the misplacement of the divider bar, see the bug description for details. The reason is that in tab dragging, it's possible the window gets activated between the drag ends and window snap, this CL makes sure this case is also properly handled. Bug: 908992 Change-Id: I767504153b153f3325dfce9212c647e5f390c0c9 Reviewed-on: https://chromium-review.googlesource.com/c/1352412 Commit-Queue: Xiaoqian Dai <xdai@chromium.org> Reviewed-by:
Xiyuan Xia <xiyuan@chromium.org> Cr-Commit-Position: refs/heads/master@{#611797}
-
Liquan(Max) Gu authored
In the initial version, the calculation crashes when it tries to intersect the object's rect with the viewport's rect for certain values. It's because intersect indicates a calculation of maxX = x + width, which has overflow issue when x and width is very big. To fix this issue, the sizes of each rect in intersect must do saturated arithmetic. Bug: 908974 Change-Id: Ida184dd69dfa0008d7c878e292ab908b1536f7ba Reviewed-on: https://chromium-review.googlesource.com/c/1352570Reviewed-by:
Steve Kobes <skobes@chromium.org> Commit-Queue: Liquan (Max) Gǔ <maxlg@chromium.org> Cr-Commit-Position: refs/heads/master@{#611796}
-
chromium-autoroll authored
https://chromium.googlesource.com/chromiumos/chromite.git/+log/d2ede8935345..e7e7a29184f5 git log d2ede8935345..e7e7a29184f5 --date=short --no-merges --format='%ad %ae %s' 2018-11-28 chrome-bot@chromium.org Update config settings by config-updater. Created with: gclient setdep -r src/third_party/chromite@e7e7a29184f5 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. TBR=chrome-os-gardeners@chromium.org Change-Id: Ic7904d9b239740f9391c2501ca717ffadb6554a5 Reviewed-on: https://chromium-review.googlesource.com/c/1352864Reviewed-by:
-
Eric Seckler authored
Bug: 900603 Change-Id: If567eedb4071af1d53072bad81c10731c4515fa0 Reviewed-on: https://chromium-review.googlesource.com/c/1353998 Commit-Queue: oysteine <oysteine@chromium.org> Reviewed-by:
-
Ken Rockot authored
Bug: 909799 Change-Id: I9fdeec4f569d7ee272f8a384012fec3a201908eb Reviewed-on: https://chromium-review.googlesource.com/c/1354220Reviewed-by:
-
Vladimir Levin authored
This patch mimics the style code in that it tries not to propagate the invalidation up the container chain if the element is locked for display When the lock commits and the element needs layout, then it marks the ancestor chain so that the code can reach it in layout. R=chrishtr@chromium.org, mstensho@chromium.org Bug: 882663 Change-Id: I499e91e6a14f61e5dfbeae74d3c93ac8b321e944 Reviewed-on: https://chromium-review.googlesource.com/c/1344825Reviewed-by:
Chris Harrelson <chrishtr@chromium.org> Commit-Queue: vmpstr <vmpstr@chromium.org> Cr-Commit-Position: refs/heads/master@{#611792}
-
Shawn Gallea authored
APIs that have Ash dependencies can only run on ChromeOS so bind them only when building for ChromeOS. This allows EXO to build on Chromecast. Bug: 896710 Test: None Change-Id: I60acf956b5a0f682f89c97b5818bb54cefcc7113 Reviewed-on: https://chromium-review.googlesource.com/c/1328702Reviewed-by:
Daniele Castagna <dcastagna@chromium.org> Reviewed-by:
Daniel Nicoara <dnicoara@chromium.org> Commit-Queue: Daniel Nicoara <dnicoara@chromium.org> Cr-Commit-Position: refs/heads/master@{#611791}
-
Sky Malice authored
Bug: 903983 Change-Id: If4a6bdd236a82267e4b8302992dc9c0d3b3a8cc0 Reviewed-on: https://chromium-review.googlesource.com/c/1344908 Commit-Queue: Sky Malice <skym@chromium.org> Reviewed-by:
Theresa <twellington@chromium.org> Reviewed-by:
Filip Gorski <fgorski@chromium.org> Cr-Commit-Position: refs/heads/master@{#611790}
-
