- 11 Jan, 2019 40 commits
-
-
Eugene But authored
Bug: 781534 Change-Id: I9558bbf6996504ecdea358119da16ef160a6ca28 Reviewed-on: https://chromium-review.googlesource.com/c/1405990 Commit-Queue: Eugene But <eugenebut@chromium.org> Reviewed-by:
Sylvain Defresne <sdefresne@chromium.org> Cr-Commit-Position: refs/heads/master@{#622102}
-
chromium-internal-autoroll authored
https://chrome-internal.googlesource.com/chrome/src-internal.git/+log/e58cda9a3da9..0ef2f1153a1b Created with: gclient setdep -r src-internal@0ef2f1153a1b The AutoRoll server is located here: https://autoroll-internal.skia.org/r/src-internal-chromium-autoroll Documentation for the AutoRoller is here: https://skia.googlesource.com/buildbot/+/master/autoroll/README.md If the roll is causing failures, please contact the current sheriff, who should be CC'd on the roll, and stop the roller if necessary. TBR=mmoss@chromium.org Change-Id: I6fb5801f3dcda4b98075c51372d2eeee6410be8b Reviewed-on: https://chromium-review.googlesource.com/c/1407249Reviewed-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@{#622101}
-
David Bokan authored
This CL moves logic specific to spatial navigation out of FocusController and into the newly created SpatialNavigationController. This is a straight move-only operation, with limited and minor name changes and comments. There should be no functional changes. Bug: 919158 Change-Id: Ib2fd763c82d612f13e0c33ccaeb9700890bb2458 Reviewed-on: https://chromium-review.googlesource.com/c/1404561 Commit-Queue: David Bokan <bokan@chromium.org> Reviewed-by:
Emil A Eklund <eae@chromium.org> Cr-Commit-Position: refs/heads/master@{#622100}
-
Chris Harrelson authored
Pauses may happen while the parser has already yielded, and in particular when a script-inserted style sheet appears that has an external resource. The failure mode before this patch was: 1. Yield & schedule unyield 2. Script task runs 3. Script task inserts stylesheet with external resource 4. Resource finished 5. ResumeParsingAfterPause called, but there is no last_chunk_before_pause_ because there was a yield, not a pause. Bug:902237 Change-Id: Id65c721fc193ecec7190e6b466a6794b65ea6acc Reviewed-on: https://chromium-review.googlesource.com/c/1389502Reviewed-by:
Kouhei Ueno <kouhei@chromium.org> Commit-Queue: Chris Harrelson <chrishtr@chromium.org> Cr-Commit-Position: refs/heads/master@{#622099}
-
Michael Lippautz authored
Switch to a uniform object construction that does not differentiate between mixins and regular garabge collected objects and as a consequence can interrupt both construction cases with garbage collections. Object construction now uniformely works as follows: 1. The object under construction is marked so in the HeapObjectHeader. 2. Upon discovering such an object it is delayed in a separate marking worklist. 3. Upon hitting the main atomic pause all such objects are conservatively scanned for pointers without using the Trace method. Special cases: a. Mixins in construction: The HeapObjectHeader cannot retrived for such objects. A default GetTraceDescriptor() implementation returning a sentinel marker is used to discover that such objects should be delayed. b. Upon reaching a safepoint (e.g. no stack), in-construction objects are moved to a marking worklist that, similar to the regular marking worklist, allows for incremental processing. Effects: - No more TLS access for no-GC scope on mixin construction. - No more memory needed for the no-GC scope marker in mixin classes - MakeGarbageCollected should require less binary size as implementations can be aligned. - Incremental marking Start and Step operations are safe to be called with stack (even though it is preferable to not do so). - Object construction is safe to be used with a concurrent marker as it is ok for any objects (mixins as well as normal ones) to be published to the object graph during constructors. This reverts commit ad406608. Bug: 911662 Change-Id: I7babc6bac25b446a63783bf2c3d19e9b525d9e9a Reviewed-on: https://chromium-review.googlesource.com/c/1405991Reviewed-by:
Kentaro Hara <haraken@chromium.org> Commit-Queue: Michael Lippautz <mlippautz@chromium.org> Cr-Commit-Position: refs/heads/master@{#622098}
-
chromium-autoroll authored
https://skia.googlesource.com/skia.git/+log/cb42fc0dc56f..9716603f9736 git log cb42fc0dc56f..9716603f9736 --date=short --no-merges --format='%ad %ae %s' 2019-01-11 nednguyen@google.com Update skia docs to stop referring to NeedsManualRebaseline 2019-01-11 fmalita@chromium.org Hack up SkShaper to pass additional callback info 2019-01-11 recipe-roller@chromium.org Roll recipe dependencies (trivial). 2019-01-11 ethannicholas@google.com fixed SPIR-V decoration 2019-01-11 jvanverth@google.com Revert "Add target SkColorSpace to SkImage_GpuYUVA." 2019-01-11 robertphillips@google.com Reduce reliance on the SkImage_Gpu's GrContext pointer 2019-01-11 benjaminwagner@google.com Omit another gm for IntelIris655 vk. 2019-01-11 jvanverth@google.com Add target SkColorSpace to SkImage_GpuYUVA. 2019-01-11 skia-autoroll@skia-public.iam.gserviceaccount.com Roll third_party/externals/angle2 18af9a5a50c6..92ee1f9f3032 (1 commits) 2019-01-11 ethannicholas@google.com Fixed GPU perlin noise shaders under rotation. Created with: gclient setdep -r src/third_party/skia@9716603f9736 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 BUG=chromium:621126 TBR=bsalomon@chromium.org Change-Id: Ic2fe98f2713fec5d3914ac2a13589999b9617e13 Reviewed-on: https://chromium-review.googlesource.com/c/1406959Reviewed-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@{#622097}
-
Scott Violet authored
Windows that want to receive drop events must call WindowTree::SetCanAcceptDrops(). This was only done for DesktopNativeWidgetAura's content-window by DesktopWindowTreeHostMus. A better approach is to implicitly call SetCanAcceptDrops() any time a DragDropDelegate is registered on a window. Going this route means DesktopWindowTreeHostMus doesn't need to explicitly call SetCanAcceptDrops() because DesktopNativeWidgetAura registers a DropDropDelegate. BUG=920789 TEST=covered by test Change-Id: I3c350d51ba0258dafc70fcee98c694b7fc39ce0b Reviewed-on: https://chromium-review.googlesource.com/c/1407082 Commit-Queue: Scott Violet <sky@chromium.org> Reviewed-by:
Xiyuan Xia <xiyuan@chromium.org> Cr-Commit-Position: refs/heads/master@{#622096}
-
Alexandre Frechette authored
(1) Adds an optional test to check serialized data against complete data. (2) Add script to move data from a single arbitrary folder to the correct locations in chromium. (3) Update data to latest version. (4) Fix serialized data to C script so that bytes to uint32 conversion is done manually. Bug: 850947 Change-Id: I84d5a6e34a99eee60c973525f2c8358c583d3a87 Reviewed-on: https://chromium-review.googlesource.com/c/1383351Reviewed-by:
anthonyvd <anthonyvd@chromium.org> Commit-Queue: Alexandre Frechette <frechette@chromium.org> Cr-Commit-Position: refs/heads/master@{#622095}
-
Antoine Labour authored
All callers pass an internalformat consistent with gpu::InternalFormatForGpuMemoryBufferFormat, so there's no need to specify redundantly the internalformat and the BufferFormat. Instead just pass in the BufferFormat, which simplifies all callers and increases consistency. Bug: 900044 Change-Id: I19916a6f53a19e1419e3312a2ea4368d79fd95e4 Reviewed-on: https://chromium-review.googlesource.com/c/1405286Reviewed-by:
Daniele Castagna <dcastagna@chromium.org> Reviewed-by:
Robert Kroeger <rjkroege@chromium.org> Commit-Queue: Antoine Labour <piman@chromium.org> Cr-Commit-Position: refs/heads/master@{#622094}
-
Steven Bennetts authored
chromeos_export.h is being included outside of src/chromeos which should not be the case. This is probably from files moved out of src/chromeos. This CL fixes those and makes chromeos_export a non public deps of //chromeos. Bug: 918682 Change-Id: I6bf394f0c933b8259f7567cfbb514c936b2d457c Reviewed-on: https://chromium-review.googlesource.com/c/1405979 Commit-Queue: Steven Bennetts <stevenjb@chromium.org> Reviewed-by:
A Olsen <olsen@chromium.org> Reviewed-by:
Nico Weber <thakis@chromium.org> Cr-Commit-Position: refs/heads/master@{#622093}
-
Aleks Totic authored
Bug: 918904 Change-Id: I95483729d4f4839d18731ed73ae979a5917cf835 Reviewed-on: https://chromium-review.googlesource.com/c/1407119 Commit-Queue: Aleks Totic <atotic@chromium.org> Reviewed-by:
Morten Stenshorne <mstensho@chromium.org> Cr-Commit-Position: refs/heads/master@{#622092}
-
David Valleau authored
This change fixes the bug where if a user attempts to change the URI of an existing configured zeroconf printer from a link local address to a static IP address the attempt to add the device would timeout and fail. This issue was caused by wrongly attempting to perform domain name resolution on the IP address instead of simply performing the configuration with the new address. Bug: 838677 Change-Id: Iba1a8fb8dfd0194138e2d0aa881592027fd713bd Reviewed-on: https://chromium-review.googlesource.com/c/1109387 Commit-Queue: David Valleau <valleau@chromium.org> Reviewed-by:
Sean Kau <skau@chromium.org> Cr-Commit-Position: refs/heads/master@{#622091}
-
Lei Zhang authored
This removes non-determinism and makes tests run more reliably. BUG=905266 Change-Id: I1350e5aa9d1c5571c2fd763f226c8282b8e5f955 Reviewed-on: https://chromium-review.googlesource.com/c/1405725Reviewed-by:
Henrik Grunell <grunell@chromium.org> Reviewed-by:
Eric Seckler <eseckler@chromium.org> Reviewed-by:
Gabriel Charette <gab@chromium.org> Commit-Queue: Lei Zhang <thestig@chromium.org> Cr-Commit-Position: refs/heads/master@{#622090}
-
Nathan Parker authored
This lets us look just at known-advanced-protection users, so we can tell when there are real errors. This also lets us count those users. Once this hits stable I'll setup Chirp alerts on this. I also added tests for metrics. Bug: 919220 Change-Id: I44b0f17b129b565003fa58b43945034a949ca9c5 Reviewed-on: https://chromium-review.googlesource.com/c/1395835Reviewed-by:
Daniel Rubery <drubery@chromium.org> Reviewed-by:
Varun Khaneja <vakh@chromium.org> Reviewed-by:
Steven Holte <holte@chromium.org> Commit-Queue: Nathan Parker <nparker@chromium.org> Cr-Commit-Position: refs/heads/master@{#622089}
-
Xiyuan Xia authored
- Add an ensureWindowServiceClientHasDrawnWindow method to check a Window Service client really paints on screen. - Add a chrome_content_browser_manifest_test_overlay.json that is only applied with --use-test-config switch so that the autotest API could make a mojo call to WindowServerTest. Bug: 908545 Change-Id: I72f9b64fa06b4acf8e7fb8440dd65375949ef3c7 Reviewed-on: https://chromium-review.googlesource.com/c/1406155Reviewed-by:
Scott Violet <sky@chromium.org> Reviewed-by:
Steven Bennetts <stevenjb@chromium.org> Reviewed-by:
Tom Sepez <tsepez@chromium.org> Commit-Queue: Xiyuan Xia <xiyuan@chromium.org> Cr-Commit-Position: refs/heads/master@{#622088}
-
Mike Wasserman authored
ShelfWindowWatcher proactively adds shelf items for certain windows. Limit this experimental behavior to multi-process Mash. Avoids unwanted shelf items for select file dialogs in SingleProcessMash. Bug: 920714 Test: Select file dialogs do not get shelf items (except in multi-process Mash). Change-Id: Ic5986a837d040f9bb8c59c203c6de87375e9091f Reviewed-on: https://chromium-review.googlesource.com/c/1407081Reviewed-by:
-
Xiyuan Xia authored
Make WindowServerTestImpl::EnsureClientHasDrawnWindow capture output of the first ClientRoot window of the client and sanity check the captured pixels. The output is considered okay when there are more than 5 colors. Bug: 908545 Change-Id: Iaa2306da6e2b40b0cf9a8379e71f564b0cf92a8b Reviewed-on: https://chromium-review.googlesource.com/c/1406189Reviewed-by:
-
Quan Nguyen authored
This caps the line height at 20 px so that the Japanese text doesn't make the bubble too large. It also sets the bubble parent container to MenuContainer so the remove-user button is clickable if there is overlap with the shelf. Bug: 917308 Change-Id: I9cfae49c6973c5fdf8fe20374fe17c7fbe60ecee Reviewed-on: https://chromium-review.googlesource.com/c/1405494Reviewed-by:
Jacob Dufault <jdufault@chromium.org> Commit-Queue: Quan Nguyen <qnnguyen@chromium.org> Cr-Commit-Position: refs/heads/master@{#622085}
-
chromium-autoroll authored
https://webrtc.googlesource.com/src.git/+log/b8b3c9918f14..83953e4d95d7 git log b8b3c9918f14..83953e4d95d7 --date=short --no-merges --format='%ad %ae %s' 2019-01-11 nisse@webrtc.org Delete method StreamInterface::ReserveSize 2019-01-11 nisse@webrtc.org Enable use of MediaTransportInterface for video streams. 2019-01-11 nisse@webrtc.org Delete enum NetEqDecoder. 2019-01-11 terelius@webrtc.org Avoid depending on testonly target in event_log_visualizer_utils. 2019-01-11 saza@webrtc.org Make voiceengine/audio transport stop using voice_detection() interface Created with: gclient setdep -r src/third_party/webrtc@83953e4d95d7 The AutoRoll server is located here: https://autoroll.skia.org/r/webrtc-chromium-autoroll Documentation for the AutoRoller is here: https://skia.googlesource.com/buildbot/+/master/autoroll/README.md If the roll is causing failures, please contact the current sheriff, who should be CC'd on the roll, and stop the roller if necessary. CQ_INCLUDE_TRYBOTS=luci.chromium.try:linux_chromium_archive_rel_ng;luci.chromium.try:mac_chromium_archive_rel_ng TBR=webrtc-chromium-sheriffs-robots@google.com Change-Id: I0dada996ff09b764a7d1264d03a1be3b738f8d05 Reviewed-on: https://chromium-review.googlesource.com/c/1406957Reviewed-by:
-
Dave Tapuska authored
Make PausableTask not inherit PausableTimer. There is no need to schedule a timer since the kJavascriptTimer task queue already is pausable. This class has no real user yet; Devlin claims that the extension code will be using it soon. BUG=907125 Change-Id: I270ef5d8e7f67c32d2377d8f8785ec56e941c204 Reviewed-on: https://chromium-review.googlesource.com/c/1378233Reviewed-by:
Alexander Timin <altimin@chromium.org> Reviewed-by:
-
Eyor Alemayehu authored
To enable pronoun resolutions using the screen context, made changes to send screen context information. An example of a query that requires pronoun resolution is "Who is he?". For such a query, the content of the topmost app can be used to resolve pronoun ("he" as in this example). When the Assistant UI is launched, we currently cache the screen context. This screen context is currently cleared when sending it to the backend. To allow re-sending the screen context multiple times in a single UI session, the screen context is no longer cleared when sent to the backend. The screen context is cleared when the Assistant UI is shown again. There is a TODO that will be done is a subsequent CL that will set a flag to true if the first query is being sent or false otherwise. Bug: b:120736468 Change-Id: Ib26b4da041cf46094c20511f6820c3b4a551c13c Reviewed-on: https://chromium-review.googlesource.com/c/1369062 Commit-Queue: Eyor Alemayehu <eyor@google.com> Reviewed-by:Xiaohui Chen <xiaohuic@chromium.org> Cr-Commit-Position: refs/heads/master@{#622082}
-
Dale Curtis authored
Prepares the way for a 50/50 beta and/or 1% stable experiment. BUG=879406 TEST=passes cq . Change-Id: I99b45acf897a20abb731fca5189b75e3042c045e Reviewed-on: https://chromium-review.googlesource.com/c/1404419Reviewed-by:
Jesse Doherty <jwd@chromium.org> Commit-Queue: Dale Curtis <dalecurtis@chromium.org> Cr-Commit-Position: refs/heads/master@{#622081}
-
danakj authored
When frozen there is no WebFrameWidget, so this cast is invalid. R=dcheng@chromium.org Change-Id: I7a103aeeeb9920d128e1b8ad0b6075fc830ca0d6 Bug: 920952, 917907 Reviewed-on: https://chromium-review.googlesource.com/c/1407038Reviewed-by:
Avi Drissman <avi@chromium.org> Commit-Queue: danakj <danakj@chromium.org> Cr-Commit-Position: refs/heads/master@{#622080}
-
chromium-autoroll authored
https://chromium.googlesource.com/chromiumos/chromite.git/+log/9e5c9014937e..f7f13a85db94 git log 9e5c9014937e..f7f13a85db94 --date=short --no-merges --format='%ad %ae %s' 2019-01-11 mikenichols@chromium.org Revert "Removed the unused drop_file from chromite/cbuildbot" Created with: gclient setdep -r src/third_party/chromite@f7f13a85db94 The AutoRoll server is located here: https://autoroll.skia.org/r/chromite-chromium-autoroll Documentation for the AutoRoller is here: https://skia.googlesource.com/buildbot/+/master/autoroll/README.md If the roll is causing failures, please contact the current sheriff, who should be CC'd on the roll, and stop the roller if necessary. TBR=chrome-os-gardeners@chromium.org Change-Id: Ie8111b7ab549da45ef92a9ad04ed2e9bc901dd46 Reviewed-on: https://chromium-review.googlesource.com/c/1406958Reviewed-by:
-
Koji Ishii authored
As Finch indicated pre-paint is slower in NG, this patch splits the loop to compute VisualRect for fragments for when there's a selection for the LayoutObject and for when not. By splitting the loop, not only reducing the work within the loop, we could try to apply simplication of VisualRect for inline fragments as legacy does. For the branch when there is a selection, yoichio@ will look into it after the end of January. Bug: 636993 Change-Id: Ia06ad95e0cd7e0e9f2d663d26ee2fc7702c05908 Reviewed-on: https://chromium-review.googlesource.com/c/1405948Reviewed-by:
Xianzhu Wang <wangxianzhu@chromium.org> Commit-Queue: Emil A Eklund <eae@chromium.org> Cr-Commit-Position: refs/heads/master@{#622078}
-
Aleks Totic authored
Bug: 635619 Change-Id: Ib1b0bc2b86f80a1758cff6bcb157c26ec76fdf4c Reviewed-on: https://chromium-review.googlesource.com/c/1407270Reviewed-by:
Ian Kilpatrick <ikilpatrick@chromium.org> Reviewed-by:
-
Antoine Labour authored
This allows further reduce our dependency on GL. Bug: None Change-Id: Ie9d606bbaabda7ee0c0055750653ef1281fedf63 Reviewed-on: https://chromium-review.googlesource.com/c/1406290Reviewed-by:
Eric Karl <ericrk@chromium.org> Commit-Queue: Antoine Labour <piman@chromium.org> Cr-Commit-Position: refs/heads/master@{#622076}
-
Philip Rogers authored
BlinkGenPropertyTrees (BGPT) has been enabled for tests for some time (status="test" & status="experimental" in runtime_enabled_features.json5 both enable the setting for tests). This patch removes the flag-specific expectations which are unused. Bug: 836886 Change-Id: I02f58268f5738e6c242cfadba8dd72c8aed87d0c Reviewed-on: https://chromium-review.googlesource.com/c/1405428Reviewed-by:
-
Eugene But authored
NavigationItemCommitted is deprecated and should be removed. DidFinishNavigation is more general purpose method, which is called when navigation is committed, replaced or aborted. Bug: 781534 Change-Id: I77d79970548e4d5291b9953c09490fddb50c6d72 Reviewed-on: https://chromium-review.googlesource.com/c/1406076Reviewed-by:
Danyao Wang <danyao@chromium.org> Commit-Queue: Eugene But <eugenebut@chromium.org> Cr-Commit-Position: refs/heads/master@{#622074}
-
Jan Krcal authored
This CL simplifies debugging sync integration tests (and possibly other tasks). Bug: 853688 Change-Id: I4a72855cc400d33743cf8fc00979270af2ac92e0 Reviewed-on: https://chromium-review.googlesource.com/c/1406972Reviewed-by:
Sebastien Seguin-Gagnon <sebsg@chromium.org> Commit-Queue: Jan Krcal <jkrcal@chromium.org> Cr-Commit-Position: refs/heads/master@{#622073}
-
Koji Ishii authored
This patch changes LayoutText to use |DirtyLinesFromChangedChild()| when it is removed. This change makes LayoutText behavior the same as LayoutInline in this regard. Before the change, LayoutText marked only the line it belongs to. When it is removed, however, it should mark the line its previous sibling belongs to. E.g., line 1<br>line 2 and removing "line 2" should invalidate the 1st line because its break token needs to be updated. Another example: <span>foo part</span>OfLongWord and removing "OfLongWord" may make "part" to fit to the previous line. There are tests that remove <span>s, but there were no tests that remove text nodes unfortunately. Given it turned out that we have separate code for each type on removal, this patch adds some tests, but will add more in following patches. Bug: 918812 Change-Id: I6ada5c9d9dbec51d03ed3f1fc9db8d70c031bdd8 Reviewed-on: https://chromium-review.googlesource.com/c/1405127Reviewed-by:
-
Wez authored
These tests were flakily hanging, with unsufficient diagnostic output for us to diagnose the issue. Now that we have per-Run() timeouts we can re-enable these tests to get diagnostic call-stacks when they hang. Bug: 913037 Change-Id: I5ef11d59e78304ac7182e3953d7a693f7ec502e4 Reviewed-on: https://chromium-review.googlesource.com/c/1406139 Commit-Queue: Wez <wez@chromium.org> Reviewed-by:
Sergey Ulanov <sergeyu@chromium.org> Cr-Commit-Position: refs/heads/master@{#622071}
-
Bin Wu authored
In QUIC, add WriteStatus WRITE_STATUS_BLOCKED_DATA_BUFFERED and use it to represent "A WritePacket caused socket to block and the packet is buffered in writer". Not protected, because this status code is not used in GFE. This is intended to be used by batch packet writers, for them, whether the packet that caused write block is buffered can vary from one WritePacket call to another. The per-class IsWriteBlockedDataBuffered() method is not enough to express that. Merge internal change: 228872041 R=rch@chromium.org Change-Id: I39bdd8f6d1b9e7442f7d51c54231002227790809 Reviewed-on: https://chromium-review.googlesource.com/c/1407030Reviewed-by:
Ryan Hamilton <rch@chromium.org> Commit-Queue: Bin Wu <wub@chromium.org> Cr-Commit-Position: refs/heads/master@{#622070}
-
Wez authored
Un-filter three tests which were flaking ~1 year ago: - DiskCacheBackendTest.InvalidEntry5 - DiskCacheEntryTest.ExternalAsyncIONoBuffer - URLRequestTest.NetworkDelegateProxyError There have been substantial filesystem and network changes in the platform, and we have enabled TestLauncher retries on the Fuchsia bots, so these should be safe to start running. Bug: 784448, 810521, 814811 Change-Id: I7166ab60ee8a5a8dc7097c0969da8c7bc57878b8 Reviewed-on: https://chromium-review.googlesource.com/c/1405595Reviewed-by:
-
Daniel Rubery authored
Since every file in a ZIP must have a valid Local File Header, we can bound the number of files in a ZIP by the number of Local File Headers. This should exactly match the number of files extracted by ZipReader, unless something fishy is going on. Bug: 873766 Change-Id: Idba1788ebd3dada29a9e847569f0af692e097c8c Reviewed-on: https://chromium-review.googlesource.com/c/1404056 Commit-Queue: Daniel Rubery <drubery@chromium.org> Reviewed-by:
Ilya Sherman <isherman@chromium.org> Reviewed-by:
Nathan Parker <nparker@chromium.org> Cr-Commit-Position: refs/heads/master@{#622068}
-
James Cook authored
The check causes low-level browser_test flake across a wide variety of test suites, usually due to check failures during test suite teardown. This is test-only code, so relax the check until the root cause can be investigated further. Bug: 466089, 871010, 913006, 915360, 919347 Test: browser_tests Change-Id: Ife2ab6a6d260471f79873b6f0fd2914a4c7bb71d Reviewed-on: https://chromium-review.googlesource.com/c/1403353Reviewed-by:
Camille Lamy <clamy@chromium.org> Commit-Queue: James Cook <jamescook@chromium.org> Cr-Commit-Position: refs/heads/master@{#622067}
-
Marijn Kruisselbrink authored
mojo::DataPipe should not be used in production code since it provides no way to deal with errors while creating the pipe. So switch usage of that class to instead just call CreateDataPipe directly. Bug: 918054 Change-Id: Idcae7fe5b587b88ea534474f0a4de7bf8113d370 Reviewed-on: https://chromium-review.googlesource.com/c/1406134Reviewed-by:
Ken Rockot <rockot@google.com> Commit-Queue: Marijn Kruisselbrink <mek@chromium.org> Cr-Commit-Position: refs/heads/master@{#622066}
-
Adithya Srinivasan authored
Tbr: changwan@chromium.org No-Try: true Bug: 921089 Change-Id: I23cd864d189a5b5e9b52a7a169ec576e479f36fe Reviewed-on: https://chromium-review.googlesource.com/c/1407371Reviewed-by:
Adithya Srinivasan <adithyas@chromium.org> Commit-Queue: Adithya Srinivasan <adithyas@chromium.org> Cr-Commit-Position: refs/heads/master@{#622065}
-
barkerd authored
This is possible because the implementation of the postMessage API was completed in https://chromium-review.googlesource.com/c/chromium/src/+/1323669. Bug: 898840 Change-Id: Ib3bfc9a9be1ea70be760204715b86ea8053e9ea8 Reviewed-on: https://chromium-review.googlesource.com/c/1406686Reviewed-by:
Michael van Ouwerkerk <mvanouwerkerk@chromium.org> Commit-Queue: David Barker <barkerd@google.com> Cr-Commit-Position: refs/heads/master@{#622064}
-
Sergey Ulanov authored
The new version adds vulkan_fuchsia.h Bug: 861853 Change-Id: Iaad12a3ce67fbdb5908abc0ea5aebce3e8d40b55 Reviewed-on: https://chromium-review.googlesource.com/c/1406308 Commit-Queue: Sergey Ulanov <sergeyu@chromium.org> Reviewed-by:
Antoine Labour <piman@chromium.org> Cr-Commit-Position: refs/heads/master@{#622063}
-
