- 29 Oct, 2018 40 commits
- 
- 
Yuly Novikov authoredconformance/rendering/gl-scissor-fbo-test.html deqp/data/gles2/shaders/swizzles.html on Win NVIDIA Passthrough Vulkan Debug. TBR=kbr@chromium.org Bug: angleproject:2939, angleproject:2940 Change-Id: I8b388eb464b42def465c231d5986852a6517e19a Reviewed-on: https://chromium-review.googlesource.com/c/1305237Reviewed-by: Yuly Novikov <ynovikov@chromium.org> Commit-Queue: Yuly Novikov <ynovikov@chromium.org> Cr-Commit-Position: refs/heads/master@{#603561} 
- 
Mason Freed authoredWe previously used these two terms semi-interchangeably, which a) was confusing, and b) hindered code search for backdrop-filter related code. I checked carefully - this CL is a pure rename operation. No functionality should change. I did not touch "background blur" and "background zoom" in ash or ui::Layer. Those are similar concepts and names, but don't appear to use the same machinery as CSS-based backdrop-filter. Change-Id: I8f5473c4014ba47b2bbc354126c1a10f4aebc61a Reviewed-on: https://chromium-review.googlesource.com/c/1300035Reviewed-by: Philip Rogers <pdr@chromium.org> Reviewed-by: Dominick Ng <dominickn@chromium.org> Reviewed-by: ccameron <ccameron@chromium.org> Commit-Queue: Mason Freed <masonfreed@chromium.org> Cr-Commit-Position: refs/heads/master@{#603560} 
- 
Ben Pastene authoredIncreases total time sleeping from 30s to 62s. Bug: 892670 Change-Id: Ib4d631da6c24fb3bb2f9f3a9c692ed82d699b397 Reviewed-on: https://chromium-review.googlesource.com/c/1305413 Commit-Queue: Ben Pastene <bpastene@chromium.org> Reviewed-by: John Budorick <jbudorick@chromium.org> Cr-Commit-Position: refs/heads/master@{#603559} 
- 
Liquan(Max) Gu authoredWe need to report Last Text Paint and Largest Text Paint to UKM and UMA. This CL uses mojo to send the metric from web performance (Blink) to the page load metrics (Browser). The browser then reports the metric to UKM and UMA. Bug: 869924 Change-Id: I44ac017cd06df6dc5bae264442f27900a027bbe7 Reviewed-on: https://chromium-review.googlesource.com/c/1301966 Commit-Queue: Liquan (Max) Gǔ <maxlg@chromium.org> Reviewed-by: Brian White <bcwhite@chromium.org> Reviewed-by: Gayane Petrosyan <gayane@chromium.org> Reviewed-by: Bryan McQuade <bmcquade@chromium.org> Reviewed-by: Martin Barbella <mbarbella@chromium.org> Cr-Commit-Position: refs/heads/master@{#603558} 
- 
John Budorick authoredI don't want to be CCed on everything. Change-Id: I14d2af4906be4626827c7cbca7ee59fbe86a8420 Reviewed-on: https://chromium-review.googlesource.com/c/1305633 Commit-Queue: John Budorick <jbudorick@chromium.org> Commit-Queue: Ben Pastene <bpastene@chromium.org> Reviewed-by: Ben Pastene <bpastene@chromium.org> Cr-Commit-Position: refs/heads/master@{#603557} 
- 
Stephane Zermatten authoredBefore this change, WebController::ElementExists always checked for both existence and visibility. With this change, WebController::ElementExists checks for existence, and WebController::ElementVisible for visibility. Element existence is used for preconditions, visibility is used for all other cases, such as when waiting for dom or waiting for an element to act on. This change also simplifies BatchElementChecker a bit by keeping element existence, visibility and field checks completely separate. Bug: 806868 Change-Id: Ibd232c317aff4ba4640e5c47a0a93a2b4637ec76 Reviewed-on: https://chromium-review.googlesource.com/c/1301783 Commit-Queue: Stephane Zermatten <szermatt@chromium.org> Reviewed-by: Ganggui Tang <gogerald@chromium.org> Cr-Commit-Position: refs/heads/master@{#603556} 
- 
Michael Moss authored- win_chrome_official moved to luci.chrome.try - chromium.chrome bots are moving to luci.chrome.ci R=hinoka@chromium.org Bug: 790153,790158,790159,790163,731383 Change-Id: Ida6dbf9cc2b7dd6d7dcd944e641863443e562946 Reviewed-on: https://chromium-review.googlesource.com/c/1302862Reviewed-by: Ryan Tseng <hinoka@chromium.org> Commit-Queue: Michael Moss <mmoss@chromium.org> Cr-Commit-Position: refs/heads/master@{#603555} 
- 
Jim Van Verth authoredSkia has revised the SkImageGenerator interface. SkYUVSizeInfo has been expanded to support alpha and renamed to SkYUVASizeInfo. In addition, the API adds the struct SkYUVAIndex to represent the mapping from a YUVA channel to the texture index and RGBA channel that represents it. This allows interleaved formats (e.g. NV12) to be represented. This CL updates Chromium to use the new interface. Bug: skia:7903 Change-Id: I64a7056e535a114b4cfc0f86c93f5e6cd2c91bd3 Reviewed-on: https://chromium-review.googlesource.com/c/1297254Reviewed-by: Fernando Serboncini <fserb@chromium.org> Reviewed-by: Khushal <khushalsagar@chromium.org> Reviewed-by: Antoine Labour <piman@chromium.org> Reviewed-by: Dan Sanders <sandersd@chromium.org> Reviewed-by: Stephen Chenney <schenney@chromium.org> Commit-Queue: Jim Van Verth <jvanverth@chromium.org> Cr-Commit-Position: refs/heads/master@{#603554} 
- 
Vladislav Kaznacheev authoredArcAppDataSearchProvider used to be created and called unconditionally which is not optimal. Play Store search is already protected by a (disabled by default) feature. Adding a flag makes it easier to debug and test. Bug: 818902 Test: manual, AboutFlagsHistogramTest Change-Id: Ifb20d90de4e1f977b320fd4f14079570bff367bf Reviewed-on: https://chromium-review.googlesource.com/c/1300293 Commit-Queue: Vladislav Kaznacheev <kaznacheev@chromium.org> Reviewed-by: Xiyuan Xia <xiyuan@chromium.org> Cr-Commit-Position: refs/heads/master@{#603553} 
- 
Tommy C. Li authoredThe "toolbar" component would be most accurately called the "location_bar" component. However, we don't have any location bars in Chrome that are not also an Omnibox. Colloquially, most users and engineers refer to the whole location bar, not just the textfield as the "Omnibox". Moreover, the toolbar model provides data that feeds both into the "Omnibox" text portion as well as the exterior "Location Bar". A followup CL will likely rename ToolbarModel to OmniboxModel. Bug: 898347 Cq-Include-Trybots: luci.chromium.try:android_optional_gpu_tests_rel;luci.chromium.try:ios-simulator-cronet;luci.chromium.try:ios-simulator-full-configs;luci.chromium.try:linux_optional_gpu_tests_rel;luci.chromium.try:linux_vr;luci.chromium.try:mac_optional_gpu_tests_rel;luci.chromium.try:win_optional_gpu_tests_rel Change-Id: I7123286da779afe6485f7c324b7fb84a72889601 Reviewed-on: https://chromium-review.googlesource.com/c/1297632 Commit-Queue: Tommy Li <tommycli@chromium.org> Reviewed-by: Mark Cogan <marq@chromium.org> Reviewed-by: Jochen Eisinger <jochen@chromium.org> Reviewed-by: Eugene But <eugenebut@chromium.org> Reviewed-by: Peter Kasting <pkasting@chromium.org> Cr-Commit-Position: refs/heads/master@{#603552} 
- 
Mike Klein authoredTBR=fmalita@chromium.org This will guard https://skia-review.googlesource.com/c/skia/+/165766. Change-Id: I6cf70a5d2e0eab1698c534bf8ccf8fdc6f81e19a Reviewed-on: https://chromium-review.googlesource.com/c/1305153Reviewed-by: Mike Klein <mtklein@chromium.org> Commit-Queue: Mike Klein <mtklein@chromium.org> Cr-Commit-Position: refs/heads/master@{#603551} 
- 
Patrick Monette authoredCreate a new function in shell_util_win.mojom to replace the old CallGetOpenFileName and CallGetSaveFileName functions that are no longer implemented. Make use of that function in chrome_select_file_dialog_factory Bug: 73098,884075 Change-Id: Ia3176fe3aefe26ea08f518332454e5637796b410 Reviewed-on: https://chromium-review.googlesource.com/c/1241563 Commit-Queue: Patrick Monette <pmonette@chromium.org> Reviewed-by: Daniel Cheng <dcheng@chromium.org> Reviewed-by: Robert Kaplow (OOO until Nov6) <rkaplow@chromium.org> Reviewed-by: Scott Violet <sky@chromium.org> Cr-Commit-Position: refs/heads/master@{#603550} 
- 
Jun Mukai authoredWith recent fixes of myself, this test now succeeds. BUG=890071 TEST=none R=sky@chromium.org Change-Id: I73e8918771225d9855cbf6299d3c852476adf707 Reviewed-on: https://chromium-review.googlesource.com/c/1305194Reviewed-by: Scott Violet <sky@chromium.org> Commit-Queue: Jun Mukai <mukai@chromium.org> Cr-Commit-Position: refs/heads/master@{#603549} 
- 
Fredrik Söderquist authoredMatches what we do in SVGIntegerOptionalInteger::CalculateAnimatedValue, and avoids undefined overflow. Bug: 899445 Change-Id: I1250a05482713780f707301ed29e015fd81e65f2 Reviewed-on: https://chromium-review.googlesource.com/c/1304483Reviewed-by: Stephen Chenney <schenney@chromium.org> Commit-Queue: Fredrik Söderquist <fs@opera.com> Cr-Commit-Position: refs/heads/master@{#603548} 
- 
Sadrul Habib Chowdhury authoredMake sure to retain a ref to the ContextProvider so that it gets destroyed on the thread it is bound on. BUG=898842 Change-Id: Iefcab772acb4cf530ff4a3e7c64e9b66e35b3f4e Reviewed-on: https://chromium-review.googlesource.com/c/1304068Reviewed-by: Antoine Labour <piman@chromium.org> Commit-Queue: Sadrul Chowdhury <sadrul@chromium.org> Cr-Commit-Position: refs/heads/master@{#603547} 
- 
v8-ci-autoroll-builder authoredSummary of changes available at: https://chromium.googlesource.com/v8/v8/+log/d2036a9f..ca9994a1 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;luci.chromium.try:linux_optional_gpu_tests_rel;luci.chromium.try:mac_optional_gpu_tests_rel;luci.chromium.try:win_optional_gpu_tests_rel;luci.chromium.try:android_optional_gpu_tests_rel TBR=hablich@chromium.org,v8-waterfall-sheriff@grotations.appspotmail.com Change-Id: Ibf5f71746c2f689b06c51cf23c1d61e378163879 Reviewed-on: https://chromium-review.googlesource.com/c/1304933Reviewed-by: v8-ci-autoroll-builder <v8-ci-autoroll-builder@chops-service-accounts.iam.gserviceaccount.com> Commit-Queue: v8-ci-autoroll-builder <v8-ci-autoroll-builder@chops-service-accounts.iam.gserviceaccount.com> Cr-Commit-Position: refs/heads/master@{#603546} 
- 
Tibor Goldschwendt authoredThis reverts commit 0a8ce562. Reason for revert: crbug/899732 Original change's description: > Reland "[vr] On-demand install VR dynamic feature module" > > This is a reland of 0275d360 with a fix > for x86 builders, which have VR disabled. > > Original change's description: > > [vr] On-demand install VR dynamic feature module > > > > With this CL we initate the module install once the user enters VR on a > > WebXR/VR page. In more detail this CL > > > > - Introduces native VrModuleDelegate used to request VR module from > > native, > > > > - Refactors GvrDevice to request the VR module as part of its > > initialization flow, > > > > - Adds native side of the VrModuleProvider for Java-native > > communication. > > > > Bug: 862689 > > Cq-Include-Trybots: luci.chromium.try:android_optional_gpu_tests_rel;luci.chromium.try:linux_optional_gpu_tests_rel;luci.chromium.try:linux_vr;luci.chromium.try:mac_optional_gpu_tests_rel;luci.chromium.try:win_optional_gpu_tests_rel > > Change-Id: I960d2bdd4d2c64a19fa6069e457d44f77ab0e73b > > Reviewed-on: https://chromium-review.googlesource.com/c/1241458 > > Commit-Queue: Tibor Goldschwendt <tiborg@chromium.org> > > Reviewed-by: Ted Choc <tedchoc@chromium.org> > > Reviewed-by: Bill Orr <billorr@chromium.org> > > Cr-Commit-Position: refs/heads/master@{#602922} > > TBR=tedchoc@chromium.org,billorr@chromium.org > > Bug: 862689 > Change-Id: I7a0f383534b622e0e4acb5acb2a1c28e547d0970 > Cq-Include-Trybots: luci.chromium.try:android_optional_gpu_tests_rel;luci.chromium.try:linux_optional_gpu_tests_rel;luci.chromium.try:linux_vr;luci.chromium.try:mac_optional_gpu_tests_rel;luci.chromium.try:win_optional_gpu_tests_rel > Reviewed-on: https://chromium-review.googlesource.com/c/1301568 > Reviewed-by: Tibor Goldschwendt <tiborg@chromium.org> > Commit-Queue: Tibor Goldschwendt <tiborg@chromium.org> > Cr-Commit-Position: refs/heads/master@{#603100} TBR=tedchoc@chromium.org,billorr@chromium.org,tiborg@chromium.org # Not skipping CQ checks because original CL landed > 1 day ago. Bug: 862689, 899732 Change-Id: I89de56edc6c300955534a701deb92178cf50683e Cq-Include-Trybots: luci.chromium.try:android_optional_gpu_tests_rel;luci.chromium.try:linux_optional_gpu_tests_rel;luci.chromium.try:linux_vr;luci.chromium.try:mac_optional_gpu_tests_rel;luci.chromium.try:win_optional_gpu_tests_rel Reviewed-on: https://chromium-review.googlesource.com/c/1305093Reviewed-by: Tibor Goldschwendt <tiborg@chromium.org> Commit-Queue: Tibor Goldschwendt <tiborg@chromium.org> Cr-Commit-Position: refs/heads/master@{#603545} 
- 
Clifford Cheng authoredThis is a reland of 01ccf727 Original change's description: > Enable the pixel by pixel comparison test for Mac OS. > > 1. Tested on Mac and verify everything works as expected. > 2. Removed the unnecessary call of GetNativeWindow(). > > Change-Id: I377e6bfb004548887b9623b0094da91fa36b4d1c > Reviewed-on: https://chromium-review.googlesource.com/c/1279301 > Reviewed-by: Mounir Lamouri <mlamouri@chromium.org> > Commit-Queue: Clifford Cheng <cliffordcheng@chromium.org> > Cr-Commit-Position: refs/heads/master@{#601879} Change-Id: Ib6fa1c1d9b3487c88b240ea9262042db8af0c19a Reviewed-on: https://chromium-review.googlesource.com/c/1296811Reviewed-by: Mounir Lamouri <mlamouri@chromium.org> Commit-Queue: Clifford Cheng <cliffordcheng@chromium.org> Cr-Commit-Position: refs/heads/master@{#603544} 
- 
Robert Ma authoredandroid_blink_rel has not been migrated to LUCI so we still need to accept and parse BuildBot URLs in git_cl when fetching try job results. Bug: 891729 Change-Id: I758feb57cb01ced6cd20644c9508fe77bec0dd3d Reviewed-on: https://chromium-review.googlesource.com/c/1303758Reviewed-by: Stephen Martinis <martiniss@chromium.org> Commit-Queue: Robert Ma <robertma@chromium.org> Cr-Commit-Position: refs/heads/master@{#603543} 
- 
Sahel Sharify authoredBug: 821237 Change-Id: I31947ac62a27ccf140630474560a38f883f73bcf Reviewed-on: https://chromium-review.googlesource.com/c/1300098Reviewed-by: Ken Buchanan <kenrb@chromium.org> Reviewed-by: David Bokan <bokan@chromium.org> Commit-Queue: Sahel Sharify <sahel@chromium.org> Cr-Commit-Position: refs/heads/master@{#603542} 
- 
Eyor Alemayehu authoredSending the experiment is controlled by a Finch experiment. Bug: b:117847982 Test: Manual Change-Id: I542a131d931254d610c8acaf5bc0f116b4d19b60 Reviewed-on: https://chromium-review.googlesource.com/c/1291355Reviewed-by: Xiaohui Chen <xiaohuic@chromium.org> Commit-Queue: Eyor Alemayehu <eyor@google.com> Cr-Commit-Position: refs/heads/master@{#603541} 
- 
Gaurav Dhol authored../..\base/bind_internal.h(415,8): error: redefinition of 'FunctorTraits<R (*)(Args...) __attribute__((stdcall))>' Bug: 893460 Change-Id: I370eb3d4cebf3171fb143cba9b022696c5a1ebb2 Reviewed-on: https://chromium-review.googlesource.com/c/1298831 Commit-Queue: Bruce Dawson <brucedawson@chromium.org> Reviewed-by: Taiju Tsuiki <tzik@chromium.org> Cr-Commit-Position: refs/heads/master@{#603540} 
- 
Matthew Jones authoredThis patch restricts the number of exposed methods for modifying the visible state of the omnibox suggestions. Most of this logic is now driven off of the focused state of the omnibox (via UrlFocusChangeListener) and whether suggestions are available. Change-Id: I2dad7eda0331f7e69ad49ebb9c89f71748f4bd6b Reviewed-on: https://chromium-review.googlesource.com/c/1293991 Commit-Queue: Matthew Jones <mdjones@chromium.org> Reviewed-by: Ted Choc <tedchoc@chromium.org> Cr-Commit-Position: refs/heads/master@{#603539} 
- 
Yi Su authoredUse local variable to store value of DOM element to avoid modifying DOM, which may cause potential danger to event-based frontend frameworks. Bug: 433824 Change-Id: Ie7d3a9d67483738e66730a794ba798c81d515e4e Reviewed-on: https://chromium-review.googlesource.com/c/1303729Reviewed-by: Olivier Robin <olivierrobin@chromium.org> Reviewed-by: Eugene But <eugenebut@chromium.org> Commit-Queue: Yi Su <mrsuyi@chromium.org> Cr-Commit-Position: refs/heads/master@{#603538} 
- 
Jan Krcal authoredThis CL re-enables a previously flaky test on Windows. The underlying cause has been fixed so let's give it a try if it stops flaking. Bug: 434438 Change-Id: I09a85811db998960711586007005853ca4bb5499 Reviewed-on: https://chromium-review.googlesource.com/c/1304475Reviewed-by: Marc Treib <treib@chromium.org> Commit-Queue: Jan Krcal <jkrcal@chromium.org> Cr-Commit-Position: refs/heads/master@{#603537} 
- 
Kevin Ellis authoredConsider the following example: .ripple { animation: ripple 1s; background: paint(ripple-painter); } @keyframes ripple { 0% { --ripple: 0; } 100% { --ripple: 1; } } Assuming --ripple is registered as a number-valued variable, we would like to be able to animate this paint worklet off of the main thread. This change guards against potential jank between the compositor and main thread if other properties depend on the variable. For example, we want to block compositing in the following case: .ripple { animation: ripple 1s; background: paint(ripple-painter); opacity: var(--ripple); } This patch is conservative in that it does not check which variable is being referenced. Erring on the side of caution to minimize performance impact. Note that off-thread paint worklets are implemented behind a flag. Change-Id: Ied3e0f10ba055f9fb830aa886478b5d13de187d5 Reviewed-on: https://chromium-review.googlesource.com/c/1271876 Commit-Queue: Kevin Ellis <kevers@chromium.org> Reviewed-by:Stephen McGruer <smcgruer@chromium.org> Reviewed-by: Anders Ruud <andruud@chromium.org> Cr-Commit-Position: refs/heads/master@{#603536} 
- 
Tom McKee authoredThe QueueingTimeEstimator had a flag that was, effectively, a disabled/enabled flag. Instead of naming it for the external state that causes updates to the flag, name it for what the flag is used for inside the QueueingTimeEstimator. Bug: 883487 Change-Id: I73c7f71ff10fcacaf1e85a3f89f5e1a123b78896 Reviewed-on: https://chromium-review.googlesource.com/c/1296822 Commit-Queue: Tom McKee <tommckee@chromium.org> Reviewed-by: Alexander Timin <altimin@chromium.org> Cr-Commit-Position: refs/heads/master@{#603535} 
- 
Nico Weber authoredWe used to have a comment saying "Older versions of RC don't support the /nologo flag", but since we've been requiring MSVC 2017 for a long time now, using `rc /nologo` should be safe by now. No intended behavior change. Bug: none Change-Id: If13d1d3fc4647daada3af5ade526e6f774a9fb7c Reviewed-on: https://chromium-review.googlesource.com/c/1303882Reviewed-by: Scott Graham <scottmg@chromium.org> Commit-Queue: Nico Weber <thakis@chromium.org> Cr-Commit-Position: refs/heads/master@{#603534} 
- 
Nico Weber authoredWe still need the wrapper to set the env and to filter out ' Assembling: foo', but less wrapper is probably better. No intended behavior change. Bug: none Change-Id: Id8c165294cc3cf0ba7fc74f86a6a8fb9f8bfa042 Reviewed-on: https://chromium-review.googlesource.com/c/1303883Reviewed-by: Scott Graham <scottmg@chromium.org> Commit-Queue: Nico Weber <thakis@chromium.org> Cr-Commit-Position: refs/heads/master@{#603533} 
- 
David 'Digit' Turner authoredThis saves about 100 kiB on 32-bit ARM libchrome.so. This optimization cannot be applied to libmonochrome.so, unfortunately, because only Android P and above support these at the system linker level. BUG=895194 R=agrieve@chromium.org,torne@chromium.org,pasko@chromium.org,pcc@chromium.org Change-Id: I5c554dc3dbc1b3ea27edb60579a5cc0dcdc7d759 Reviewed-on: https://chromium-review.googlesource.com/c/1304487 Commit-Queue: David Turner <digit@chromium.org> Reviewed-by: agrieve <agrieve@chromium.org> Cr-Commit-Position: refs/heads/master@{#603532} 
- 
Xianzhu Wang authored- New failures: synthetic_gestures/animated-wheel-tiny-delta.html compositing/gestures/gesture-tapHighlight-with-filter.html (not spv2-specific) - New passes: Mostly about composited border radiuis. TBR=pdr@chromium.org Change-Id: I0ffd5baf139205cd06e6ef5a388efded79ebfb44 Reviewed-on: https://chromium-review.googlesource.com/c/1305214Reviewed-by: Xianzhu Wang <wangxianzhu@chromium.org> Commit-Queue: Xianzhu Wang <wangxianzhu@chromium.org> Cr-Commit-Position: refs/heads/master@{#603531} 
- 
A Olsen authoredIn ScopedCrosSettingsTestHelper, there was one call to CrosSettings::Get()->Set - this was just used for testing to set any CrosSetting to a particular value. If every call to CrosSetting::Get()->Set can be removed, that means we can delete the entire write path in DeviceSettingsProvider, which will make code simpler. That means all writes will be using OwnerSettingsService, which is better because it is profile-aware. In practise, most tests are using a StubCrosSettingsProvider, so they don't need to use the write path of DeviceSettingsService, or the OwnerSettingsService - they can just write it directly to the StubCrosSettingsProvider. This test modifies ScopedCrosSettingsTestHelper so it can also use the StubCrosSettingsProvider when it was initialized by CrosSettings itself. It also modifies 2 tests that were not using a StubCrosSettingsProvider, but need to call Set, so that now they are using a StubCrosSettingsProvider. For more background, see go/writepath Bug: 433840 Change-Id: I85a68d6ea116061b13f3c5e5582aa0298af1a386 Reviewed-on: https://chromium-review.googlesource.com/c/1299002Reviewed-by: Steven Bennetts <stevenjb@chromium.org> Reviewed-by: Maksim Ivanov <emaxx@chromium.org> Commit-Queue: A Olsen <olsen@chromium.org> Cr-Commit-Position: refs/heads/master@{#603530} 
- 
Dan Harrington authoredAdded thumbnail_url, favicon_url, snippet, and attribution. These are not yet set by any code. We anticipate wanting these fields for experiments on the net error page, and for integration with Jardin. Change-Id: I5cd7eb0f20a82e9d6af87f2f03252de80113acbc Reviewed-on: https://chromium-review.googlesource.com/c/1185914 Commit-Queue: Dan H <harringtond@google.com> Reviewed-by: Carlos Knippschild <carlosk@chromium.org> Cr-Commit-Position: refs/heads/master@{#603529} 
- 
Becca Hughes authoredAdd support for Previous/NextTrack to MediaController and the underlying MediaSession. This will be mapped to the previous and next track media keys on CrOS. BUG=893296 Change-Id: I582cd70a04c24af69c80ca891a281311b4d05434 Reviewed-on: https://chromium-review.googlesource.com/c/1287268Reviewed-by: Jochen Eisinger <jochen@chromium.org> Reviewed-by: Daniel Cheng <dcheng@chromium.org> Reviewed-by: Mounir Lamouri <mlamouri@chromium.org> Commit-Queue: Becca Hughes <beccahughes@chromium.org> Cr-Commit-Position: refs/heads/master@{#603528} 
- 
Fernando Serboncini authoredthis allows V8 to know when we are using too much memory, which triggers a GC. Bug: 897733,842693 Change-Id: I74c7589ab78a753132e54d5eac3c28793580c266 Reviewed-on: https://chromium-review.googlesource.com/c/1297067 Commit-Queue: Fernando Serboncini <fserb@chromium.org> Reviewed-by: Mohammad Reza Zakerinasab <zakerinasab@chromium.org> Cr-Commit-Position: refs/heads/master@{#603527} 
- 
chromium-internal-autoroll authoredhttps://chrome-internal.googlesource.com/chrome/src-internal.git/+log/0a47db54fa15..4fea579b6e9b Created with: gclient setdep -r src-internal@4fea579b6e9b 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: I2e203966a621c22efe2ce46ff92e16a070ba73c1 Reviewed-on: https://chromium-review.googlesource.com/c/1304379Reviewed-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@{#603526} 
- 
chromium-autoroll authoredhttps://webrtc.googlesource.com/src.git/+log/72864963fe62..12048c71501a git log 72864963fe62..12048c71501a --date=short --no-merges --format='%ad %ae %s' 2018-10-29 nisse@webrtc.org Fix error handling in hex_decode. 2018-10-29 srte@webrtc.org Adds GetSentPacket to PacketResult. 2018-10-29 srte@webrtc.org Adds unit tests for safe reset trial. Created with: gclient setdep -r src/third_party/webrtc@12048c71501a 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: I510e1a5040ff75129c0280a63e569f82e8aa5671 Reviewed-on: https://chromium-review.googlesource.com/c/1304913Reviewed-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@{#603525} 
- 
David 'Digit' Turner authoredSupporting RELR relocations, as described in [1] has the potential to drastically reduce the size of native libraries, while speeding up loading time (see related bug for details). This CL adds support for RELR relocations to the crazy linker. Note that this supports both the Android-specific dynamic table entries, and the default ones supported by lld. Note that this only modifies the linker. Another CL is required to enable RELR relocations when building native libraries for Chrome. [1] https://reviews.llvm.org/D48247 BUG=895194 R=pasko@chromium.org, agrieve@chromium.org, torne@chromium.org, pcc@chromium.org Change-Id: Ia074a2228ac7a81418fe706be6e7e57274ae18a0 Reviewed-on: https://chromium-review.googlesource.com/c/1304484 Commit-Queue: David Turner <digit@chromium.org> Reviewed-by: Egor Pasko <pasko@chromium.org> Reviewed-by: agrieve <agrieve@chromium.org> Cr-Commit-Position: refs/heads/master@{#603524} 
- 
Boris Sazonov authoredThis CL changes help context string introduced by https://crrev.com/c/1299133 for "?" button in "Sync and Google services" settings page. Bug: 885333 Change-Id: Ibe24f7f85ad4080dea0eae55efeb7810cb026430 Reviewed-on: https://chromium-review.googlesource.com/c/1304555Reviewed-by: Theresa <twellington@chromium.org> Commit-Queue: Boris Sazonov <bsazonov@chromium.org> Cr-Commit-Position: refs/heads/master@{#603523} 
- 
Scott Violet authoredThere are two paths to unminimize/restoring a window: . Implicitly by calling Show()/Activate(). In this scenario the expectation is the Widget returns to it's pre-minimized state. DesktopWindowTreeHostMus does *not* cache the pre-minimized state, only the server knows it. While waiting for the server to change the show state it's important IsMinimized() returns false. To deal with this an observer is added to wait for the show state to change. . By calling Restore(). Restore sets the state to normal, circumventing the pre-minimized state in the server. This mirrors what NativeWidgetAura does. BUG=883523 TEST=covered by tests Change-Id: I79a89a419032f125d905a98a01fe906c7ca82efd Reviewed-on: https://chromium-review.googlesource.com/c/1299913 Commit-Queue: Scott Violet <sky@chromium.org> Reviewed-by: Michael Wasserman <msw@chromium.org> Cr-Commit-Position: refs/heads/master@{#603522} 
 
-