- 05 Dec, 2017 40 commits
-
-
Sebastien Marchand authored
Also moves the tab_stats_data_store tests to unit_tests as they're not browser tests. I'm adding some new stats to this class and having the browser/unit_tests separation will ease the unittesting. Bug: 720131 Change-Id: I9a2a9c06e0968ef9b537e2eb119934590038d3f0 Reviewed-on: https://chromium-review.googlesource.com/804435 Commit-Queue: Sébastien Marchand <sebmarchand@chromium.org> Reviewed-by:
Alexei Svitkine <asvitkine@chromium.org> Cr-Commit-Position: refs/heads/master@{#521885}
-
Sean Kau authored
Read the new external policy, enforce the selected policies, provide printers to clients. Code path is not connected yet. Bug: 758680 Change-Id: I178d0afd1e0e60d5c0cdaa8adfebb14d1d5d7070 Reviewed-on: https://chromium-review.googlesource.com/677865Reviewed-by:
Xiaoqian Dai <xdai@chromium.org> Commit-Queue: Sean Kau <skau@chromium.org> Cr-Commit-Position: refs/heads/master@{#521884}
-
Hiroshige Hayashizaki authored
Preparation for removing ScriptSourceCode::GetResource(). Bug: 788828 Change-Id: Ic1196997f2b7a0dd6f7febca476d45c4f4128598 Reviewed-on: https://chromium-review.googlesource.com/786891 Commit-Queue: Hiroshige Hayashizaki <hiroshige@chromium.org> Reviewed-by:
Kouhei Ueno <kouhei@chromium.org> Cr-Commit-Position: refs/heads/master@{#521883}
-
Derek Cheng authored
This reverts commit 363560ef. Reason for revert: The revert broke compile. Will reland and disable the failing test instead. Original change's description: > Revert "Run constraints processing for screen capture on the main thread." > > This reverts commit 112c4ff7. > > Reason for revert: This might have broken browser_tests on Mac ASan 64 Tests (1): https://uberchromegw.corp.google.com/i/chromium.memory/builders/Mac%20ASan%2064%20Tests%20%281%29/builds/36626 > > Original change's description: > > Run constraints processing for screen capture on the main thread. > > > > Some code paths require access to Blink strings that can only be > > accessed on the main thread. > > A test that triggers the bug is coming up on crrev.com/c/738040. > > > > Bug: 791992 > > Change-Id: I55ae617ae623766d4bd975a19070bde410d8af1d > > Reviewed-on: https://chromium-review.googlesource.com/809005 > > Reviewed-by: Henrik Boström <hbos@chromium.org> > > Commit-Queue: Guido Urdaneta <guidou@chromium.org> > > Cr-Commit-Position: refs/heads/master@{#521708} > > TBR=hbos@chromium.org,guidou@chromium.org > > Change-Id: Ibe39bacdedfbf54fac48a6469ec59c4f32381fd1 > No-Presubmit: true > No-Tree-Checks: true > No-Try: true > Bug: 791992 > Reviewed-on: https://chromium-review.googlesource.com/809240 > Reviewed-by: Derek Cheng <imcheng@chromium.org> > Commit-Queue: Derek Cheng <imcheng@chromium.org> > Cr-Commit-Position: refs/heads/master@{#521872} TBR=hbos@chromium.org,imcheng@chromium.org,guidou@chromium.org Change-Id: Ia5cf4ca4e14d3f83aa93c6e9aad71c3348a2467d No-Presubmit: true No-Tree-Checks: true No-Try: true Bug: 791992 Reviewed-on: https://chromium-review.googlesource.com/809887Reviewed-by:
Derek Cheng <imcheng@chromium.org> Commit-Queue: Derek Cheng <imcheng@chromium.org> Cr-Commit-Position: refs/heads/master@{#521882}
-
CrystalFaith authored
Sample extension that uses command API to give user alternate choice of keyboard shortcuts to switch between tabs. Also moves existing sample command extension into a new directory. Bug: None Change-Id: Ia5c4ae7134c51a1730260a5f23f23c091231ff38 Reviewed-on: https://chromium-review.googlesource.com/749115 Commit-Queue: Crystal Lambert <crystallambert@chromium.org> Reviewed-by:
Devlin <rdevlin.cronin@chromium.org> Cr-Commit-Position: refs/heads/master@{#521881}
-
Lei Zhang authored
Originally, it was only used when Print Preview is enabled, but the original user no longer exists, and the only user is the Settings page. Transitively only build settings::PrintingHandler when printing is enabled. BUG=789790 Change-Id: I8e8b9d0df66b1a5ead01abeeaad78982f16e61ec Reviewed-on: https://chromium-review.googlesource.com/806447Reviewed-by:
Dave Schuyler <dschuyler@chromium.org> Commit-Queue: Lei Zhang <thestig@chromium.org> Cr-Commit-Position: refs/heads/master@{#521880}
-
Michael Thiessen authored
Currently we've got some bugs that can lead to crashes on standalone VR headsets (https://vr.google.com/daydream/standalonevr/), so we should detect devices that boot into VR, and disable VR there. This will make sure we just use the fallback 2D-in-VR path that is supposed to work for all apps :) This also fixes a bug in scheduling rAF when presentation is rejected that lead to the rAF loop doubling up every time presentation was rejected. Bug: 791090 Change-Id: I7511299935486dd018b558f3489ddd00f314ba8e Reviewed-on: https://chromium-review.googlesource.com/809194 Commit-Queue: Michael Thiessen <mthiesse@chromium.org> Reviewed-by:
Brandon Jones <bajones@chromium.org> Cr-Commit-Position: refs/heads/master@{#521879}
-
skia-deps-roller@chromium.org authored
https://skia.googlesource.com/skia.git/+log/06ab3836f779..a3e9271ec41d $ git log 06ab3836f..a3e9271ec --date=short --no-merges --format='%ad %ae %s' 2017-12-04 csmartdalton CCPR: Eliminate texel buffers 2017-12-05 reed add serial procs to pictures 2017-12-05 scroggo Reland "Hide SkEncodedInfo" 2017-12-05 mtklein ANGLE -> Clang 2017-12-05 csmartdalton Add analytic clip FPs that read from the CCPR atlas 2017-12-05 mtklein remove mips support and bots 2017-12-05 bsalomon Avoid GrEllipseEffect for small radii on devices without 32 bit float. 2017-12-05 brianosman Keep animation running while moving the mouse in viewer (on Windows) 2017-12-05 scroggo Revert "Hide SkEncodedInfo" 2017-12-05 halcanary android_build_app: fixes 2017-12-04 brianosman Fix sample modifier keys for mouse events 2017-12-04 scroggo Hide SkEncodedInfo Created with: roll-dep src/third_party/skia The AutoRoll server is located here: https://autoroll.skia.org 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=master.tryserver.blink:linux_trusty_blink_rel;master.tryserver.chromium.linux:linux_optional_gpu_tests_rel;master.tryserver.chromium.mac:mac_optional_gpu_tests_rel;master.tryserver.chromium.win:win_optional_gpu_tests_rel TBR=rmistry@chromium.org Change-Id: I457e7c515acd46b4cc0d4ee7a974b12d815248d0 Reviewed-on: https://chromium-review.googlesource.com/809806Reviewed-by:
Skia Deps Roller <skia-deps-roller@chromium.org> Commit-Queue: Skia Deps Roller <skia-deps-roller@chromium.org> Cr-Commit-Position: refs/heads/master@{#521878}
-
Quinten Yearsley authored
This changes the WPT import script to use the URL https://build.chromium.org/deprecated/chromium/all_rotations.js for fetching sheriff rotations. Bug: 792193 Change-Id: I64bc3396bc921cc7f1e0420f872881fb2b25ebac Reviewed-on: https://chromium-review.googlesource.com/809616Reviewed-by:
Robert Ma <robertma@chromium.org> Commit-Queue: Robert Ma <robertma@chromium.org> Cr-Commit-Position: refs/heads/master@{#521877}
-
Eugene But authored
WebStateImpl will cache favicon urls in OnFaviconUrlUpdated and will call WebStateObserver::FaviconUrlUpdated in OnNavigationFinished for same document navigations. Also removed favicon urls caching in WebFaviconDriver introduced in crrev.com/c/695761. This CL is not exactly a revert of crrev.com/c/695761, because deprecated NavigationItemCommitted is replaced with DidFinishNavigation. NavigationItemCommitted is not called for push/replace state same-document navigaiton, which may cause other bugs for favicons caching. Bug: 789581 Cq-Include-Trybots: master.tryserver.chromium.mac:ios-simulator-cronet;master.tryserver.chromium.mac:ios-simulator-full-configs Change-Id: I42922ab4d7812380787e350713640efe4d599680 Reviewed-on: https://chromium-review.googlesource.com/797072Reviewed-by:
Sylvain Defresne <sdefresne@chromium.org> Reviewed-by:
Gauthier Ambard <gambard@chromium.org> Commit-Queue: Eugene But <eugenebut@chromium.org> Cr-Commit-Position: refs/heads/master@{#521876}
-
catapult-deps-roller@chromium.org authored
https://chromium.googlesource.com/catapult.git/+log/37c17ee783f3..09fc536c66c6 $ git log 37c17ee78..09fc536c6 --date=short --no-merges --format='%ad %ae %s' 2017-12-04 rnephew [Telemetry/TADA] Give StoryExpectations the ability to decode TADA expectation files 2017-12-05 simonhatch Pinpoint - Grab histogram summary metric equivalent. 2017-12-05 rnephew [Telemetry] Add expectations path to telemetry config object. 2017-12-05 rnephew [Telemetry/TADA] Fix bug where story name cannot be url. Created with: roll-dep src/third_party/catapult The AutoRoll server is located here: https://catapult-roll.skia.org 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=master.tryserver.chromium.android:android_optional_gpu_tests_rel TBR=sullivan@chromium.org Change-Id: I9aaa02600b553b062112b92852278de0b310868d Reviewed-on: https://chromium-review.googlesource.com/809586 Reviewed-by: <catapult-deps-roller@chromium.org> Commit-Queue: <catapult-deps-roller@chromium.org> Cr-Commit-Position: refs/heads/master@{#521875}
-
rbpotter authored
Make test and existing code changes as needed to incorporate native layer into the new UI. No behavior changes. Bug: 773928 Cq-Include-Trybots: master.tryserver.chromium.linux:closure_compilation Change-Id: I660a4707a2a54d75e5ca7dde2b9ec10496c5055b Reviewed-on: https://chromium-review.googlesource.com/807585Reviewed-by:
Demetrios Papadopoulos <dpapad@chromium.org> Commit-Queue: Rebekah Potter <rbpotter@chromium.org> Cr-Commit-Position: refs/heads/master@{#521874}
-
Lei Zhang authored
Change-Id: Ib087b7ea70bae6bad8c7512cdc7f6c5d84b58661 Reviewed-on: https://chromium-review.googlesource.com/809292Reviewed-by:
dsinclair <dsinclair@chromium.org> Commit-Queue: Lei Zhang <thestig@chromium.org> Cr-Commit-Position: refs/heads/master@{#521873}
-
Derek Cheng authored
This reverts commit 112c4ff7. Reason for revert: This might have broken browser_tests on Mac ASan 64 Tests (1): https://uberchromegw.corp.google.com/i/chromium.memory/builders/Mac%20ASan%2064%20Tests%20%281%29/builds/36626 Original change's description: > Run constraints processing for screen capture on the main thread. > > Some code paths require access to Blink strings that can only be > accessed on the main thread. > A test that triggers the bug is coming up on crrev.com/c/738040. > > Bug: 791992 > Change-Id: I55ae617ae623766d4bd975a19070bde410d8af1d > Reviewed-on: https://chromium-review.googlesource.com/809005 > Reviewed-by: Henrik Boström <hbos@chromium.org> > Commit-Queue: Guido Urdaneta <guidou@chromium.org> > Cr-Commit-Position: refs/heads/master@{#521708} TBR=hbos@chromium.org,guidou@chromium.org Change-Id: Ibe39bacdedfbf54fac48a6469ec59c4f32381fd1 No-Presubmit: true No-Tree-Checks: true No-Try: true Bug: 791992 Reviewed-on: https://chromium-review.googlesource.com/809240Reviewed-by:
Derek Cheng <imcheng@chromium.org> Commit-Queue: Derek Cheng <imcheng@chromium.org> Cr-Commit-Position: refs/heads/master@{#521872}
-
Scott Violet authored
The ui service DCHECKs if the root_location and location don't match. This is always the case for events coming in from the platform, but RemoteEventInjector circumvents that and could supply different locations. This makes EventInjector set the locations to match. Non-mus path does this by way of LocatedEvent::UpdateForRootTransform. BUG=786453 TEST=none Change-Id: If70378c18d354b29290d0545cfa6bf2f546a60f7 Reviewed-on: https://chromium-review.googlesource.com/809610Reviewed-by:
Elliot Glaysher <erg@chromium.org> Commit-Queue: Scott Violet <sky@chromium.org> Cr-Commit-Position: refs/heads/master@{#521871}
-
Robert Sesek authored
In this configuration, StackTrace may produce an empty result, which causes the is_set() assertion to fail. Bug: 791223, 598854 Change-Id: I4c4cabf2eb1176def431cd5798a2c80808bf8074 Reviewed-on: https://chromium-review.googlesource.com/809102Reviewed-by:
Mark Mentovai <mark@chromium.org> Commit-Queue: Robert Sesek <rsesek@chromium.org> Cr-Commit-Position: refs/heads/master@{#521870}
-
Erik Luo authored
Rendering console logs with lots of text can stall DevTools. In this CL, very long text will get truncated. A 'Show N more' button reveals the full text and a 'Copy' button copies the full text to clipboard. Screenshot: https://imgur.com/a/0dHJI Bug: 658525 Change-Id: Ic27e4836b8aaf9069f7ff0643b39ded4a83ce91f Reviewed-on: https://chromium-review.googlesource.com/597012 Commit-Queue: Erik Luo <luoe@chromium.org> Reviewed-by:
Dmitry Gozman <dgozman@chromium.org> Cr-Commit-Position: refs/heads/master@{#521869}
-
Justin Novosad authored
This change makes it possible to decouple the actions of capturing an animation frame from a canvas and sending it to the compositor. This is a prerequisite for mergin OffscreenCanvasResourceProdiver with CanvasResourceProvider. Bug: 788439, 776801 Cq-Include-Trybots: master.tryserver.chromium.linux:linux_layout_tests_slimming_paint_v2 Change-Id: Iaf4563069ede53b1bba8bde5ea8b693d4c1a2b73 Reviewed-on: https://chromium-review.googlesource.com/803945 Commit-Queue: Justin Novosad <junov@chromium.org> Reviewed-by:
Olivia Lai <xlai@chromium.org> Cr-Commit-Position: refs/heads/master@{#521868}
-
Ran Ji authored
Replace "WebApk.Permission.ChromePermissionDenied" UMA by "WebApk.Permission.ChromePermissionDenied2" It previously logs all Chrome permission denied. Fix to only include WebApk. Bug: 780934 Change-Id: I32e2d9009423365e1f10ae68dc53ff870e52ea06 Reviewed-on: https://chromium-review.googlesource.com/797431Reviewed-by:
Ilya Sherman <isherman@chromium.org> Reviewed-by:
Yaron Friedman <yfriedman@chromium.org> Commit-Queue: Xi Han <hanxi@chromium.org> Cr-Commit-Position: refs/heads/master@{#521867}
-
Hongchan Choi authored
AudioContextTest.cpp tests the audio latency and is not involved any AudioWorklet operation. Removing AudioWorkletThread addresses the data race found in the bug. Locally confirmed that TSAN with this patch does not crash with the data race any more. Bug: 792140 Change-Id: I297533a176c42fb1af5025fdcd77a1cf4e68ee56 Reviewed-on: https://chromium-review.googlesource.com/809825Reviewed-by:
Kenneth Russell <kbr@chromium.org> Reviewed-by:
Raymond Toy <rtoy@chromium.org> Commit-Queue: Hongchan Choi <hongchan@chromium.org> Cr-Commit-Position: refs/heads/master@{#521866}
-
Dmitry Skiba authored
On low-end Android devices in-proc GPU is started on demand, when CompositorImpl::SetVisible(true) is called for the first time. Due to the fact that native / Java messages on Android are not strictly ordered (see crbug.com/788008) sometimes GPU startup is delayed for too long, stalling the renderer (see the bug). We already have code to start GPU early (from BrowserThreadsStarted), but that code includes explicit !UsingInProcessGpu() check. However, the check (which was there from 2012), seems obsolete now. As sievers@ and boliu@ discussed in crbug.com/450396#c4 the check was there to avoid a race setting ChildProcess::current, but became obsolete once ChildProcess::current switched to TLS. This CL removes !UsingInProcessGpu() check, causing in-proc GPU to start early. Bug: 787647 Change-Id: Ifb6764e8b479003412674ce667dd6c3ea08dcb43 Reviewed-on: https://chromium-review.googlesource.com/804144Reviewed-by:
Antoine Labour <piman@chromium.org> Commit-Queue: Dmitry Skiba <dskiba@chromium.org> Cr-Commit-Position: refs/heads/master@{#521865}
-
Xiaocheng Hu authored
This is a reland of e62d46f6 The original change was reverted due to failures in the new unit tests, which assume box fragment creation for inline elements <b>, <i> or <u>, which however is not true. This reland ensures box fragment creation by adding border to those elements. Original change's description: > [LayoutNG] Introduce NGInlineFragmentTraversal in place of NGInlineFragmentIterator > > This patch introduces a utility class NGInlineFragmentTraversal, so that we can > conveniently retrieve: > - ancestors of a given physical fragment in a container > - descendants of a given physical fragment > - physical fragments in a container with the given layout object > > The utility class supersedes the existing NGInlineFragmentIterator class. Follow > up patches will convert clients of NGInlineFragmentIterator to use the traversal > class instead. > > Bug: 636993 > Cq-Include-Trybots: master.tryserver.chromium.linux:linux_layout_tests_layout_ng > Change-Id: I7f3f553df18291df50f60075ab84294dd394d86c > Reviewed-on: https://chromium-review.googlesource.com/807758 > Reviewed-by: Koji Ishii <kojii@chromium.org> > Commit-Queue: Xiaocheng Hu <xiaochengh@chromium.org> > Cr-Commit-Position: refs/heads/master@{#521758} Tbr: eae@chromium.org, kojii@chromium.org, yoichio@chromium.org, yosin@chromium.org Bug: 636993 Change-Id: I616a89ac0ca1360681e47bbc45317ffca5d99d90 Cq-Include-Trybots: master.tryserver.chromium.linux:linux_layout_tests_layout_ng Reviewed-on: https://chromium-review.googlesource.com/809302Reviewed-by:
Xiaocheng Hu <xiaochengh@chromium.org> Commit-Queue: Xiaocheng Hu <xiaochengh@chromium.org> Cr-Commit-Position: refs/heads/master@{#521864}
-
Manuel Rego Casasnovas authored
This patch makes that when you select empty lines on a contenteditable element, the selection is painted similar to what happens on a textarea. There are 2 changes to make this happen. On one side, newlines are no longer marked as InlineBox::KnownToHaveNoOverflow(). On the other side, InlineFlowBox::ComputeOverflow() has been modified in order to extend the overflow rect for newlines. With this the behavior of selection in contenteditable elements and textareas is more consistent. BUG=789508 Cq-Include-Trybots: master.tryserver.chromium.linux:linux_layout_tests_slimming_paint_v2 Change-Id: I55f33e8c688a92e760e3d359c2cf54deacd005af Reviewed-on: https://chromium-review.googlesource.com/793036Reviewed-by:
Chris Harrelson <chrishtr@chromium.org> Reviewed-by:
yosin (OOO Dec 11 to Jan 8) <yosin@chromium.org> Commit-Queue: Manuel Rego Casasnovas <rego@igalia.com> Cr-Commit-Position: refs/heads/master@{#521863}
-
v8-autoroll authored
Summary of changes available at: https://chromium.googlesource.com/v8/v8/+log/b88605f7..7af7216b Please follow these instructions for assigning/CC'ing issues: https://github.com/v8/v8/wiki/Triaging%20issues Please close rolling in case of a roll revert: https://v8-roll.appspot.com/ This only works with a Google account. CQ_INCLUDE_TRYBOTS=master.tryserver.blink:linux_trusty_blink_rel;master.tryserver.chromium.linux:linux_optional_gpu_tests_rel;master.tryserver.chromium.mac:mac_optional_gpu_tests_rel;master.tryserver.chromium.win:win_optional_gpu_tests_rel;master.tryserver.chromium.android:android_optional_gpu_tests_rel TBR=hablich@chromium.org,machenbach@chromium.org,kozyatinskiy@chromium.org,sergiyb@chromium.org Change-Id: If31e6aa7c3984e6be8324f3101055e00206f1958 Reviewed-on: https://chromium-review.googlesource.com/809193Reviewed-by:
v8 autoroll <v8-autoroll@chromium.org> Commit-Queue: v8 autoroll <v8-autoroll@chromium.org> Cr-Commit-Position: refs/heads/master@{#521862}
-
Daniel Park authored
Records infobar displayed when a survey is accepted. Calls closeInfoBar instead of onCloseButtonClicked when accepting a survey to prevent logging the new action. Bug: 791752 Change-Id: I488720882832ee20946cc1a56937d234c4cce620 Reviewed-on: https://chromium-review.googlesource.com/806367Reviewed-by:
Theresa <twellington@chromium.org> Reviewed-by:
Matthew Jones <mdjones@chromium.org> Reviewed-by:
Jesse Doherty <jwd@chromium.org> Commit-Queue: Daniel Park <danielpark@chromium.org> Cr-Commit-Position: refs/heads/master@{#521861}
-
Gregory Chatzinoff authored
Update the New Tab Bubble Promo String to "Add tabs and switch between pages". Bug: 779656 Cq-Include-Trybots: master.tryserver.chromium.mac:ios-simulator-cronet;master.tryserver.chromium.mac:ios-simulator-full-configs Change-Id: Ia42386ecc45e45ccf93105d48ff5bdd898cf4058 Reviewed-on: https://chromium-review.googlesource.com/809844Reviewed-by:
edchin <edchin@chromium.org> Commit-Queue: Gregory Chatzinoff <gchatz@chromium.org> Cr-Commit-Position: refs/heads/master@{#521860}
-
Derek Cheng authored
Revert "Reland "Enable video capture service by default on platforms where it is rolling out with M61."" This reverts commit 0fdb7a80. Reason for revert: broke webkit_layout_tests: https://uberchromegw.corp.google.com/i/chromium.webkit/builders/WebKit%20Linux%20Trusty%20Leak/builds/12888 Original change's description: > Reland "Enable video capture service by default on platforms where it is rolling out with M61." > > This is a reland of 1e2e99ef > > The reason for the revert was a Webkit layout test failing with this CL. With > https://chromium-review.googlesource.com/c/chromium/src/+/806442 landed to fix that test, > this CL should now be able to reland without modification. > > Original change's description: > > Enable video capture service by default on platforms where it is rolling > > out with M61. > > > > Bug: 721812 > > Change-Id: I1de0f625a1fb106e2dea664675e94267e66fc2fd > > Reviewed-on: https://chromium-review.googlesource.com/719416 > > Commit-Queue: Christian Fremerey <chfremer@chromium.org> > > Reviewed-by: Emircan Uysaler <emircan@chromium.org> > > Cr-Commit-Position: refs/heads/master@{#517761} > > TBR=emircan@chromium.org > > Bug: 721812 > Change-Id: If69cdb660b0b97a28084b6bd0610234d5abb232f > Reviewed-on: https://chromium-review.googlesource.com/809530 > Reviewed-by: Christian Fremerey <chfremer@chromium.org> > Commit-Queue: Christian Fremerey <chfremer@chromium.org> > Cr-Commit-Position: refs/heads/master@{#521800} TBR=emircan@chromium.org,chfremer@chromium.org Change-Id: Iee20ddb4dabf156cd3709151792cc374b0b50194 No-Presubmit: true No-Tree-Checks: true No-Try: true Bug: 721812 Reviewed-on: https://chromium-review.googlesource.com/809770Reviewed-by:
Derek Cheng <imcheng@chromium.org> Commit-Queue: Derek Cheng <imcheng@chromium.org> Cr-Commit-Position: refs/heads/master@{#521859}
-
Takumi Fujimoto authored
This is a reland of 52dda118 Original change's description: > [Media Router] Better sink availability tracking > > - When sink availability changes for an MRP, start/stop observing sinks only for that MRP > - Make WiredDisplayMRP report sink availability updates > > Bug: 789277,777650 > Change-Id: I3d1a087ca39a4830403d6d2f0f00bf1411388f33 > Reviewed-on: https://chromium-review.googlesource.com/794106 > Reviewed-by: Derek Cheng <imcheng@chromium.org> > Commit-Queue: Takumi Fujimoto <takumif@chromium.org> > Cr-Commit-Position: refs/heads/master@{#521550} Bug: 789277, 777650 Change-Id: I0a2edc4af121fa675548f9d545bb6048b05e2f13 Reviewed-on: https://chromium-review.googlesource.com/809333Reviewed-by:
Derek Cheng <imcheng@chromium.org> Commit-Queue: Takumi Fujimoto <takumif@chromium.org> Cr-Commit-Position: refs/heads/master@{#521858}
-
Yuke Liao authored
This CL adds support to filter code coverage results by list of directories or files so that the script can run faster without having to generate a report for every executed file. Bug: 789691 Change-Id: I2e954872b7f1113cd04b8e609ffdabf0578db762 Reviewed-on: https://chromium-review.googlesource.com/809295 Commit-Queue: Yuke Liao <liaoyuke@chromium.org> Reviewed-by:
Abhishek Arya <inferno@chromium.org> Reviewed-by:
Max Moroz <mmoroz@chromium.org> Cr-Commit-Position: refs/heads/master@{#521857}
-
Wez authored
Previously the targets to be compiled (but not run) were specified explicitly, effectively duplicating the logic already present in the GN build configuration files. Bug: 707031 Change-Id: I8c6715f9f862a04a49c81030ba9d0212f0468ec9 Reviewed-on: https://chromium-review.googlesource.com/802189 Commit-Queue: Wez <wez@chromium.org> Reviewed-by:
John Budorick <jbudorick@chromium.org> Reviewed-by:
Dirk Pranke <dpranke@chromium.org> Reviewed-by:
Scott Graham <scottmg@chromium.org> Cr-Commit-Position: refs/heads/master@{#521856}
-
Jacob Dufault authored
This is more consistent. Bug: 784495 Change-Id: I39bdff1cf126d6d3abf245620ce3c803d382db34 Reviewed-on: https://chromium-review.googlesource.com/806644 Commit-Queue: Jacob Dufault <jdufault@chromium.org> Reviewed-by:
Achuith Bhandarkar <achuith@chromium.org> Cr-Commit-Position: refs/heads/master@{#521855}
-
Steven Bennetts authored
This change requests focusing of the first item after selecting an item. Generally that should focus the newly connecting item as soon as the list is updated. Bug: 739633 Cq-Include-Trybots: master.tryserver.chromium.linux:closure_compilation Change-Id: I728658d1bbc916c56793960c576e47fe5bc91354 Reviewed-on: https://chromium-review.googlesource.com/806440Reviewed-by:
Alexander Alekseev <alemate@chromium.org> Commit-Queue: Steven Bennetts <stevenjb@chromium.org> Cr-Commit-Position: refs/heads/master@{#521854}
-
Steven Holte authored
Bug: 754889 Change-Id: I60bc810728bed4759dac3a406f0eae36a1e11820 Reviewed-on: https://chromium-review.googlesource.com/794491Reviewed-by:
Bryan McQuade <bmcquade@chromium.org> Commit-Queue: Steven Holte <holte@chromium.org> Cr-Commit-Position: refs/heads/master@{#521853}
-
Sky Malice authored
This avoid hitting a DCHECK that was introduced in https://chromiumcodereview.appspot.com/2401083003 where we tried clean up notifications to the model thread about encryption information. By re-using the existing ApplyUpdates methods (and DCHECKs) we now realize that we sometimes call ApplyUpdates before the initial sync is completed. This is probably wrong and definitely, since the initial sync will call ApplyUpdates and pass encryption information. Sending it before that will just be confusing for the proc. Bug: 789950 Change-Id: Ifc2b01efbc71145574f9106c67ce9aebfc8f6b99 Reviewed-on: https://chromium-review.googlesource.com/807664Reviewed-by:
Pavel Yatsuk <pavely@chromium.org> Reviewed-by:
Mikel Astiz <mastiz@chromium.org> Commit-Queue: Sky Malice <skym@chromium.org> Cr-Commit-Position: refs/heads/master@{#521852}
-
Chris Bentzel authored
Bug: 167187 Change-Id: If36e8f8eb3667a505818edec7cdff3bb36bd8831 Reviewed-on: https://chromium-review.googlesource.com/804663Reviewed-by:
Eric Roman <eroman@chromium.org> Reviewed-by:
Will Harris <wfh@chromium.org> Commit-Queue: Chris Bentzel <cbentzel@chromium.org> Cr-Commit-Position: refs/heads/master@{#521851}
-
Sergey Ulanov authored
Added kNoClockless flag in the test. This is expected to fix the flake. Bug: 788387 Change-Id: Ife4a443ef30d4f9b656e52ece3c1025cee5c0a96 Reviewed-on: https://chromium-review.googlesource.com/809507 Commit-Queue: Sergey Ulanov <sergeyu@chromium.org> Reviewed-by:
Dale Curtis <dalecurtis@chromium.org> Cr-Commit-Position: refs/heads/master@{#521850}
-
Wez authored
This requires fixing NativeValueTraitsImplTest.IDLCallbackFunction to use ASSERT_DEATH_IF_SUPPORTED. The conditional handling for Official vs normal builds is also removed, for simplicity. Bug: 779820, 707031 Change-Id: Ib2563d8b6ca792ce916d7be907956d33329e4ba4 Reviewed-on: https://chromium-review.googlesource.com/802355 Commit-Queue: Wez <wez@chromium.org> Reviewed-by:
Jochen Eisinger <jochen@chromium.org> Cr-Commit-Position: refs/heads/master@{#521849}
-
pdfium-deps-roller@chromium.org authored
https://pdfium.googlesource.com/pdfium.git/+log/12ec6760afd9..c45271e053c8 $ git log 12ec6760a..c45271e05 --date=short --no-merges --format='%ad %ae %s' 2017-12-05 thestig Remove redundant field in struct FXCMAP_CMap. 2017-12-05 thestig Fix CXFA_SimpleParser member destruction order. 2017-12-05 thestig Avoid integer overflows in CPDF_FixedMatrix::Transform(). Created with: roll-dep src/third_party/pdfium BUG=791616,791048 The AutoRoll server is located here: https://pdfium-roll.skia.org 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=dsinclair@chromium.org Change-Id: Id9a3a2c72218d95099dc5a350d51e2050758e565 Reviewed-on: https://chromium-review.googlesource.com/809312 Reviewed-by: <pdfium-deps-roller@chromium.org> Commit-Queue: <pdfium-deps-roller@chromium.org> Cr-Commit-Position: refs/heads/master@{#521848}
-
Kenneth Russell authored
These machines were moved to the chromium.android waterfall a while back but their JSON entries were forgotten. Remove the definitions of these bots and the test suite exceptions referring to them. (The presubmit checks caught all of the stray exceptions.) Add cast_junit_test_lists to Cast Android (dbg) on chromium.android, which is presuambly where it should have been placed during this migration. BUG=662541 Change-Id: Iaa0453b17e06e31096ecf4e770557ad6aa7d7fd6 Reviewed-on: https://chromium-review.googlesource.com/809633Reviewed-by:
John Budorick <jbudorick@chromium.org> Reviewed-by:
Dirk Pranke <dpranke@chromium.org> Commit-Queue: Kenneth Russell <kbr@chromium.org> Cr-Commit-Position: refs/heads/master@{#521847}
-
Doug Arnett authored
Bug: 792133 Change-Id: Iaa99ae16b359560f60e23c07cace1241781b6bbd Reviewed-on: https://chromium-review.googlesource.com/809296Reviewed-by:
Tarun Bansal <tbansal@chromium.org> Reviewed-by:
Matt Menke <mmenke@chromium.org> Commit-Queue: Doug Arnett <dougarnett@chromium.org> Cr-Commit-Position: refs/heads/master@{#521846}
-