- 19 Aug, 2019 40 commits
-
-
Alison Maher authored
fast/css/forced-colors-mode/forced-colors-mode-12.html was detected as flaky. This tests animations in forced colors mode. Looking at other animation tests, it looks like the issue was that I needed to add "media="screen"" to the style element. Bug: 994465 Change-Id: I982343bf05e3aa4ac4df2caab0373103f5165cb4 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1758706Reviewed-by:
Rune Lillesveen <futhark@chromium.org> Commit-Queue: Alison Maher <almaher@microsoft.com> Cr-Commit-Position: refs/heads/master@{#688316}
-
Eric Foo authored
This is CL is generated by a script. Data is pulled from review conducted in Q2 2019. See https://tinyurl.com/yxau5sc4 for context. CR+1 from all parties. NOPRESUBMIT=true Set due to folks in OWNERS files lacking permissions to LGTM. Bug: 977050 Change-Id: I3f82b04e138e65cd808670db46ab82a48dd580b0 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1700855 Commit-Queue: Eric Foo <efoo@chromium.org> Reviewed-by:
Michael Case <mikecase@chromium.org> Reviewed-by:
Jan Krcal <jkrcal@chromium.org> Auto-Submit: Eric Foo <efoo@chromium.org> Cr-Commit-Position: refs/heads/master@{#688315}
-
Eric Foo authored
This is CL is generated by a script. Data is pulled from review conducted in Q2 2019. See https://tinyurl.com/yxau5sc4 for context. CR+1 from all parties. NOPRESUBMIT=true Set due to folks in OWNERS files lacking permissions to LGTM. Bug: 977050 Change-Id: I17a6e403810ad7c2e1c011e561e7fce832ee541a Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1700912Reviewed-by:
Xianzhu Wang <wangxianzhu@chromium.org> Reviewed-by:
Kristi Park <kristipark@chromium.org> Commit-Queue: Eric Foo <efoo@chromium.org> Auto-Submit: Eric Foo <efoo@chromium.org> Cr-Commit-Position: refs/heads/master@{#688314}
-
Charlene Yan authored
This mirror's MenuButton's implementation. Bug: 995182 Change-Id: I98f64d067980aed77a666fd8491c52b49d234da4 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1760143 Commit-Queue: Charlene Yan <cyan@chromium.org> Reviewed-by:
Peter Boström <pbos@chromium.org> Cr-Commit-Position: refs/heads/master@{#688313}
-
Alex Newcomer authored
Fix for a crash that occurs while the AppList is closing. We assume the layer exists, yet a few optimizations have been implemented which may result in layers being destroyed when they are no longer needed. Bug: 994409 Change-Id: I1d99f34c32e18ef70a3784b1e5337c1530cba872 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1761308 Auto-Submit: Alex Newcomer <newcomer@chromium.org> Reviewed-by:
Jun Mukai <mukai@chromium.org> Commit-Queue: Alex Newcomer <newcomer@chromium.org> Cr-Commit-Position: refs/heads/master@{#688312}
-
chromium-autoroll authored
https://skia.googlesource.com/skia.git/+log/eb3eb98eb6b8..0d6f77537a4c git log eb3eb98eb6b8..0d6f77537a4c --date=short --no-merges --format='%ad %ae %s' 2019-08-19 csmartdalton@google.com Add GrvkGpuRTCommandBuffer::appendSampledTexture 2019-08-19 brianosman@google.com Temporary Lua hook to measure impact of conservative convexity 2019-08-19 recipe-mega-autoroller@chops-service-accounts.iam.gserviceaccount.com Roll recipe dependencies (trivial). Created with: gclient setdep -r src/third_party/skia@0d6f77537a4c 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=egdaniel@google.com Bug: None Change-Id: Ief5f4886603b66ca5b5bc619fb6218f0dfd00bec Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1761196Reviewed-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@{#688311}
-
Simon Que authored
This was a typo that resulted in empty Milo UI entries: https://ci.chromium.org/p/chromium/g/chromium.goma.migration/console R=bpastene@chromium.org, jbudorick@chromium.org, tikuta@chromium.org Bug: 950413 Change-Id: I206fa2f090299f563ce62da6e54c1b711ee871c5 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1761342 Auto-Submit: Simon Que <sque@chromium.org> Reviewed-by:
Takuto Ikuta <tikuta@chromium.org> Reviewed-by:
John Budorick <jbudorick@chromium.org> Commit-Queue: Takuto Ikuta <tikuta@chromium.org> Cr-Commit-Position: refs/heads/master@{#688310}
-
chromium-autoroll authored
https://chromium.googlesource.com/chromiumos/chromite.git/+log/d272ed1e68f8..199e517c52c6 git log d272ed1e68f8..199e517c52c6 --date=short --no-merges --format='%ad %ae %s' 2019-08-19 akeshet@chromium.org test_stages: fix temporary hack for moblab boards Created with: gclient setdep -r src/third_party/chromite@199e517c52c6 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. CQ_INCLUDE_TRYBOTS=luci.chromium.try:chromeos-kevin-rel TBR=chrome-os-gardeners@google.com Bug: None Change-Id: I283fbbee90d450c4a1698f8449224f246140e0bd Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1761197Reviewed-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@{#688309}
-
Matt Falkenhagen authored
The thread ServiceWorkerContextCore lives on (the "core thread") will move from the IO thread to the UI thread. This makes service_worker_browsertest.cc core thread-aware instead of assuming the IO thread. Some tests start to pass when the flag is enabled locally. After this, the plan is to fix tests that fail. Bug: 824858 Change-Id: Id7032f875c068e4a5af2ea620d7bdaaa9146e7b2 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1760573 Commit-Queue: Kenichi Ishibashi <bashi@chromium.org> Reviewed-by:
Kenichi Ishibashi <bashi@chromium.org> Cr-Commit-Position: refs/heads/master@{#688308}
-
Eric Foo authored
This is CL is generated by a script. Data is pulled from review conducted in Q2 2019. See https://tinyurl.com/yxau5sc4 for context. Bug: 977050 Change-Id: Ib86df4c2ffa933a36af6abf86437956c418e1818 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1700913Reviewed-by:
Chris Palmer <palmer@chromium.org> Reviewed-by:
Abhishek Arya <inferno@chromium.org> Reviewed-by:
Kinuko Yasuda <kinuko@chromium.org> Commit-Queue: Eric Foo <efoo@chromium.org> Auto-Submit: Eric Foo <efoo@chromium.org> Cr-Commit-Position: refs/heads/master@{#688307}
-
Devlin Cronin authored
Currently, URLPattern will trim the host piece from whitespace. However, this doesn't really make sense: - It doesn't handle whitespace within host components - It doesn't handle other invalid host characters Rather than do this, just rely on the host canonicalization (which is done through net::CanonicalizeHost()). Note that GURL (and net::CanonicalizeHost()) both allow various host inputs which would normally be invalid, include spaces (which are escaped to %20, but '%' is also an invalid character in a host). However, this is better than arbitrarily treating whitespace differently. Bug: None Change-Id: Ia32c26071e6afdd35386775b345a658937acbe0e Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1749579 Commit-Queue: Devlin <rdevlin.cronin@chromium.org> Reviewed-by:
Karan Bhatia <karandeepb@chromium.org> Cr-Commit-Position: refs/heads/master@{#688306}
-
Kevin Marshall authored
Allows embedders to handle popup window requests from web content (e.g. "window.open", navigation to links with target="_blank"). Bug: 888131 Change-Id: I7fef0525b6b043494cd462deb66f854e657af670 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1548586 Commit-Queue: Kevin Marshall <kmarshall@chromium.org> Reviewed-by:
Wez <wez@chromium.org> Cr-Commit-Position: refs/heads/master@{#688305}
-
Eric Foo authored
This is CL is generated by a script. Data is pulled from review conducted in Q2 2019. See https://tinyurl.com/yxau5sc4 for context. Bug: 977050 Change-Id: I81ceb84acc15c073b92c2d24531df7802bd9ba49 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1699949Reviewed-by:
Ryan Sturm <ryansturm@chromium.org> Reviewed-by:
Vlad Tsyrklevich <vtsyrklevich@chromium.org> Reviewed-by:
Yuki Shiino <yukishiino@chromium.org> Commit-Queue: Ryan Sturm <ryansturm@chromium.org> Auto-Submit: Eric Foo <efoo@chromium.org> Cr-Commit-Position: refs/heads/master@{#688304}
-
Findit authored
This reverts commit 3ea87442. Reason for revert: Findit (https://goo.gl/kROfz5) identified CL at revision 688173 as the culprit for failures in the build cycles as shown on: https://analysis.chromium.org/waterfall/culprit?key=ag9zfmZpbmRpdC1mb3ItbWVyRAsSDVdmU3VzcGVjdGVkQ0wiMWNocm9taXVtLzNlYTg3NDQyYmY1MTk4Mzc1ZWQzODU2MmEwYTZhMTEzOTI0M2QwNjIM Sample Failed Build: https://ci.chromium.org/buildbot/chromium.memory/Mac%20ASan%2064%20Tests%20%281%29/55899 Sample Failed Step: browser_tests Original change's description: > Better success detection for pw change > > Only slightly improves success detection at this stage, > by adding Azure support - Ping and Okta are still pretty > lacking - but makes room in the code for better success > detection, and adds a unit test. > > Bug: 930109 > Change-Id: I59a173095301f7c641031f79cb3a6c946c5efc09 > Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1754013 > Reviewed-by: Xiyuan Xia <xiyuan@chromium.org> > Reviewed-by: Lutz Justen <ljusten@chromium.org> > Commit-Queue: A Olsen <olsen@chromium.org> > Cr-Commit-Position: refs/heads/master@{#688173} Change-Id: I2e20b20503381c181ed94b133a4099c70e5104fa No-Presubmit: true No-Tree-Checks: true No-Try: true Bug: 930109 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1761344 Cr-Commit-Position: refs/heads/master@{#688303}
-
Antonio Gomes authored
This CL addressed guidou's remark on [1], that was accidentally left off when splitting the CL into smaller pieces. [1] https://crrev.com/c/1757490/4/third_party/blink/renderer/modules/modules_initializer.cc#185 BUG=704136 R=guidou@chromium.org, haraken@chromium.org Change-Id: Ie46cc5eeb9963dc54538d122b0c922a5ec5c4e3c Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1760407Reviewed-by:
Guido Urdaneta <guidou@chromium.org> Reviewed-by:
Kentaro Hara <haraken@chromium.org> Commit-Queue: Kentaro Hara <haraken@chromium.org> Auto-Submit: Antonio Gomes <tonikitoo@igalia.com> Cr-Commit-Position: refs/heads/master@{#688302}
-
Eric Foo authored
This is CL is generated by a script. Data is pulled from review conducted in Q2 2019. See https://tinyurl.com/yxau5sc4 for context. Bug: 977050 Change-Id: Ie7f2051e4b6c9c4af7880597550949c02162d5a0 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1700854Reviewed-by:
Jinsuk Kim <jinsukkim@chromium.org> Reviewed-by:
Reilly Grant <reillyg@chromium.org> Reviewed-by:
Greg Thompson <grt@chromium.org> Commit-Queue: Eric Foo <efoo@chromium.org> Auto-Submit: Eric Foo <efoo@chromium.org> Cr-Commit-Position: refs/heads/master@{#688301}
-
Reilly Grant authored
This adds code to recognize platform serial ports which aren't on a particular bus like USB. They are identifiable because they end up in the /devices/platform namespace. Bug: 981483 Change-Id: I95613d57fd81a1edd060b140fd9b47f5297037b6 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1760986Reviewed-by:
Ovidio de Jesús Ruiz-Henríquez <odejesush@chromium.org> Commit-Queue: Reilly Grant <reillyg@chromium.org> Cr-Commit-Position: refs/heads/master@{#688300}
-
Simon Que authored
This reverts commit 5e608df4. Reason for revert: The problem was a typo ("dEQP" not "eDQP"), not a missing builder Original change's description: > infra: Delete eDQP builder Milo entries > > These builders no longer exist. The Milo UI entries are blank: > https://ci.chromium.org/p/chromium/g/chromium.goma.migration/console > > R=bpastene@chromium.org, jbudorick@chromium.org, tikuta@chromium.org > > Bug: 950413 > Change-Id: I362d1e4260f949c9ce32f228828ffb6ed0910f8b > Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1761332 > Reviewed-by: John Budorick <jbudorick@chromium.org> > Reviewed-by: Takuto Ikuta <tikuta@chromium.org> > Commit-Queue: Simon Que <sque@chromium.org> > Cr-Commit-Position: refs/heads/master@{#688292} TBR=sque@chromium.org,tikuta@chromium.org,bpastene@chromium.org,jbudorick@chromium.org Change-Id: Ib5531f38fceaf47211eb8409ea9309198a462474 No-Presubmit: true No-Tree-Checks: true No-Try: true Bug: 950413 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1761406Reviewed-by:
Simon Que <sque@chromium.org> Commit-Queue: Simon Que <sque@chromium.org> Cr-Commit-Position: refs/heads/master@{#688299}
-
Kenneth Russell authored
Geoff has nearly single-handedly developed the passthrough command decoder and is well versed in the code in this directory. Bug: none Change-Id: Ic2e87bd485b4274270abbd562a729b84d3637c63 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1758737Reviewed-by:
Zhenyao Mo <zmo@chromium.org> Reviewed-by:
Victor Miura <vmiura@chromium.org> Commit-Queue: Kenneth Russell <kbr@chromium.org> Cr-Commit-Position: refs/heads/master@{#688298}
-
Hesen Zhang authored
- Moved IconConverter ownership to IconStore. - Added MockIconConverter in the unittest for future use. - Cleaned-up all other legacy icon_converter uses in codebase. Bug: 963290 Change-Id: I3ae60dcc8df936033d1c72eb806d54f485f0519a Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1759206 Commit-Queue: Hesen Zhang <hesen@chromium.org> Reviewed-by:
David Trainor <dtrainor@chromium.org> Reviewed-by:
Xing Liu <xingliu@chromium.org> Cr-Commit-Position: refs/heads/master@{#688297}
-
Tomasz Wiszkowski authored
This fixes Finch presubmit warning for GroupSuggestionsBySearchVsUrl experiment. Bug: 990865 Change-Id: Id294e8de22098e7d5750fb393f6f7ed158ad12c3 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1760459Reviewed-by:
Ender <ender@google.com> Reviewed-by:
Jesse Doherty <jwd@chromium.org> Auto-Submit: Ender <ender@google.com> Commit-Queue: Jesse Doherty <jwd@chromium.org> Cr-Commit-Position: refs/heads/master@{#688296}
-
Theresa Wellington authored
Bug: 994293 Change-Id: I2446ed845f08b5173ac25fe836e50b114f58cf45 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1756545 Commit-Queue: Theresa <twellington@chromium.org> Reviewed-by:
David Trainor <dtrainor@chromium.org> Cr-Commit-Position: refs/heads/master@{#688295}
-
Michael Thiessen authored
When we get asked to close a tab, actually close the tab before finishing the Activity for tests that test that a tab actually closes (and proactively defend against tabs that were closed sticking around after relaunching). Bug: 983743 Change-Id: I61771961edd8524b851bffc741a9b37e28e524e0 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1758629 Commit-Queue: Michael Thiessen <mthiesse@chromium.org> Reviewed-by:
Yaron Friedman <yfriedman@chromium.org> Cr-Commit-Position: refs/heads/master@{#688294}
-
Manu Cornet authored
Add a test to prevent future regressions. BUG=991919 Change-Id: I41ebd4f5cee6375f0cb2d510b8d4e91d356a1bc0 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1761226Reviewed-by:
Xiyuan Xia <xiyuan@chromium.org> Commit-Queue: Manu Cornet <manucornet@chromium.org> Cr-Commit-Position: refs/heads/master@{#688293}
-
Simon Que authored
These builders no longer exist. The Milo UI entries are blank: https://ci.chromium.org/p/chromium/g/chromium.goma.migration/console R=bpastene@chromium.org, jbudorick@chromium.org, tikuta@chromium.org Bug: 950413 Change-Id: I362d1e4260f949c9ce32f228828ffb6ed0910f8b Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1761332Reviewed-by:
John Budorick <jbudorick@chromium.org> Reviewed-by:
Takuto Ikuta <tikuta@chromium.org> Commit-Queue: Simon Que <sque@chromium.org> Cr-Commit-Position: refs/heads/master@{#688292}
-
chromium-internal-autoroll authored
https://chrome-internal.googlesource.com/chrome/src-internal.git/+log/7dcd6b790228..1c3b93db4826 Created with: gclient setdep -r src-internal@1c3b93db4826 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 Change-Id: Id327494c09eae96e9c3ba79cb5621538f88f2747 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1761195Reviewed-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@{#688291}
-
Ted Choc authored
BUG=992838 Change-Id: Idc6ab38c6f40224188312f9f6804857fcf48828f Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1757299 Commit-Queue: Ted Choc <tedchoc@chromium.org> Reviewed-by:
Ender <ender@google.com> Cr-Commit-Position: refs/heads/master@{#688290}
-
Robert Sesek authored
- chrome_content_browser_client.cc: Include the --disk-cache-dir switch value when computing GetNetworkContextsParentDirectory(). - network_quality_estimator_prefs_browsertest.cc: Create the network state file in the profile directory, rather than a temp directory, since the network service sandbox does not permit accessing the latter. - service_process_control_browsertest.cc: Only override the child process path when launching the service process, so that the path override does not affect other child process types. - shell_browser_main_parts_mac.mm: Expand the sandbox for the //net test certificates when running the shell outside of browser tests. - shell_content_browser_client.cc: Implement GetNetworkContextsParentDirectory() for Content Shell's profile. Bug: 915910 Change-Id: Ifbe481abdfed9d00c694865282ba316033749867 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1756153Reviewed-by:
Avi Drissman <avi@chromium.org> Commit-Queue: Robert Sesek <rsesek@chromium.org> Cr-Commit-Position: refs/heads/master@{#688289}
-
Takuto Ikuta authored
For https://chromium-review.googlesource.com/c/chromium/src/+/1755125 Bug: 876922 Change-Id: I944eb771d41b4aa93db271a58eff025139d64aea Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1757956 Commit-Queue: John Budorick <jbudorick@chromium.org> Reviewed-by:
John Budorick <jbudorick@chromium.org> Auto-Submit: Takuto Ikuta <tikuta@chromium.org> Cr-Commit-Position: refs/heads/master@{#688288}
-
siyua authored
Strings provided in go/auth-cm Bug: 991037 Change-Id: I8e64c447484aee2d03038fa2385cbcc174797608 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1758594 Commit-Queue: Siyu An <siyua@chromium.org> Reviewed-by:
Jared Saul <jsaul@google.com> Cr-Commit-Position: refs/heads/master@{#688287}
-
Peter Kotwicz authored
This CL: - Removes id parameter from WebApkInfo#create() - Introduces builder class for WebApkInfo - WebApkInfoBuilder - and refactors tests to use it - Removes test-only 14 arg WebappInfo#create() method and switches tests to use WebappInfo#create() version which takes intent as input - Introduces method to create minimal intent for building WebappInfo - WebappTestHelper#createMinimalWebappIntent() - Changes WebApkUkmRecorder to work with tests which where WebApkInfo#manifestUrl() returns null BUG=None Change-Id: I7d2eb49c47509f6581e579fb05f3b9c7330eeca6 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1759927Reviewed-by:
Dominick Ng <dominickn@chromium.org> Reviewed-by:
Yaron Friedman <yfriedman@chromium.org> Commit-Queue: Peter Kotwicz <pkotwicz@chromium.org> Cr-Commit-Position: refs/heads/master@{#688286}
-
Jeremy Roman authored
Specifically, this bit: https://wicg.github.io/portals/#htmlportalelement-may-have-a-guest-browsing-context And explicit tracking of the adopted predecessor element and just-adopted flag: https://wicg.github.io/portals/#portalactivateevent-adopted-predecessor-element https://wicg.github.io/portals/#htmlportalelement-just-adopted-flag Some additional refactoring to follow. Change-Id: If7ef036ed8ab6e95e7857d143cfeada41a9eb8fc Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1760682Reviewed-by:
Lucas Gadani <lfg@chromium.org> Commit-Queue: Jeremy Roman <jbroman@chromium.org> Cr-Commit-Position: refs/heads/master@{#688285}
-
Matt Falkenhagen authored
The thread ServiceWorkerContextCore lives on (the "core thread") will move from the IO thread to the UI thread. This makes some code used in navigation be core thread-aware instead of assuming the IO thread. This allows many content_browsertests for ServiceWorker to pass when ServiceWorkerOnUI is enabled. Bug: 824858 Change-Id: I3e70cf88f615e77a8f45370eea002b6b344e1511 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1760829 Commit-Queue: Matt Falkenhagen <falken@chromium.org> Reviewed-by:
John Abd-El-Malek <jam@chromium.org> Cr-Commit-Position: refs/heads/master@{#688284}
-
Avery Musbach authored
Change-Id: I543f2206d7c3ab0ef32433f98139ea052bd93bd5 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1761318Reviewed-by:
Sammie Quon <sammiequon@chromium.org> Commit-Queue: Avery Musbach <amusbach@chromium.org> Cr-Commit-Position: refs/heads/master@{#688283}
-
Herb Derby authored
Most of the fields for Settings are not used. Just pass the distance field text capability as a bool. See matching CL: https://skia-review.googlesource.com/c/skia/+/235102 Change-Id: I0c0d4db0386ce98dbd5856129e5a60ab28d6ac61 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1758438Reviewed-by:
Ben Wagner <bungeman@chromium.org> Reviewed-by:
Khushal <khushalsagar@chromium.org> Commit-Queue: Herb Derby <herb@google.com> Cr-Commit-Position: refs/heads/master@{#688282}
-
Dominik Röttsches authored
This will prefer performing DWrite API based font fallback on all Windowses, particularly on Windows 8.0 and below, where API based fallback was previously not available and hard-coded lookup tables were used. A content feature is used to flip the switch on this as a last-resort deactivation method, should this lead to unexpected fallback problems. Bug: 976737 Change-Id: I65d160a573a625ea60a866bacfd12dd0d8a5935a Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1753006 Commit-Queue: Dominik Röttsches <drott@chromium.org> Commit-Queue: Avi Drissman <avi@chromium.org> Auto-Submit: Dominik Röttsches <drott@chromium.org> Reviewed-by:
Avi Drissman <avi@chromium.org> Cr-Commit-Position: refs/heads/master@{#688281}
-
Wei Li authored
Add a new feature flag HappinessTrackingSurveysForDesktopDemo to allow designer/dev/testing team to directly invoke the survey related UI for tests. This flag allows users to bypass the triggering mechanisms to directly invoke Survey bubble UI on NTP. BUG=979530 TEST=either use command line "--enable-features=HappinessTrackingSurveysForDesktopDemo" or by changing the corresponding flag in chrome://flags, user can directly see user happiness survey bubble on a New Tab Page. Change-Id: Ibc17ba729fb01cb33c9245ba25ce1dc2cbbf304c Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1759287Reviewed-by:
Peter Kasting <pkasting@chromium.org> Reviewed-by:
Robert Liao <robliao@chromium.org> Commit-Queue: Wei Li <weili@chromium.org> Cr-Commit-Position: refs/heads/master@{#688280}
-
Alex Newcomer authored
Either we are attempting to emplace() past the last element of the vector, or the index is negative somehow. Fix the first condition and check for the second. Bug: 992344 Change-Id: Id3f6a71a9d5ab3ad41079f9ed1fa7067238faae8 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1761295 Commit-Queue: Alex Newcomer <newcomer@chromium.org> Reviewed-by:
Matthew Mourgos <mmourgos@chromium.org> Cr-Commit-Position: refs/heads/master@{#688279}
-
Marijn Kruisselbrink authored
Don't try to force every error into a base::File::Error, instead have our own enum and a custom error message. This should make it easier for web developers to debug what is going on. Also switches some DOMException constructor usage to using V8ThrowDOMException which results in the exception messages actually being shown in dev-tools. Bug: 971268, 991544 Change-Id: Id5128363bf5410ed08c1f4e65ebbe4b4bee93d7a Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1738654 Commit-Queue: Marijn Kruisselbrink <mek@chromium.org> Reviewed-by:
Daniel Cheng <dcheng@chromium.org> Reviewed-by:
Victor Costan <pwnall@chromium.org> Cr-Commit-Position: refs/heads/master@{#688278}
-
Takuto Ikuta authored
Bug: 988833 Change-Id: I0e6fa963b46363415226dbda74933984c180653d Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1760827 Auto-Submit: Takuto Ikuta <tikuta@chromium.org> Commit-Queue: Kenneth Russell <kbr@chromium.org> Reviewed-by:
Glenn Hartmann <hartmanng@chromium.org> Reviewed-by:
Kenneth Russell <kbr@chromium.org> Cr-Commit-Position: refs/heads/master@{#688277}
-