- 09 Aug, 2019 40 commits
-
-
chromium-autoroll authored
https://skia.googlesource.com/skia.git/+log/b2151310068b..26cedb4c603b git log b2151310068b..26cedb4c603b --date=short --no-merges --format='%ad %ae %s' 2019-08-09 egdaniel@google.com Fix emplace on GrTRecorder. Created with: gclient setdep -r src/third_party/skia@26cedb4c603b 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=reed@google.com Bug: None Change-Id: I4f992b3e0b70ad4dffcd4ea075590112c81ec124 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1746304Reviewed-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@{#685563}
-
Sahel Sharify authored
This cl adds PaymentRequest.TimeToCheckout.[Completed|UserAborted|OtherAborted] metrics for Desktop and Android. Bug: 989604 Change-Id: I585b2592ab19362fdebd50190a930cf238dd3439 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1728264Reviewed-by:
Sahel Sharify <sahel@chromium.org> Reviewed-by:
Steven Holte <holte@chromium.org> Reviewed-by:
Rouslan Solomakhin <rouslan@chromium.org> Reviewed-by:
Danyao Wang <danyao@chromium.org> Commit-Queue: Sahel Sharify <sahel@chromium.org> Cr-Commit-Position: refs/heads/master@{#685562}
-
Carlos Caballero authored
MessageLoopForUI will go away, eventually. ScopedTaskEnvironment will per default start a ThreadPool, which should be fine in most of the cases. If you believe your test needs to make sure that no ThreadPool runs let me know and I will update the patch. BUG=891670 This CL was uploaded by git cl split. R=msw@chromium.org Change-Id: I682631b97e8a874f9d586520a937929d91953e44 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1718025Reviewed-by:
Michael Wasserman <msw@chromium.org> Commit-Queue: Carlos Caballero <carlscab@google.com> Cr-Commit-Position: refs/heads/master@{#685561}
-
Carlos Caballero authored
MessageLoopForIO will go away soon use ScopedTaskEnvironment instead. ScopedTaskEnvironment will per default start a ThreadPool, which should be fine in most of the cases. If you believe your test needs to make sure that no ThreadPool runs let me know and I will update the patch. BUG=891670 This CL was uploaded by git cl split. R=zea@chromium.org Change-Id: I85626fd1fddb7d36b61b60574b5a6868630d4ed5 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1733550Reviewed-by:
Nicolas Zea <zea@chromium.org> Commit-Queue: Carlos Caballero <carlscab@google.com> Cr-Commit-Position: refs/heads/master@{#685560}
-
Sami Kyostila authored
*** Note: There is no behavior change from this patch. *** The PostTask APIs will shortly be changed to require all tasks to explicitly specify their thread affinity, i.e., whether the task should run on the thread pool or a specific named thread such as a BrowserThread. This patch updates all call sites with thread affinity annotation. We also remove the "WithTraits" suffix to make the call sites more readable. Before: // Thread pool task. base::PostTaskWithTraits(FROM_HERE, {...}, ...); // UI thread task. base::PostTaskWithTraits(FROM_HERE, {BrowserThread::UI, ...}, ...); After: // Thread pool task. base::PostTask(FROM_HERE, {base::ThreadPool(), ...}, ...); // UI thread task. base::PostTask(FROM_HERE, {BrowserThread::UI, ...}, ...); This patch was semi-automatically prepared with these steps: 1. Patch in https://chromium-review.googlesource.com/c/chromium/src/+/1635827 to make thread affinity a build-time requirement. 2. Run an initial pass with a clang rewriter: https://chromium-review.googlesource.com/c/chromium/src/+/1635623 3. ninja -C out/Debug | grep 'requested here' | cut -d: -f1-3 | sort | \ uniq > errors.txt 4. while read line; do f=$(echo $line | cut -d: -f 1) r=$(echo $line | cut -d: -f 2) c=$(echo $line | cut -d: -f 3) sed -i "${r}s/./&base::ThreadPool(),/$c" $f done < errors.txt 5. GOTO 3 until build succeeds. 6. Remove the "WithTraits" suffix from task API call sites: $ tools/git/mffr.py -i <(cat <<EOF [ ["PostTaskWithTraits", "PostTask"], ["PostDelayedTaskWithTraits", "PostDelayedTask"], ["PostTaskWithTraitsAndReply", "PostTaskAndReply"], ["CreateTaskRunnerWithTraits", "CreateTaskRunner"], ["CreateSequencedTaskRunnerWithTraits", "CreateSequencedTaskRunner"], ["CreateUpdateableSequencedTaskRunnerWithTraits", "CreateUpdateableSequencedTaskRunner"], ["CreateSingleThreadTaskRunnerWithTraits", "CreateSingleThreadTaskRunner"], ["CreateCOMSTATaskRunnerWithTraits", "CreateCOMSTATaskRunner"] ] EOF ) This CL was uploaded by git cl split. R=alemate@chromium.org, atwilson@chromium.org Bug: 968047 Change-Id: If9e04d35f6c3b796f0973eed20a9a98bf6f72ab4 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1728878 Auto-Submit: Sami Kyöstilä <skyostil@chromium.org> Reviewed-by:Alexander Alekseev <alemate@chromium.org> Reviewed-by:
Maksim Ivanov <emaxx@chromium.org> Commit-Queue: Maksim Ivanov <emaxx@chromium.org> Cr-Commit-Position: refs/heads/master@{#685559}
-
Carlos Caballero authored
MessageLoopForIO will go away soon use ScopedTaskEnvironment instead. ScopedTaskEnvironment will per default start a ThreadPool, which should be fine in most of the cases. If you believe your test needs to make sure that no ThreadPool runs let me know and I will update the patch. BUG=891670 This CL was uploaded by git cl split. R=stevenjb@chromium.org Change-Id: I9460fd5fe4a0e3b73ac80f5decd1fd9980d8b6ba Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1733552Reviewed-by:
Steven Bennetts <stevenjb@chromium.org> Commit-Queue: Carlos Caballero <carlscab@google.com> Cr-Commit-Position: refs/heads/master@{#685558}
-
Anton Bikineev authored
Before this CL the rules for GarbageCollectedFinalized were diverging from standard C++ std::is_trivially_destructible because heap collections had a destructor that would be ignored because the classes were not required to be GarbageCollectedFinalized. After the CL, std::is_trivially_destructible can be used as replacement for GarbageCollectedFinalized. Benchmarks (blink_perf.dom, blink_perf.bindings, speedometer) show no regressions. Bug: 990913 Change-Id: Ie1722b5c7f97bc7197c34c7d26120e7147893fd5 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1742147 Commit-Queue: Anton Bikineev <bikineev@chromium.org> Reviewed-by:
Michael Lippautz <mlippautz@chromium.org> Reviewed-by:
Kentaro Hara <haraken@chromium.org> Cr-Commit-Position: refs/heads/master@{#685557}
-
Ben Kelly authored
This CL also flips the respective virtual test suites to disable their features to maintain test coverage of the non-trial configurations. Bug: 985379 Change-Id: I57b2eb98e9d932b0b714804c58b89560bcf7afad Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1736928Reviewed-by:
-
v8-ci-autoroll-builder authored
Summary of changes available at: https://chromium.googlesource.com/v8/v8/+log/520e310f..63ed65e3 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: I7fcde6acb6171c99722ade67a3e55f89b4050208 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1745276Reviewed-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@{#685555}
-
Elly Fong-Jones authored
This change removes all the remaining places where LocationBarBubbleDelegateView's constructor is called with a value other than an in-line gfx::Point(). The next CL of this series will remove that parameter from LocationBarBubbleDelegateView itself and clean up documentation and helper functions. Bug: 989080 Change-Id: I524f836f898365ba662a677ca92118a11cdeb658 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1744638 Commit-Queue: Elly Fong-Jones <ellyjones@chromium.org> Auto-Submit: Elly Fong-Jones <ellyjones@chromium.org> Reviewed-by:
Robert Liao <robliao@chromium.org> Cr-Commit-Position: refs/heads/master@{#685554}
-
Sami Kyostila authored
*** Note: There is no behavior change from this patch. *** The PostTask APIs will shortly be changed to require all tasks to explicitly specify their thread affinity, i.e., whether the task should run on the thread pool or a specific named thread such as a BrowserThread. This patch updates all call sites with thread affinity annotation. We also remove the "WithTraits" suffix to make the call sites more readable. Before: // Thread pool task. base::PostTaskWithTraits(FROM_HERE, {...}, ...); // UI thread task. base::PostTaskWithTraits(FROM_HERE, {BrowserThread::UI, ...}, ...); After: // Thread pool task. base::PostTask(FROM_HERE, {base::ThreadPool(), ...}, ...); // UI thread task. base::PostTask(FROM_HERE, {BrowserThread::UI, ...}, ...); This patch was semi-automatically prepared with these steps: 1. Patch in https://chromium-review.googlesource.com/c/chromium/src/+/1635827 to make thread affinity a build-time requirement. 2. Run an initial pass with a clang rewriter: https://chromium-review.googlesource.com/c/chromium/src/+/1635623 3. ninja -C out/Debug | grep 'requested here' | cut -d: -f1-3 | sort | \ uniq > errors.txt 4. while read line; do f=$(echo $line | cut -d: -f 1) r=$(echo $line | cut -d: -f 2) c=$(echo $line | cut -d: -f 3) sed -i "${r}s/./&base::ThreadPool(),/$c" $f done < errors.txt 5. GOTO 3 until build succeeds. 6. Remove the "WithTraits" suffix from task API call sites: $ tools/git/mffr.py -i <(cat <<EOF [ ["PostTaskWithTraits", "PostTask"], ["PostDelayedTaskWithTraits", "PostDelayedTask"], ["PostTaskWithTraitsAndReply", "PostTaskAndReply"], ["CreateTaskRunnerWithTraits", "CreateTaskRunner"], ["CreateSequencedTaskRunnerWithTraits", "CreateSequencedTaskRunner"], ["CreateUpdateableSequencedTaskRunnerWithTraits", "CreateUpdateableSequencedTaskRunner"], ["CreateSingleThreadTaskRunnerWithTraits", "CreateSingleThreadTaskRunner"], ["CreateCOMSTATaskRunnerWithTraits", "CreateCOMSTATaskRunner"] ] EOF ) This CL was uploaded by git cl split. TBR=stevenjb@chromium.org Bug: 968047 Change-Id: Ie78d666f009a78c204cd8d161b3a6aab22b31a28 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1728556Reviewed-by:Sami Kyöstilä <skyostil@chromium.org> Commit-Queue: Sami Kyöstilä <skyostil@chromium.org> Auto-Submit: Sami Kyöstilä <skyostil@chromium.org> Cr-Commit-Position: refs/heads/master@{#685553}
-
Md. Hasanur Rashid authored
RRectFBuilder is implemented to make the parameter setting easier for RRectF. Unit tests are provided that matches the existing ways of declaration for initialization of RRectF. There is change in rrect_f_mojom_traits.h file to show how the RRectFBuilder transforms the initialization of RRectF more understandable. Bug: 927033 Change-Id: Iad4aa493e29d4f7578b102cc04ae337acdf85107 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1730150Reviewed-by:
Mason Freed <masonfreed@chromium.org> Reviewed-by:
Sadrul Chowdhury <sadrul@chromium.org> Reviewed-by:
Dominick Ng <dominickn@chromium.org> Commit-Queue: Mason Freed <masonfreed@chromium.org> Cr-Commit-Position: refs/heads/master@{#685552}
-
Robert Phillips authored
The bug fix will require some cc unit test rebaselines and some layout rebaselines. Here is the Skia CL in question: https://skia-review.googlesource.com/c/skia/+/233556/ (Fix exactify for explicit resource allocation (take 2)) Bug: b/138674523 Change-Id: Icc4507129198dbf327d0dff4a2e3fe04b8fd91a4 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1745440 Commit-Queue: Robert Phillips <robertphillips@google.com> Commit-Queue: Florin Malita <fmalita@chromium.org> Reviewed-by:
Florin Malita <fmalita@chromium.org> Cr-Commit-Position: refs/heads/master@{#685551}
-
Javier Ernesto Flores Robles authored
UX approve stamped this green as the replacement for the old palette. Bug: 976675 Change-Id: Id9be807b8315d171678605b3aeda5061ff4fe18c Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1746177 Commit-Queue: Javier Ernesto Flores Robles <javierrobles@chromium.org> Commit-Queue: Louis Romero <lpromero@chromium.org> Auto-Submit: Javier Ernesto Flores Robles <javierrobles@chromium.org> Reviewed-by:
Louis Romero <lpromero@chromium.org> Cr-Commit-Position: refs/heads/master@{#685550}
-
chromium-autoroll authored
https://android.googlesource.com/platform/external/perfetto.git/+log/34e00069a965..f3b064fc4807 git log 34e00069a965..f3b064fc4807 --date=short --no-merges --format='%ad %ae %s' 2019-08-09 skyostil@google.com Merge "gen_amalgamated: Move tests from gen_all to CI" Created with: gclient setdep -r src/third_party/perfetto@f3b064fc4807 The AutoRoll server is located here: https://autoroll.skia.org/r/perfetto-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=perfetto-bugs@google.com Bug: None Change-Id: I7488909c6b3df9e644a0ccd9b7b772b7a25d1565 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1746298Reviewed-by:
-
Maksim Ivanov authored
Pipe the authenticating user's AccountId down to the UI-related code that shows security token PIN dialogs, so that the UI layer has ability to correctly tie the PIN UI with the needed user. Before this CL, the UI code has to assume that the currently selected user is the one that for which the PIN request should be shown. However, this assumption is unreliable in general: for example, the user could have already clicked a different user pod before the PIN request arrived, and as a result the PIN dialog would be shown on a wrong user pod. This CL extracts the AccountId based on the AccountIdentifier information supplied by the cryptohomed to the org.chromium.CryptohomeKeyDelegateInterface D-Bus service exposed by the browser. Bug: 983103 Change-Id: I55999c64f3c9af29fa8c231210ad3282643da885 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1742172 Commit-Queue: Maksim Ivanov <emaxx@chromium.org> Reviewed-by:
Igor <igorcov@chromium.org> Reviewed-by:
Ryo Hashimoto <hashimoto@chromium.org> Cr-Commit-Position: refs/heads/master@{#685548}
-
Alexandre Frechette authored
Change-Id: I7b8dafb5a97f93104147f2482f3fbda0857d34ef Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1744333Reviewed-by:
Theresa <twellington@chromium.org> Reviewed-by:
anthonyvd <anthonyvd@chromium.org> Commit-Queue: Alexandre Frechette <frechette@chromium.org> Cr-Commit-Position: refs/heads/master@{#685547}
-
Sami Kyostila authored
*** Note: There is no behavior change from this patch. *** The PostTask APIs will shortly be changed to require all tasks to explicitly specify their thread affinity, i.e., whether the task should run on the thread pool or a specific named thread such as a BrowserThread. This patch updates all call sites with thread affinity annotation. We also remove the "WithTraits" suffix to make the call sites more readable. Before: // Thread pool task. base::PostTaskWithTraits(FROM_HERE, {...}, ...); // UI thread task. base::PostTaskWithTraits(FROM_HERE, {BrowserThread::UI, ...}, ...); After: // Thread pool task. base::PostTask(FROM_HERE, {base::ThreadPool(), ...}, ...); // UI thread task. base::PostTask(FROM_HERE, {BrowserThread::UI, ...}, ...); This patch was semi-automatically prepared with these steps: 1. Patch in https://chromium-review.googlesource.com/c/chromium/src/+/1635827 to make thread affinity a build-time requirement. 2. Run an initial pass with a clang rewriter: https://chromium-review.googlesource.com/c/chromium/src/+/1635623 3. ninja -C out/Debug | grep 'requested here' | cut -d: -f1-3 | sort | \ uniq > errors.txt 4. while read line; do f=$(echo $line | cut -d: -f 1) r=$(echo $line | cut -d: -f 2) c=$(echo $line | cut -d: -f 3) sed -i "${r}s/./&base::ThreadPool(),/$c" $f done < errors.txt 5. GOTO 3 until build succeeds. 6. Remove the "WithTraits" suffix from task API call sites: $ tools/git/mffr.py -i <(cat <<EOF [ ["PostTaskWithTraits", "PostTask"], ["PostDelayedTaskWithTraits", "PostDelayedTask"], ["PostTaskWithTraitsAndReply", "PostTaskAndReply"], ["CreateTaskRunnerWithTraits", "CreateTaskRunner"], ["CreateSequencedTaskRunnerWithTraits", "CreateSequencedTaskRunner"], ["CreateUpdateableSequencedTaskRunnerWithTraits", "CreateUpdateableSequencedTaskRunner"], ["CreateSingleThreadTaskRunnerWithTraits", "CreateSingleThreadTaskRunner"], ["CreateCOMSTATaskRunnerWithTraits", "CreateCOMSTATaskRunner"] ] EOF ) This CL was uploaded by git cl split. R=tbarzic@chromium.org Bug: 968047 Change-Id: I48ea6b3d73e532cef45ce42a7601a2e02b3eb36e Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1729155 Commit-Queue: Sami Kyöstilä <skyostil@chromium.org> Auto-Submit: Sami Kyöstilä <skyostil@chromium.org> Reviewed-by:Ben Wells <benwells@chromium.org> Cr-Commit-Position: refs/heads/master@{#685546}
-
Sami Kyostila authored
*** Note: There is no behavior change from this patch. *** The PostTask APIs will shortly be changed to require all tasks to explicitly specify their thread affinity, i.e., whether the task should run on the thread pool or a specific named thread such as a BrowserThread. This patch updates all call sites with thread affinity annotation. We also remove the "WithTraits" suffix to make the call sites more readable. Before: // Thread pool task. base::PostTaskWithTraits(FROM_HERE, {...}, ...); // UI thread task. base::PostTaskWithTraits(FROM_HERE, {BrowserThread::UI, ...}, ...); After: // Thread pool task. base::PostTask(FROM_HERE, {base::ThreadPool(), ...}, ...); // UI thread task. base::PostTask(FROM_HERE, {BrowserThread::UI, ...}, ...); This patch was semi-automatically prepared with these steps: 1. Patch in https://crrev.com/c/1635827 to make thread affinity a build-time requirement. 2. Run an initial pass with a clang rewriter: https://chromium-review.googlesource.com/c/chromium/src/+/1635623 3. ninja -C out/Debug | grep 'requested here' | cut -d: -f1-3 | sort | \ uniq > errors.txt 4. while read line; do f=$(echo $line | cut -d: -f 1) r=$(echo $line | cut -d: -f 2) c=$(echo $line | cut -d: -f 3) sed -i "${r}s/./&base::ThreadPool(),/$c" $f done < errors.txt 5. GOTO 3 until build succeeds. 6. Remove the "WithTraits" suffix from task API call sites: $ tools/git/mffr.py -i <(cat <<EOF [ ["PostTaskWithTraits", "PostTask"], ["PostDelayedTaskWithTraits", "PostDelayedTask"], ["PostTaskWithTraitsAndReply", "PostTaskAndReply"], ["CreateTaskRunnerWithTraits", "CreateTaskRunner"], ["CreateSequencedTaskRunnerWithTraits", "CreateSequencedTaskRunner"], ["CreateUpdateableSequencedTaskRunnerWithTraits", "CreateUpdateableSequencedTaskRunner"], ["CreateSingleThreadTaskRunnerWithTraits", "CreateSingleThreadTaskRunner"], ["CreateCOMSTATaskRunnerWithTraits", "CreateCOMSTATaskRunner"] ] EOF ) This CL was uploaded by git cl split. TBR=avi@chromium.org Bug: 968047 Change-Id: If81258c798697344b2a536dfc199e217a8eb1e98 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1739706Reviewed-by: -
Ramin Halavati authored
Traffic annotation auditor binary needs to be updated due to a recent updated of enterprise policies. TBR: nicolaso@chromium.org Change-Id: I9a636966235f1d9437f6c117ec0c2e1058a33cf4 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1746254Reviewed-by:
Ramin Halavati <rhalavati@chromium.org> Reviewed-by:
Martin Šrámek <msramek@chromium.org> Commit-Queue: Ramin Halavati <rhalavati@chromium.org> Cr-Commit-Position: refs/heads/master@{#685544}
-
Vlad Tsyrklevich authored
I accidentally used the wrong parameters in this field trial testing config (it should sample processes half as often.) Bug: 973167 Change-Id: I0a26dd365e3a3cb5bbcc860ba9693295e1f912bc Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1744688 Auto-Submit: Vlad Tsyrklevich <vtsyrklevich@chromium.org> Reviewed-by:
Brian White <bcwhite@chromium.org> Commit-Queue: Brian White <bcwhite@chromium.org> Cr-Commit-Position: refs/heads/master@{#685543}
-
chromium-autoroll authored
https://chromium.googlesource.com/angle/angle.git/+log/eb0479e245f0..e54c3e3aa01c git log eb0479e245f0..e54c3e3aa01c --date=short --no-merges --format='%ad %ae %s' 2019-08-09 angle-autoroll@skia-public.iam.gserviceaccount.com Roll ./third_party/spirv-tools/src b029d3697ea3..22ce39c8e1ec (5 commits) Created with: gclient setdep -r src/third_party/angle@e54c3e3aa01c The AutoRoll server is located here: https://autoroll.skia.org/r/angle-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=ynovikov@chromium.org Bug: chromium:None Change-Id: I19bfff4d79cf5fcdbcc1b74c882ef774170e275e Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1746192Reviewed-by:
-
Ivana Zuzic authored
The changes of password data are saved by clicking on the save button in PasswordentryEditor. After that, PasswordEntryEditor gets closed and the changes are visible in PasswordEntryViewer and on the list of passwords. Bug: 377410 Change-Id: Ia0e9707d508b6a6449fb97263e31e5747016a945 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1736685 Commit-Queue: Ivana Zuzic <izuzic@google.com> Reviewed-by:
Friedrich [CET] <fhorschig@chromium.org> Reviewed-by:
Vasilii Sukhanov <vasilii@chromium.org> Cr-Commit-Position: refs/heads/master@{#685541}
-
Mugdha Lakhani authored
This lives under the Service Worker section of the Application tab in Devtools and allows dispatching a periodicsync event with a given tag so that developers can test how they respond to it. Screenshots here: https://bugs.chromium.org/p/chromium/issues/detail?id=961238#c4 Bug: 961238 Change-Id: I2f5fe2c41ad9d2c50db99c44d873d8cc61964913 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1725893Reviewed-by:
Joel Einbinder <einbinder@chromium.org> Reviewed-by:
Dmitry Gozman <dgozman@chromium.org> Reviewed-by:
Andrey Kosyakov <caseq@chromium.org> Reviewed-by:
Rayan Kanso <rayankans@chromium.org> Commit-Queue: Mugdha Lakhani <nator@chromium.org> Auto-Submit: Mugdha Lakhani <nator@chromium.org> Cr-Commit-Position: refs/heads/master@{#685540}
-
Josh Nohle authored
Bug: 899080 Change-Id: I15c11a7f41f76d8f0c49a3580f70ebeeaa151d2b Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1737669Reviewed-by:
Kyle Horimoto <khorimoto@chromium.org> Reviewed-by:
Elly Fong-Jones <ellyjones@chromium.org> Commit-Queue: Elly Fong-Jones <ellyjones@chromium.org> Cr-Commit-Position: refs/heads/master@{#685539}
-
Yang Guo authored
When built using debug_devtools=true, allow using --custom-devtools-frontend command line flag to specify path to DevTools frontend resource files. Bug: 988990 Change-Id: I90e84ba797397af77477838030961ebd6f2f8cac Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1741813Reviewed-by:
-
Friedrich Horschig authored
This is a reland of 3cd4817b The original CL was based on https://crrev.com/c/1729592 which was reverted. Once the reland https://crrev.com/c/1741916 is merged, this CL should work as intended. TBR=jdoerrie@chromium.org,ioanap@chromium.org Original change's description: > [Mfill Android] Render credit card warning in TextView > > This CL ensures that the unsafe origin warning is rendered as TextView > and that fields which are marked as disabled are actually disabled. > > Bug: 965500 > Change-Id: I2c0c66e86075b3152431dcd18b4922c1167786e9 > Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1736355 > Reviewed-by: Ioana Pandele <ioanap@chromium.org> > Reviewed-by: Jan Wilken Dörrie <jdoerrie@chromium.org> > Commit-Queue: Friedrich [CET] <fhorschig@chromium.org> > Cr-Commit-Position: refs/heads/master@{#685112} Bug: 965500 Change-Id: I0cbe46c4d7e0a6d2fb936a1b0dcb99c657af2a9e Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1742236Reviewed-by:
-
Peter Marshall authored
Part of an ongoing cleanup to use Detach consistently. This CL is a mechanical renaming and should not change any logic. Bug: chromium:913887 Change-Id: I224b80b910bd84076880915063ee8b798ef91689 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1738697Reviewed-by:
Mike West <mkwst@chromium.org> Commit-Queue: Peter Marshall <petermarshall@chromium.org> Cr-Commit-Position: refs/heads/master@{#685536}
-
Chromium WPT Sync authored
Using wpt-import in Chromium 39263da7. With Chromium commits locally applied on WPT: 329c91e5 "Attempt to make test more stable" Note to sheriffs: This CL imports external tests and adds expectations for those tests; if this CL is large and causes a few new failures, please fix the failures by adding new lines to TestExpectations rather than reverting. See: https://chromium.googlesource.com/chromium/src/+/master/docs/testing/web_platform_tests.md Directory owners for changes in this CL: foolip@chromium.org, lpz@chromium.org, robertma@chromium.org: external/wpt/tools NOAUTOREVERT=true TBR=robertma No-Export: true Change-Id: Ic0f4f587e5b36811d31e6ee540f67fe9be8164ba Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1745271Reviewed-by:
WPT Autoroller <wpt-autoroller@chops-service-accounts.iam.gserviceaccount.com> Commit-Queue: WPT Autoroller <wpt-autoroller@chops-service-accounts.iam.gserviceaccount.com> Cr-Commit-Position: refs/heads/master@{#685535}
-
chromium-internal-autoroll authored
https://chrome-internal.googlesource.com/chrome/src-internal.git/+log/2bd4849cc139..794dd735e4f8 Created with: gclient setdep -r src-internal@794dd735e4f8 The AutoRoll server is located here: https://skia-autoroll.corp.goog/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. CQ_INCLUDE_TRYBOTS=luci.chrome.try:linux-chromeos-chrome TBR=jbudorick@google.com Bug: chromium:None Change-Id: I1eb438afd71e5170ea30effdaa702ad936851f5f Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1746295Reviewed-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@{#685534}
-
Marina Ciocea authored
Drive-by fix crash when the browser is closed while sharing a tab. Bug: 991896 Change-Id: I00fe5791cd6856f8258a279a7fa5f6e7d276237f Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1746169 Commit-Queue: Marina Ciocea <marinaciocea@chromium.org> Commit-Queue: Guido Urdaneta <guidou@chromium.org> Reviewed-by:
Guido Urdaneta <guidou@chromium.org> Cr-Commit-Position: refs/heads/master@{#685533}
-
Javier Ernesto Flores Robles authored
This CL updates most colors to their dynamic counterparts. Exceptions: Background Color of the Cells and grey trailing icons. Screenshots: https://drive.google.com/drive/folders/1FnJz38j2xyZAWURcujmqYuTrwrkYnUnR Bug: 976675 Change-Id: I8f622ad673c7e2cf7baf77011fe91d45655b0e17 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1744489 Commit-Queue: Javier Ernesto Flores Robles <javierrobles@chromium.org> Commit-Queue: Louis Romero <lpromero@chromium.org> Reviewed-by:
-
v8-ci-autoroll-builder authored
Summary of changes available at: https://chromium.googlesource.com/v8/v8/+log/46afc4f9..520e310f 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: I1d8e189781e9058f7b0015ac84a3ff365e45feef Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1745273Reviewed-by:
-
Lowell Manners authored
This test is interested in the state of documents as they're deleted during navigation. Change-Id: I4db86fb97d45771aecf33f8a7afdc93c00602d21 Bug: 990354 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1733526 Commit-Queue: Lowell Manners <lowell@chromium.org> Reviewed-by:
Nasko Oskov <nasko@chromium.org> Reviewed-by:
Arthur Sonzogni <arthursonzogni@chromium.org> Cr-Commit-Position: refs/heads/master@{#685530}
-
Rouslan Solomakhin authored
When a payment handler sends a "payment method change" event to the merchant, the developer tools can display this event and the corresponding price update from the merchant. Bug: 980249 Change-Id: Ia3f16215cadf54d0a3f44b6b5f471839ca498f13 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1742207Reviewed-by:
Avi Drissman <avi@chromium.org> Commit-Queue: Rouslan Solomakhin <rouslan@chromium.org> Cr-Commit-Position: refs/heads/master@{#685529}
-
Eric Lawrence authored
Save 28786 bytes of disk space by removing unneeded metadata from two PNG files. Bug: 991654 Change-Id: I6b29592501c2cd6737c392e146f0770464453855 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1742609Reviewed-by:
-
chromium-autoroll authored
https://webrtc.googlesource.com/src.git/+log/273e263d25b4..e4ba4ee837ac git log 273e263d25b4..e4ba4ee837ac --date=short --no-merges --format='%ad %ae %s' 2019-08-09 nisse@webrtc.org Delete placeholder file rtc_base/function_view.h 2019-08-09 nisse@webrtc.org Use StreamStatistician::BitrateReceived to produce total_bitrate_bps for GetStats. 2019-08-09 nisse@webrtc.org Delete rtc_base/gunit_prod.h 2019-08-09 nisse@webrtc.org Add missing AppKit dependency Created with: gclient setdep -r src/third_party/webrtc@e4ba4ee837ac The AutoRoll server is located here: https://autoroll.skia.org/r/webrtc-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=webrtc-chromium-sheriffs-robots@google.com Bug: chromium:None,chromium:None,chromium:None Change-Id: I9f5c151715a1b5b6dc3e003d6aa65d7323056fe9 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1746291Reviewed-by:
-
Eric Seckler authored
We're seeing crashes within AddTraceEvent called from UpdateDuration when attempting to access the trace event's category pointer. The category pointer is nullptr in this case, which leads me to believe that UpdateDuration may attempt to access an uninitialized TraceEvent from the complete_event_stack_. One piece of code that looks suspicious to me is the update of current_stack_depth_ in UpdateDuration in case the handle's event_index and current_stack_depth_ mismatch for some reason. Provided the DCHECK holds, it still shouldn't cause an invalid TraceEvent to be accessed in later invocations of UpdateDuration, but this is an attempt to make the update a little safer nevertheless. Bug: 983307 Change-Id: I6c6e4dc02bdde5e5e324c65ffb6e5cfe1028c6dd Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1741919 Auto-Submit: Eric Seckler <eseckler@chromium.org> Reviewed-by:
-
Nico Weber authored
After https://chromium-review.googlesource.com/c/chromium/tools/build/+/1744467 the clang mac tot bots should have a system Xcode install that we want to use instead. Bug: 976593,985971 Change-Id: I1bf0aaacf4f6485352b4741e6a263cbfbe7d4318 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1743104Reviewed-by:
Hans Wennborg <hans@chromium.org> Commit-Queue: Nico Weber <thakis@chromium.org> Cr-Commit-Position: refs/heads/master@{#685525}
-
Sami Kyostila authored
*** Note: There is no behavior change from this patch. *** The PostTask APIs will shortly be changed to require all tasks to explicitly specify their thread affinity, i.e., whether the task should run on the thread pool or a specific named thread such as a BrowserThread. This patch updates all call sites with thread affinity annotation. We also remove the "WithTraits" suffix to make the call sites more readable. Before: // Thread pool task. base::PostTaskWithTraits(FROM_HERE, {...}, ...); // UI thread task. base::PostTaskWithTraits(FROM_HERE, {BrowserThread::UI, ...}, ...); After: // Thread pool task. base::PostTask(FROM_HERE, {base::ThreadPool(), ...}, ...); // UI thread task. base::PostTask(FROM_HERE, {BrowserThread::UI, ...}, ...); This patch was semi-automatically prepared with these steps: 1. Patch in https://chromium-review.googlesource.com/c/chromium/src/+/1635827 to make thread affinity a build-time requirement. 2. Run an initial pass with a clang rewriter: https://chromium-review.googlesource.com/c/chromium/src/+/1635623 3. ninja -C out/Debug | grep 'requested here' | cut -d: -f1-3 | sort | \ uniq > errors.txt 4. while read line; do f=$(echo $line | cut -d: -f 1) r=$(echo $line | cut -d: -f 2) c=$(echo $line | cut -d: -f 3) sed -i "${r}s/./&base::ThreadPool(),/$c" $f done < errors.txt 5. GOTO 3 until build succeeds. 6. Remove the "WithTraits" suffix from task API call sites: $ tools/git/mffr.py -i <(cat <<EOF [ ["PostTaskWithTraits", "PostTask"], ["PostDelayedTaskWithTraits", "PostDelayedTask"], ["PostTaskWithTraitsAndReply", "PostTaskAndReply"], ["CreateTaskRunnerWithTraits", "CreateTaskRunner"], ["CreateSequencedTaskRunnerWithTraits", "CreateSequencedTaskRunner"], ["CreateUpdateableSequencedTaskRunnerWithTraits", "CreateUpdateableSequencedTaskRunner"], ["CreateSingleThreadTaskRunnerWithTraits", "CreateSingleThreadTaskRunner"], ["CreateCOMSTATaskRunnerWithTraits", "CreateCOMSTATaskRunner"] ] EOF ) This CL was uploaded by git cl split. R=eugenebut@chromium.org Bug: 968047 Change-Id: Ibebd81e88c85b0436356b25ec53495b86d6eac1a Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1729075 Auto-Submit: Sami Kyöstilä <skyostil@chromium.org> Reviewed-by:Rohit Rao <rohitrao@chromium.org> Commit-Queue: Rohit Rao <rohitrao@chromium.org> Cr-Commit-Position: refs/heads/master@{#685524}
-
