- 13 Aug, 2019 40 commits
-
-
Julian Pastarmov authored
s/nrparker/nparker/g BUG=none Change-Id: I215bfa4e238720ae0cd28524ff322bde900575bf Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1751249 Commit-Queue: Julian Pastarmov <pastarmovj@chromium.org> Commit-Queue: Lutz Justen <ljusten@chromium.org> Auto-Submit: Julian Pastarmov <pastarmovj@chromium.org> Reviewed-by:
Lutz Justen <ljusten@chromium.org> Cr-Commit-Position: refs/heads/master@{#686369}
-
Mario Sanchez Prada authored
Adapted the original CL to the change done in the precursor CL to rename the new blink::mojom::SurroundingText Mojo interface into blink::mojom::Frame instead. Original change's description: > This class is now only used from blink::SurroundingTextImpl, so we can > move it down into renderer/core/editing and remove it from the public API. > > Also, make the most of this move to make the new blink::SurroundingText > class use internal Blink types (i.e. WTF::String, LocalFrame) instead of > the public Web* types (i.e. WebString, WebLocalFrame), which will allow > migrating blink::EditingImpl away from WebLocalFrame on a follow-up CL. > > Bug: 980151, 919392 > Change-Id: I2a6ba1aa9e00603d40b4ccb9acb69f46baf5931b > Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1699608 > Commit-Queue: Mario Sanchez Prada <mario@igalia.com> > Reviewed-by: Kentaro Hara <haraken@chromium.org> > Reviewed-by: Daniel Cheng <dcheng@chromium.org> > Cr-Commit-Position: refs/heads/master@{#680007} Bug: 980151, 919392 Change-Id: I0666c5714e8db7060c4a65dc89d9a43f5bcc7b23 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1738555 Commit-Queue: Mario Sanchez Prada <mario@igalia.com> Reviewed-by:
Kentaro Hara <haraken@chromium.org> Reviewed-by:
Daniel Cheng <dcheng@chromium.org> Cr-Commit-Position: refs/heads/master@{#686368}
-
Pavel Shmakov authored
In WebAPKs, showing splash screens before browser launch is restricted to Android N+ because of visual glitches on M-. The visual glitches occur because translucency flipping doesn't play well with SurfaceViews. To support this behavior in TWAs, we add a parameter allowing to opt-out from translucency flipping. Change-Id: Ibde67c49577afff32207c239b0736829b178520e Bug: 989603 Change-Id: Ibde67c49577afff32207c239b0736829b178520e Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1744412Reviewed-by:
Peter Conn <peconn@chromium.org> Reviewed-by:
Peter Kotwicz <pkotwicz@chromium.org> Commit-Queue: Pavel Shmakov <pshmakov@chromium.org> Cr-Commit-Position: refs/heads/master@{#686367}
-
Chromium WPT Sync authored
Using wpt-import in Chromium ebcf8f6f. 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 NOAUTOREVERT=true TBR=robertma No-Export: true Change-Id: I011ed52d436c2203cb805317c5e690d1477d5b8a Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1751282Reviewed-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@{#686366}
-
Koji Ishii authored
This patch adds |MoveToNextSkippingChildren| to |NGInlineCursor|. To support this, this patch also adds |ChildrenCount| to |NGFragmentItem| so that we can compute the box tree structure from a flat list of |FragmentItem|. Bug: 982194 Change-Id: I0a5768d7cc8d522a52c5d7bb1556390583e6d172 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1746333Reviewed-by:
Emil A Eklund <eae@chromium.org> Reviewed-by:
Yoshifumi Inoue <yosin@chromium.org> Commit-Queue: Koji Ishii <kojii@chromium.org> Cr-Commit-Position: refs/heads/master@{#686365}
-
Vladislav Kuzkokov authored
Use return value instead of output parameter. Bug: 964948 Change-Id: I8a5cbc329b9d95dfb77e0fbb66b0c1d7d178e7e0 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1749091Reviewed-by:
Bo <boliu@chromium.org> Commit-Queue: Vladislav Kuzkokov <vkuzkokov@chromium.org> Cr-Commit-Position: refs/heads/master@{#686364}
-
Julie Jeongeun Kim authored
This CL converts DevToolsSessionHost and DevToolsSession to new Mojo types. It updates BindDevToolsAgent from frame.mojom, OnReadyForInspection from embedded_worker.mojom and OnReadyForInspection from shared_worker_host.mojom with new Mojo types. Along with updating mojom interfaces, it also updates methods and members which take DevToolsAgentPtrInfo, DevToolsAgentPtr, DevToolsAgentHostRequest, DevToolsAgentHostAssociatedPtrInfo, DevToolsAgentAssociatedRequest with new Mojo types. Bug: 955171, 978694 Change-Id: Ida673d762d4e5bb7dd2696cba28bc681a6f04552 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1748906 Commit-Queue: Julie Kim <jkim@igalia.com> Reviewed-by:
Ken Rockot <rockot@google.com> Reviewed-by:
Andrey Kosyakov <caseq@chromium.org> Reviewed-by:
Kinuko Yasuda <kinuko@chromium.org> Cr-Commit-Position: refs/heads/master@{#686363}
-
Thomas Tangl authored
Bug: 966388 Change-Id: I09461f0f9f64f162d53cece7c81a013b84d2ba2c Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1749090 Commit-Queue: Thomas Tangl <tangltom@chromium.org> Reviewed-by:
Marc Treib <treib@chromium.org> Cr-Commit-Position: refs/heads/master@{#686362}
-
Kenichi Ishibashi authored
We don't support on-the-main-thread script fetch anymore. Bug: 967272 Change-Id: I25cd1a56e16e928709cda11b705089e62bc840d9 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1745110Reviewed-by:
Hiroki Nakagawa <nhiroki@chromium.org> Commit-Queue: Kenichi Ishibashi <bashi@chromium.org> Cr-Commit-Position: refs/heads/master@{#686361}
-
Mitsuru Oshima authored
The frame layout depends on window state as well as immersive state. However SetEnabeldForWidget may skip updating the frame if the target state is same. Make sure we relayout the frame when it is called. Bug: b/128864049 Test: Covered by unit test. manually tested on the device. Change-Id: I254b7326f01acb359cd18a66c2624c57072957ca Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1714113 Commit-Queue: Mitsuru Oshima (OOO 8/12,13) <oshima@chromium.org> Reviewed-by:
Eliot Courtney <edcourtney@chromium.org> Cr-Commit-Position: refs/heads/master@{#686360}
-
Balazs Engedy authored
ImportantSitesUtil::GetImportantRegisterableDomains tried to access the SearchPermissionsService under certain circumstances in incognito mode, but the service is not instantiated for off-the-record profiles. This CL only fixes the crash, and does not otherwise change the logic, which also seems to be broken, as documented in crbug.com/993021. Bug: 989890 Change-Id: I9b93bd5007855e1906fcc67c43ed625030ed7142 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1747011Reviewed-by:
Raymes Khoury <raymes@chromium.org> Commit-Queue: Balazs Engedy <engedy@chromium.org> Cr-Commit-Position: refs/heads/master@{#686359}
-
Robbie Gibson authored
This CL adds dark mode colors to the share extension. Besides colors, it also removes the blur effect view. Before, there was a blur effect over the background. There weren't any other views behind it though, so the blur just helped color the background. Now, all the colors are chrome-specific colors, so the background can be colored automatically. Screenshots can be found in the linked bug. Also, when building on iOS 13, the default presentation style for the extension has changed to automatic, so the dimmed background is now added automatically. Finally, this modernizes some parts of share_extension_view.mm. Bug: 992391 Change-Id: I16e5321ebc9018b41bfe617ad304fd3c3b6bb32e Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1748754 Commit-Queue: Robbie Gibson <rkgibson@google.com> Reviewed-by:
Rohit Rao <rohitrao@chromium.org> Cr-Commit-Position: refs/heads/master@{#686358}
-
Benoît Lizé authored
This test fails on CFI bots, see attached bug. This is due to it being too strict. It would be correct if there are no PA allocations triggered elsewhere in the process, which is an assumption that can break with test harness changes. Make the assert more tolerant to these changes, by asserting that memory is reduced, instead of having a strict value. Bug: 993219 Change-Id: I1e5e3b1f7e1f14357171a06a31a2eba9380babfe Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1751183Reviewed-by:
-
Vasilii Sukhanov authored
Bug: 986298 Change-Id: Idbac7135d6ed958529bea6968d4405f5e689c84b Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1746476Reviewed-by:
Ioana Pandele <ioanap@chromium.org> Commit-Queue: Vasilii Sukhanov <vasilii@chromium.org> Cr-Commit-Position: refs/heads/master@{#686356}
-
Julian Watson authored
BUG=992293 Change-Id: Id54f0b0f26e06b1ce5c2c7c4e725889867e88334 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1750516 Commit-Queue: Julian Watson <juwa@google.com> Auto-Submit: Julian Watson <juwa@google.com> Reviewed-by:
Joel Hockey <joelhockey@chromium.org> Cr-Commit-Position: refs/heads/master@{#686355}
-
chromium-autoroll authored
The AutoRoll server is located here: https://autoroll.skia.org/r/fuchsia-sdk-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:fuchsia-arm64-cast;luci.chromium.try:fuchsia-x64-cast TBR=cr-fuchsia+bot@chromium.org Change-Id: I90311089fd94cae99be12b4b3261555f83bcadd2 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1750900Reviewed-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@{#686354}
-
Nathan Rogers authored
Address the previous TODO by replacing the iterator based loop with a range-based for loop. Change-Id: I1f68e03233fc0a37b91aa6c8e29c29660a3a1abb Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1750932Reviewed-by:
Kouhei Ueno <kouhei@chromium.org> Commit-Queue: Nathan Rogers <nathanrogers@google.com> Cr-Commit-Position: refs/heads/master@{#686353}
-
Julie Jeongeun Kim authored
This CL converts DevToolsSessionHost and DevToolsSession to new Mojo types. The changes are two CLs and it is the first CL and the second on is [1]. It updates AttachDevToolsSession and ChildWorkerCreated from devtools_agent.mojom It also converted methods and members which take these DevToolsAgent and DevToolsAgentHost to new Mojo types. SharedWorkerDevToolsAgentHost::WorkerReadyForInspection and ServiceWorkerDevToolsAgentHost::WorkerReadyForInspection still keep old types and these are handled at [1] with the rest of code which still uses old types. [1] https://crrev.com/c/1748906 Bug: 955171, 978694 Change-Id: I2d67a390b3299bfd71d4539857973481f349849e Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1745944Reviewed-by:
-
Dominik Röttsches authored
Previously this test relied on font fallback and broke up runs unnecessarily. Make font selection for the Devanagari tests explicit. Rephrase test expectations to rely less on font-selection-induced run splitting, but rather create assertions that test for the script properties independent of run layout. Preparation for moving font fallback on Windows to an OOP call instead of hardcoding our fallback tables. Unit tests cannot perform such OOP calls as the browser side fallback Mojo service is not available. Bug: 976737 Change-Id: I7126979efe40848325b1859c2e0b8ecfef64d775 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1748946 Commit-Queue: Dominik Röttsches <drott@chromium.org> Reviewed-by:
-
Hiroshige Hayashizaki authored
As a preparation for adding parser-related tests. Bug: 990561 Change-Id: I8fdb07059514fe088438e8499cd4ac20e1c0b528 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1734328 Commit-Queue: Hiroshige Hayashizaki <hiroshige@chromium.org> Reviewed-by:
-
Christopher Gunadi authored
Add test to ensure the "Install with Linux" context menu option is hidden/shown when Crostini root access is allowed or disallowed by kUserCrostiniRootAccessAllowedByPolicy policy preference control. Feature flag kCrostiniAdvancedAccessControls is enabled for file manager browser tests to test this functionality which is hidden behind the flag. Follow up CL to crosreview.com/1730907 Bug: 983997 Change-Id: I12f70ea516528275b327525119587d0f0742a50d Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1738686 Commit-Queue: Christopher Gunadi <chrisgunadi@google.com> Reviewed-by:
Luciano Pacheco <lucmult@chromium.org> Reviewed-by:
Noel Gordon <noel@chromium.org> Cr-Commit-Position: refs/heads/master@{#686349}
-
Mario Sanchez Prada authored
We can't construct a WTF::String using the default constructor when running the GetTextSurroundingSelectionCallback directly for those cases where the SurroundingText is empty, since that will construct a null string instead of an empty string, causing a crash at the Mojo level due to trying to send a null message over the wire, which is not permitted. To avoid this crash in Mojo, make sure we create a WTF::String out of an empty string, instead of using the default constructor. Last, this CL also incorporates the required changes to address Daniel's comments after the original patch landed in CL 1696968 [1], to rename the newly added Mojo interface to become blink::mojom::Frame. [1] https://crrev.com/c/1696968/11/third_party/blink/public/mojom/editing/editing.mojom#9 Original change's description: > Move this method from the content.mojom.Frame Mojo interface into Blink, > as part of a new blink.mojom.SurroundingText Mojo interface, and add a new > class there to implement such interface (i.e. blink::SurroundingTextImpl), > to replace usages of content.mojom.Frame's GetTextSurroundingSelection() > from the browser process. > > Note that this SurroundingTextImpl class still relies on WebSurroundingText > and WebLocalFrame to keep the change smaller, but this is a temporary step > that will be corrected once WebSurroundingText has been moved out of the > public API and into renderer/core, which will happen on a follow-up CL. > > Bug: 980151 > Change-Id: I62fb2d5b24a98773c1acddbe1c58217c1a2d53e4 > Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1696968 > Commit-Queue: Mario Sanchez Prada <mario@igalia.com> > Reviewed-by: Mike West <mkwst@chromium.org> > Reviewed-by: Ken Buchanan <kenrb@chromium.org> > Reviewed-by: Kentaro Hara <haraken@chromium.org> > Cr-Commit-Position: refs/heads/master@{#679981} Bug: 980151, 987214, 987191, 989438 Change-Id: Ia32be36c1a7524930ddcecc79e7fac5bcb514a13 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1735459Reviewed-by:
Ken Buchanan <kenrb@chromium.org> Reviewed-by:
Mike West <mkwst@chromium.org> Reviewed-by:
-
Daniel Bratell authored
33bits.org is a pointer that is supposed to load 33bits.wordpress.com but the mapping isn't there anymore so point directly to 33bits.wordpress.com. Change-Id: Ia423858d5d59c8a0195fd0c69143faae64be55fc Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1749186 Auto-Submit: Daniel Bratell <bratell@opera.com> Reviewed-by:
Chris Palmer <palmer@chromium.org> Commit-Queue: Daniel Bratell <bratell@opera.com> Cr-Commit-Position: refs/heads/master@{#686347}
-
Yang Guo authored
R=szuend@chromium.org Change-Id: Ibef42325a778acd4d513e3c038a76fbb266a36c4 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1748890 Commit-Queue: Yang Guo <yangguo@chromium.org> Auto-Submit: Yang Guo <yangguo@chromium.org> Reviewed-by:
Alexei Filippov <alph@chromium.org> Cr-Commit-Position: refs/heads/master@{#686346}
-
Yi Su authored
TabGridNewTabButton is inited with page=TabGridPageIncognitoTabs, and depends on "setPage:" method to set the image of button. However this "setPage" has an early return for same page value, so when app is launched in incognito mode, the image will be missing. Bug: 991994 Change-Id: I7fd4f90a57486392f12ffab339f81517dbf25bcf Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1746262 Commit-Queue: Yi Su <mrsuyi@chromium.org> Reviewed-by:
edchin <edchin@chromium.org> Cr-Commit-Position: refs/heads/master@{#686345}
-
Fergus Dall authored
Tests: external/wpt/fetch/cross-origin-resource-policy/fetch.https.any.worker.html virtual/blink-cors/external/wpt/fetch/cross-origin-resource-policy/fetch.https.any.worker.html At time of writing, the findit flake score for these tests is 704. Mark them as timing out on more platforms. TBR=guidou@chromium.org TBR=qyearsley@chromium.org TBR=kojii@chromium.org TBR=jsbell@chromium.org TBR=dpranke@chromium.org TBR=foolip@chromium.org Bug: 626703 Change-Id: I7749eecaaafee8e0a3bc3167a60005616ea6f4e6 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1750538Reviewed-by:
Fergus Dall <sidereal@google.com> Cr-Commit-Position: refs/heads/master@{#686344}
-
Joanmarie Diggs authored
Text leaf nodes and line breaks were being exposed as ATK_ROLE_TEXT. However, the definition of ATK_ROLE_TEXT is: An interactive widget that supports multiple lines of text and optionally accepts user input, but whose purpose is not to solicit user input. In contrast, ATK_ROLE_STATIC is defined as: A generic non-container object whose purpose is to display a brief amount of information to the user and whose role is known by the implementor but lacks semantic value for the user. Thus text leaf nodes and line breaks should be exposed with the latter role; not the former. Bug: 984590 Change-Id: I4dfbf8d8d90a3f894803a62914a6d9882828957e Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1749145 Commit-Queue: Martin Robinson <mrobinson@igalia.com> Auto-Submit: Joanmarie Diggs <jdiggs@igalia.com> Reviewed-by:
Martin Robinson <mrobinson@igalia.com> Cr-Commit-Position: refs/heads/master@{#686343}
-
Rika Fujimaki authored
In a precious CL(1743245), I added LocalRecord() temporarily to return v8::Local<v8::Module> instead of ModuleRecord. Now is the time to replace all Record() with LocalRecord() to remove ModuleRecord. And, I renamed Record() into V8Module() for a conventional reason. Bug: 991863 Change-Id: I34f57226ab5d471a4909794fe3d43e1748fb020b Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1746209 Commit-Queue: Rika Fujimaki <rikaf@google.com> Reviewed-by:
Yuki Shiino <yukishiino@chromium.org> Reviewed-by:
Keishi Hattori <keishi@chromium.org> Reviewed-by:
-
Virender Singh authored
This CL updates the Link structure to include the underlying text range. The underlying text range is being populated using the newly added FPDFLink_GetTextRange() API. This would enable us to position links along with text in the accessibility tree. Bug: 981448 Change-Id: I4551f70c21e2846f9a85c19c982f17dc52e0b25c Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1710275 Commit-Queue: Virender Singh <virens@microsoft.com> Reviewed-by:
Lei Zhang <thestig@chromium.org> Reviewed-by:
Kevin Babbitt <kbabbitt@microsoft.com> Cr-Commit-Position: refs/heads/master@{#686341}
-
Fergus Dall authored
TBR=dom-dev@chromium.org Bug: 993238 Change-Id: Id1b55d9a908ffd38094e33dd4dea261153b72e40 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1750473Reviewed-by:
-
Gayane Petrosyan authored
This is a partial revert of https://chromium-review.googlesource.com/c/chromium/src/+/1702769 TBR=estade@chromium.org Bug: 988055 Change-Id: Id6e67a5a69fa97c23083f44b9054526792cb58ff Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1750790Reviewed-by:
Gayane Petrosyan <gayane@chromium.org> Commit-Queue: Gayane Petrosyan <gayane@chromium.org> Cr-Commit-Position: refs/heads/master@{#686339}
-
Hajime Hoshi authored
This CL adds a new test for BackForwardCache with another isolated world. Multiple worlds are used by extensions. Bug: 980111 Change-Id: Ifdff84fb52679ed010b38cd54933481edd45e955 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1709793 Commit-Queue: Hajime Hoshi <hajimehoshi@chromium.org> Reviewed-by:
Arthur Sonzogni <arthursonzogni@chromium.org> Reviewed-by:
Alexander Timin <altimin@chromium.org> Reviewed-by:
Nasko Oskov <nasko@chromium.org> Cr-Commit-Position: refs/heads/master@{#686338}
-
Eric Willigers authored
inset is one to four values, each auto or a length-percentage. https://drafts.csswg.org/css-logical/#propdef-inset Change-Id: I3f1599e200fec16a709cc6404246a28c404805b7 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1747488Reviewed-by:
Rune Lillesveen <futhark@chromium.org> Commit-Queue: Eric Willigers <ericwilligers@chromium.org> Cr-Commit-Position: refs/heads/master@{#686337}
-
Fergus Dall authored
Test: http/tests/devtools/indexeddb/live-update-indexeddb-list.js This test has been observed flaking on Windows, and on release builds. Mark this as expected behaviour. At time of writing, the findit score for this test was 714 TBR=dmurph@chromium.org Bug: 989860 Change-Id: Id6f4815867e0d33322c52289bad5393784961876 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1750472Reviewed-by:
-
Fergus Dall authored
virtual/scalefactor200/css3/filters/effect-reference-zoom-hw.html This test currently has a findit flake score of 844, apparently entierly from timeouts. TBR=fs@opera.com, liberato@chromium.org TBR=mlamouri@google.com, yhirano@chromium.org Bug: 658305 Change-Id: I120323bb81c8e8e6b62ae059ebcba664f77a5388 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1750471Reviewed-by:
-
Sophie Chang authored
Bug: 993156 Change-Id: Ie6dfc9269805104d3a48fe96e48477ca700cbac7 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1750924Reviewed-by:
Robert Ogden <robertogden@chromium.org> Commit-Queue: Sophie Chang <sophiechang@chromium.org> Cr-Commit-Position: refs/heads/master@{#686334}
-
Fergus Dall authored
This reverts commit e3121f99. Reason for revert: Looks like it broke Win10 (Debug|Release) (NVIDIA) See https://ci.chromium.org/p/chromium/builders/ci/Win10%20Debug%20%28NVIDIA%29/23156 for example failure. Original change's description: > Switch Windows 10 GPU testing to 64-bit. > > See > https://chromium-review.googlesource.com/c/chromium/tools/build/+/1750303 > > Bug: 988833 > Change-Id: If6082a035eec474d3299afb89bd78ca225d95ed6 > Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1747005 > Reviewed-by: Kenneth Russell <kbr@chromium.org> > Reviewed-by: Takuto Ikuta <tikuta@chromium.org> > Commit-Queue: Takuto Ikuta <tikuta@chromium.org> > Cr-Commit-Position: refs/heads/master@{#686321} TBR=ynovikov@chromium.org,kbr@chromium.org,tikuta@chromium.org Change-Id: If0922864c7beaa2209933d47c1bf7385c3b39a42 No-Presubmit: true No-Tree-Checks: true No-Try: true Bug: 988833 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1750470Reviewed-by:
-
Fergus Dall authored
Tests changed: virtual/gpu/fast/canvas/canvas-composite-stroke-alpha.html This test has been observed timing out on all platforms TBR=fserb@chromium.org Bug: 984467 Change-Id: Iaa0fda0cfd0f3ca314c184ccc2c9efcd5cacf532 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1750532Reviewed-by:
-
Christian Biesinger authored
R=eae@chromium.org, sajos@microsoft.com Change-Id: I4ca39e7299b400d9a34e41d3b0909bbee2e1222d Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1750386 Auto-Submit: Christian Biesinger <cbiesinger@chromium.org> Commit-Queue: Emil A Eklund <eae@chromium.org> Reviewed-by:
-
Rika Fujimaki authored
Add v8::Local<v8::Module> argument to Evaluate and V8Namespace to make ModuleRecord's functions static. It leads to remove ModuleRecord. Bug: 991863 Change-Id: I4470413d3107d2427eb9cc1f4977dad100d4e66c Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1746115 Commit-Queue: Rika Fujimaki <rikaf@google.com> Reviewed-by:
-
