- 06 Aug, 2019 40 commits
-
-
Mustafa Emre Acer authored
Change-Id: I9a6c3772e33e67cfe99f2d20da97c0f52aaa8fbd Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1739543Reviewed-by:
Tommy Li <tommycli@chromium.org> Commit-Queue: Mustafa Emre Acer <meacer@chromium.org> Cr-Commit-Position: refs/heads/master@{#684444}
-
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=rdevlin.cronin@chromium.org Change-Id: I011ff9c037debc4092850caff94f52f4d7667ea0 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1733610Reviewed-by:
Devlin <rdevlin.cronin@chromium.org> Commit-Queue: Devlin <rdevlin.cronin@chromium.org> Cr-Commit-Position: refs/heads/master@{#684443}
-
Brian Sheedy authored
Increases the swarming expiration and overall builder execution timeout for Win10 FYI x64 Release (Intel UHD 630) since only having one device is causing shard timeouts on heavily sharded tests and the builder is getting dangerously close to its current 6 hour timeout. Bug: 986939 Change-Id: I30f01c869b405cea3aae0e11b09f0d81769d62e6 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1739590 Commit-Queue: Brian Sheedy <bsheedy@chromium.org> Commit-Queue: Kenneth Russell <kbr@chromium.org> Auto-Submit: Brian Sheedy <bsheedy@chromium.org> Reviewed-by:
Kenneth Russell <kbr@chromium.org> Cr-Commit-Position: refs/heads/master@{#684442}
-
K Moon authored
Due to the loose, asynchronous coupling between the C++ and JavaScript portions of the layout code, we need to pass certain layout "options" (such as the default page orientation) back and forth by value. As a result, it makes sense to have a separate Options class to hold these settings, which then can be used to recalculate the layout. The expected layout lifecycle eventually should look like this: 1. Update desired layout options one or more times. 2. Estimate layout size with new layout options. 3. Send options and estimated size to JavaScript. 4. Receive updated options from JavaScript. 5. Apply final options to current layout. 6. Repaint using updated layout. As a side effect of this change, I added the desired_layout_options_ field to PDFiumEngine. This new field holds the mutable state about page orientation that was split off into DocumentLayout::Options. Miscellaneous changes: * Revised some of the API comments for DocumentLayout * Eliminated need for unsigned math by replacing -1 with +3 (mod 4) * DocumentLayout no longer needs to be copyable, so it isn't Bug: 885110 Change-Id: I89714c4c2cba6c0ab354c0696ec3943327c979d0 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1733795 Commit-Queue: K Moon <kmoon@chromium.org> Reviewed-by:
Lei Zhang <thestig@chromium.org> Cr-Commit-Position: refs/heads/master@{#684441}
-
John Abd-El-Malek authored
Bug: 934009 Change-Id: I859a008b9a68313d3d0479225d20b2b23acf4de4 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1738667 Commit-Queue: John Abd-El-Malek <jam@chromium.org> Auto-Submit: John Abd-El-Malek <jam@chromium.org> Reviewed-by:
Andrey Kosyakov <caseq@chromium.org> Cr-Commit-Position: refs/heads/master@{#684440}
-
chromium-autoroll authored
https://android.googlesource.com/platform/external/perfetto.git/+log/600679a1c326..4ec51d883391 git log 600679a1c326..4ec51d883391 --date=short --no-merges --format='%ad %ae %s' 2019-08-06 fmayer@google.com Merge "Add nogncheck to conditional include." Created with: gclient setdep -r src/third_party/perfetto@4ec51d883391 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: I7eaa4ec19b4d18f4a3dc97c6ee813c9a9953c99d Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1738999Reviewed-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@{#684439}
-
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. R=battre@chromium.org Bug: 968047 Change-Id: Ia14e3392662a15f96dc0675265f93397cf5a22f4 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1739813 Auto-Submit: Sami Kyöstilä <skyostil@chromium.org> Reviewed-by:
Dominic Battré <battre@chromium.org> Commit-Queue: Dominic Battré <battre@chromium.org> Cr-Commit-Position: refs/heads/master@{#684438}
-
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. R=rbpotter@chromium.org Bug: 968047 Change-Id: I5ec1af17a9d1408c6dc656abc9c44a5fcd47bb74 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1739866 Auto-Submit: Sami Kyöstilä <skyostil@chromium.org> Reviewed-by:
Lei Zhang <thestig@chromium.org> Commit-Queue: Lei Zhang <thestig@chromium.org> Cr-Commit-Position: refs/heads/master@{#684437}
-
chrome://omniboxmanuk authored
Whitespace is important for inline autocomplete, and possibly other fields. Change-Id: I8dfcb1f2140e20b6d3365f72b9fd832b8eba9303 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1739325Reviewed-by:
Tommy Li <tommycli@chromium.org> Commit-Queue: manuk hovanesian <manukh@chromium.org> Cr-Commit-Position: refs/heads/master@{#684436}
-
chromium-autoroll authored
https://chromium.googlesource.com/angle/angle.git/+log/99f494e42246..6201d134b3bb git log 99f494e42246..6201d134b3bb --date=short --no-merges --format='%ad %ae %s' 2019-08-06 timvp@google.com Vulkan: Suppress KHR-GLES2 ASTC 3D Texture Tests Created with: gclient setdep -r src/third_party/angle@6201d134b3bb 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: None Change-Id: I8fb2d5ea7dec802417c8968d3b8f971a6a5a82f7 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1738997Reviewed-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@{#684435}
-
chromium-internal-autoroll authored
https://chrome-internal.googlesource.com/chrome/src-internal.git/+log/fc9e4928e7c0..838e9713fa32 Created with: gclient setdep -r src-internal@838e9713fa32 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,chromium:None,chromium:None Change-Id: Ia2e0cb8a2f15aa728c6eb0a28232c634690c7d5f Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1739002Reviewed-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@{#684434}
-
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. R=tbansal@chromium.org Bug: 968047 Change-Id: Ib877f88fa151617c51533a56adfa399706db5b0f Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1739708 Auto-Submit: Sami Kyöstilä <skyostil@chromium.org> Reviewed-by:
Tarun Bansal <tbansal@chromium.org> Commit-Queue: Tarun Bansal <tbansal@chromium.org> Cr-Commit-Position: refs/heads/master@{#684433}
-
redatawfik authored
1)Checks if there is any saved credit card with the same number. 2)If there is any, then update saved credit card object with the new data update it in DB by PersonalDataManager. 3)Adds method for updating credit card properties. Change-Id: Ife5efb47a0bee6439ae05981116bc891b7289f7e Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1731839 Commit-Queue: Reda Tawfik <redatawfik@google.com> Reviewed-by:
Sergio Collazos <sczs@chromium.org> Reviewed-by:
Javier Ernesto Flores Robles <javierrobles@chromium.org> Cr-Commit-Position: refs/heads/master@{#684432}
-
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. R=tommycli@chromium.org Bug: 968047 Change-Id: Id69af8c66a5b886be639f1f22c0dc027a49e7758 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1738702 Auto-Submit: Sami Kyöstilä <skyostil@chromium.org> Reviewed-by:
Tommy Li <tommycli@chromium.org> Commit-Queue: Sami Kyöstilä <skyostil@chromium.org> Cr-Commit-Position: refs/heads/master@{#684431}
-
John Abd-El-Malek authored
Bug: 934009 Change-Id: If1af2ec4d66ead9c3fa2f7f36ccabfb726bbec7d Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1739110 Commit-Queue: John Abd-El-Malek <jam@chromium.org> Commit-Queue: Clark DuVall <cduvall@chromium.org> Auto-Submit: John Abd-El-Malek <jam@chromium.org> Reviewed-by:
Clark DuVall <cduvall@chromium.org> Cr-Commit-Position: refs/heads/master@{#684430}
-
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. R=rouslan@chromium.org Bug: 968047 Change-Id: I695547fcd8f3859e16252bf24b8e78d6374ede4c Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1739867 Auto-Submit: Sami Kyöstilä <skyostil@chromium.org> Reviewed-by:
Rouslan Solomakhin <rouslan@chromium.org> Commit-Queue: Sami Kyöstilä <skyostil@chromium.org> Cr-Commit-Position: refs/heads/master@{#684429}
-
Benjamin Beaudry authored
The problem was here: for (size_t i = children().size() - 1; i >= 0; --i) { The loop variable |size_t i| was of an unsigned int type, so |i >= 0| was always true. Repro on Windows by exploring the tree with Inspect on the link provided in the crbug description. Bug: 988153 Change-Id: I1cae093da68f12ed1630a568ec7529703c73cf89 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1737870 Commit-Queue: Benjamin Beaudry <benjamin.beaudry@microsoft.com> Reviewed-by:
Aaron Leventhal <aleventhal@chromium.org> Reviewed-by:
Nektarios Paisios <nektar@chromium.org> Cr-Commit-Position: refs/heads/master@{#684428}
-
Michael Thiessen authored
This is a speculative fix. Bug: 979565 Change-Id: I140d9a3fa2334f7137e1491c40c61d96bcded8b1 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1729922 Commit-Queue: Michael Thiessen <mthiesse@chromium.org> Reviewed-by:
David Bokan <bokan@chromium.org> Cr-Commit-Position: refs/heads/master@{#684427}
-
Ian Kilpatrick authored
Patch: https://chromium-review.googlesource.com/c/chromium/src/+/1692627 Made us consider inline-level OOF-positioned nodes as adjoining, as their static-position depends on where other floats are placed. During the first pass the BFC block-offset gets resolved here: https://cs.chromium.org/chromium/src/third_party/blink/renderer/core/layout/ng/ng_block_layout_algorithm.cc?l=986&rcl=e6157bb7afb778d5c15bb2bb09a457d9632151da This aborts as it has an adjoining inline-OOF, and *doesn't* resolve the BFC block-offset at |child_bfc_offset_estimate| instead applying clearance. Upon the second pass we now hit the DCHECK as neither of the options has clearance applied. Bug: 987004 Change-Id: I2634d56726a4664c869b2ecfa1e7a8a13458e78f Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1731917 Commit-Queue: Ian Kilpatrick <ikilpatrick@chromium.org> Reviewed-by:
Morten Stenshorne <mstensho@chromium.org> Cr-Commit-Position: refs/heads/master@{#684426}
-
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. R=zmin@chromium.org Bug: 968047 Change-Id: I5af561f465c55acc9b49898abe8051f475f1f873 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1739826 Auto-Submit: Sami Kyöstilä <skyostil@chromium.org> Reviewed-by:
Owen Min <zmin@chromium.org> Commit-Queue: Owen Min <zmin@chromium.org> Cr-Commit-Position: refs/heads/master@{#684425}
-
Tiancong Wang authored
This is another manual update on the Chrome OS orderfile name, to reflect the recent change on the orderfile name format. This is another preparation for the autoroller. BUG=chromium:950627 TEST=The orderfile is tested on Chrome OS. No code is using the orderfile for now Change-Id: If2a88ec35d89e397c00904f8b5fc093cc7b77dfa Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1739117 Auto-Submit: Tiancong Wang <tcwang@google.com> Reviewed-by:
Eric Boren <borenet@chromium.org> Commit-Queue: Eric Boren <borenet@chromium.org> Cr-Commit-Position: refs/heads/master@{#684424}
-
Abhijeet Kandalkar authored
The primary goal of this CL is to use IsA<HTMLDirectoryElement>(element) in place of IsHTMLDirectoryElement(element) Bug: 891908 Change-Id: Iab622a1653b9b24d27894bf2ede0ba18a59daf61 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1739487Reviewed-by:
Kentaro Hara <haraken@chromium.org> Commit-Queue: Abhijeet Kandalkar <abhijeet@igalia.com> Cr-Commit-Position: refs/heads/master@{#684423}
-
Aaron Gable authored
Bug: 982122 Change-Id: I9dbf678992caec7ac97ef028e9ec16c2da7db02c Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1735523Reviewed-by:
Dan Sanders <sandersd@chromium.org> Reviewed-by:
Ben Pastene <bpastene@chromium.org> Commit-Queue: Aaron Gable <agable@chromium.org> Cr-Commit-Position: refs/heads/master@{#684422}
-
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. R=rsesek@chromium.org Bug: 968047 Change-Id: Ia108448a6cc6ffdcee488ef32c4d9fa77f3839a4 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1739707 Auto-Submit: Sami Kyöstilä <skyostil@chromium.org> Reviewed-by:
Robert Sesek <rsesek@chromium.org> Commit-Queue: Sami Kyöstilä <skyostil@chromium.org> Cr-Commit-Position: refs/heads/master@{#684421}
-
edchin authored
Updates the expiry of these histograms, which are still useful to keep around. IOS.EnterTabSwitcherSnapshotResult IOS.PageLoadedSnapshotResult Bug: 974919 Change-Id: I2c5b8f20c02f89c76200a18e53f9c094dda3b81e Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1737898Reviewed-by:
edchin <edchin@chromium.org> Reviewed-by:
Nik Bhagat <nikunjb@chromium.org> Auto-Submit: edchin <edchin@chromium.org> Commit-Queue: edchin <edchin@chromium.org> Cr-Commit-Position: refs/heads/master@{#684420}
-
Alexey Baskakov authored
A writer of a test will see a callstack anyway. Bug: 877898 Change-Id: I0a77bf5374d59de3eb1249d89db0be8852b16fa0 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1738366Reviewed-by:
Eric Willigers <ericwilligers@chromium.org> Commit-Queue: Eric Willigers <ericwilligers@chromium.org> Cr-Commit-Position: refs/heads/master@{#684419}
-
Karan Bhatia authored
It should never be null. BUG=None Change-Id: Ic1e72b42e4664177cd42af61d38caa4930c5713c Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1737741 Commit-Queue: Devlin <rdevlin.cronin@chromium.org> Auto-Submit: Karan Bhatia <karandeepb@chromium.org> Reviewed-by:
Devlin <rdevlin.cronin@chromium.org> Cr-Commit-Position: refs/heads/master@{#684418}
-
chrome-bot authored
Bug: 967779 Change-Id: I456c2218ca23d9cd4878588f94a82410bfbf056e Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1692980 Auto-Submit: Mathieu Binette <mbinette@google.com> Reviewed-by:
Owen Min <zmin@chromium.org> Commit-Queue: Owen Min <zmin@chromium.org> Cr-Commit-Position: refs/heads/master@{#684417}
-
Adam Langley authored
r684202 set the wrong object member. (This fix is equally untested because I have to wait for Canary in order to try it.) BUG=990992 Change-Id: I140d081e53f2d10070a35cff8af312db7f3c77d8 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1739395 Commit-Queue: Adam Langley <agl@chromium.org> Commit-Queue: Hector Carmona <hcarmona@chromium.org> Auto-Submit: Adam Langley <agl@chromium.org> Reviewed-by:
Hector Carmona <hcarmona@chromium.org> Cr-Commit-Position: refs/heads/master@{#684416}
-
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. R=thestig@chromium.org Bug: 968047 Change-Id: Ie4c905f2e8a679550fe00953a010c62ceaa9b806 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1739709 Auto-Submit: Sami Kyöstilä <skyostil@chromium.org> Reviewed-by:
Lei Zhang <thestig@chromium.org> Commit-Queue: Lei Zhang <thestig@chromium.org> Commit-Queue: Sami Kyöstilä <skyostil@chromium.org> Cr-Commit-Position: refs/heads/master@{#684415}
-
Varun Khaneja authored
R=drubery TBR=holte (since http://cl/261944627) Bug: 965587 Change-Id: Ic8a8c8a05be012bae3bf375d511399eba47c1ed5 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1739588 Commit-Queue: Varun Khaneja <vakh@chromium.org> Auto-Submit: Varun Khaneja <vakh@chromium.org> Reviewed-by:
Daniel Rubery <drubery@chromium.org> Cr-Commit-Position: refs/heads/master@{#684414}
-
Gayane Petrosyan authored
Set picker value to the exact value that user selected instead of the darkest of the calculated colors. These 2 colors are similar but not always the same. Therefor picker value to the exact value so that selected color doesn't jump after OnThemeUpdate. Bug: 990379 Change-Id: Iac9ddae122be5daf01ec95bca571ec832aa5a981 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1737330 Commit-Queue: Gayane Petrosyan <gayane@chromium.org> Reviewed-by:
Kristi Park <kristipark@chromium.org> Reviewed-by:
Mustafa Emre Acer <meacer@chromium.org> Cr-Commit-Position: refs/heads/master@{#684413}
-
Ella Ge authored
With flag kConsolidatedMovementXY enabled, we compare the incoming event's coordinates with the |synthetic_move_position_| to tell if the event is a synthetized move that comes from |MoveCursorTo| call. The check didn't consider the error introduced by converting from pixel to dip, which causes a failure when there is a fractional device_scale_factor. This CL makes the check use the same logic as w/o kConsolidatedMovementXY. This also removes |synthetic_move_sent_| boolean and use |synthetic_move_position_| for both the flag enabled/disabled. Bug: 802067 Change-Id: I4847397e317d29b62328d02910b7cb3e8de13759 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1704743Reviewed-by:
Ken Buchanan <kenrb@chromium.org> Reviewed-by:
Mustaq Ahmed <mustaq@chromium.org> Commit-Queue: Ella Ge <eirage@chromium.org> Cr-Commit-Position: refs/heads/master@{#684412}
-
Sadrul Habib Chowdhury authored
BUG=984159 Change-Id: I18d9882adc86ad163a7ef161bd8f48ee2484f80e Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1739113 Auto-Submit: Sadrul Chowdhury <sadrul@chromium.org> Reviewed-by:
Caleb Rouleau <crouleau@chromium.org> Commit-Queue: Caleb Rouleau <crouleau@chromium.org> Cr-Commit-Position: refs/heads/master@{#684411}
-
Ben Kelly authored
This is a reland of 053b22e8 The CacheStorageManagerTest.TestErrorInitializingCache test has been modified to reduce flakiness. Original change's description: > CacheStorage: Write index after simple disk_cache. > > When we load the cache_storage index we try to determine if its fresh > or stale by comparing its timestamp against the timestamp of the > various simple disk_cache directories. Both cache_storage and simple > disk_cache write their index files out using a delay. The > cache_storage delay is 5 seconds and the simple disk_cache delay is 20 > seconds. This means that the simple disk_cache index is always written > last and as a result the cache_storage index is almost always > considered stale. > > This CL fix this by extending the cache_storage delay to match the > simple disk_cache delay. It also implements a shorter delay on android > when in the background like simple disk_cache uses to avoid losing > data when the app is killed by the operating system. Finally, the CL > also now properly flushes the index to disk if there is a pending write > when the cache_storage subsystem is torn down. > > Over time this should cause ServiceWorkerCache.UsedIndexFileSize to > show more cases where the index file is used to provide the size. > > Bug: 988001 > Change-Id: Idaa09b956edcfa9e70ac1fdb9b43adcd73ec0508 > Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1721389 > Commit-Queue: Daniel Murphy <dmurph@chromium.org> > Reviewed-by: Daniel Murphy <dmurph@chromium.org> > Cr-Commit-Position: refs/heads/master@{#682409} Bug: 988001 Change-Id: I12aa006bf1d18be5ae121c0330d97e51a97ff906 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1739408Reviewed-by:
Daniel Murphy <dmurph@chromium.org> Commit-Queue: Ben Kelly <wanderview@chromium.org> Cr-Commit-Position: refs/heads/master@{#684410}
-
chromium-autoroll authored
https://swiftshader.googlesource.com/SwiftShader.git/+log/060fcf777159..3aec8a3be749 git log 060fcf777159..3aec8a3be749 --date=short --no-merges --format='%ad %ae %s' 2019-08-06 chrisforbes@google.com Tidy around sampler handling Created with: gclient setdep -r src/third_party/swiftshader@3aec8a3be749 The AutoRoll server is located here: https://autoroll.skia.org/r/swiftshader-chromium-autoroll Documentation for the AutoRoller is here: https://skia.googlesource.com/buildbot/+/master/autoroll/README.md If the roll is causing failures, please contact the current sheriff, who should be CC'd on the roll, and stop the roller if necessary. CQ_INCLUDE_TRYBOTS=luci.chromium.try:android_optional_gpu_tests_rel;luci.chromium.try:linux_optional_gpu_tests_rel;luci.chromium.try:mac_optional_gpu_tests_rel;luci.chromium.try:win_optional_gpu_tests_rel TBR=swiftshader-team+autoroll@google.com Bug: chromium:b/134584057 Change-Id: I3b892e49f28ce5261a3ad54b056c681e8e1012bc Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1738992Reviewed-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@{#684409}
-
Rayan Kanso authored
UKM collection review doc (Google-only): https://docs.google.com/document/d/17roFDpjKXeglB9_dzfX7uGykoWeRCVTa_4SDDJyv04Y/ Bug: 973844 Change-Id: I2a8d36286ff7b0472c64313b222f7404f88e241e Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1724077Reviewed-by:
Robert Kaplow <rkaplow@chromium.org> Reviewed-by:
Richard Knoll <knollr@chromium.org> Commit-Queue: Rayan Kanso <rayankans@chromium.org> Cr-Commit-Position: refs/heads/master@{#684408}
-
Kristi Park authored
- Move order of menu header items (e.g. back button and daily toggle) so that they follow the submenu nav buttons using keyboard navigation. - Support selection on SPACE for color/background tiles. - Like backgrounds/colors, use arrowkeys instead of TAB to navigate the shortcut options (i.e. TAB to focus the shortcuts menu, then use arrowkeys to move between the shortcut options). - Add role=button to upload & default background tiles for screenreader support. Bug: 953822 Change-Id: Iae8c1b8390d917f1513e234b46f0b75cc9bae3ee Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1737743 Commit-Queue: Kristi Park <kristipark@chromium.org> Reviewed-by:
Gayane Petrosyan <gayane@chromium.org> Cr-Commit-Position: refs/heads/master@{#684407}
-
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=eladalon@chromium.org Bug: 968047 Change-Id: I32f6d432ffa2550f58d433775e0adef216cf9ad5 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1738593 Auto-Submit: Sami Kyöstilä <skyostil@chromium.org> Reviewed-by:
Elad Alon <eladalon@chromium.org> Commit-Queue: Sami Kyöstilä <skyostil@chromium.org> Cr-Commit-Position: refs/heads/master@{#684406}
-
chromium-autoroll authored
https://chromium.googlesource.com/chromium/tools/depot_tools.git/+log/36756e459041..c5f95c03ada3 git log 36756e459041..c5f95c03ada3 --date=short --no-merges --format='%ad %ae %s' 2019-08-06 recipe-mega-autoroller@chops-service-accounts.iam.gserviceaccount.com Roll recipe dependencies (trivial). Created with: gclient setdep -r src/third_party/depot_tools@c5f95c03ada3 The AutoRoll server is located here: https://autoroll.skia.org/r/depot-tools-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=agable@chromium.org Bug: None Change-Id: Id342cf080995fdd38d5ece41d466daca5b6ea30f Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1738995Reviewed-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@{#684405}
-