- 09 Mar, 2018 40 commits
-
-
https://chromium.googlesource.com/catapult.git/+log/8f3d6b77acab..b90b97cf8dec $ git log 8f3d6b77a..b90b97cf8 --date=short --no-merges --format='%ad %ae %s' 2018-03-02 perezju Reland "[Telemetry] Store Android backend settings as namedtuples" 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. TBR=sullivan@chromium.org Change-Id: I7ac217979b7cc88b0247c6fdadab8aa63e416ba7 Reviewed-on: https://chromium-review.googlesource.com/957102Reviewed-by:
catapult-chromium-autoroll <catapult-chromium-autoroll@skia-buildbots.google.com.iam.gserviceaccount.com> Commit-Queue: catapult-chromium-autoroll <catapult-chromium-autoroll@skia-buildbots.google.com.iam.gserviceaccount.com> Cr-Commit-Position: refs/heads/master@{#542133}
-
Josh Karlin authored
Bug: 817365 Change-Id: I106d6b49c78f66b47a37d7f429b62b506ea5a225 Reviewed-on: https://chromium-review.googlesource.com/951823 Commit-Queue: Josh Karlin <jkarlin@chromium.org> Reviewed-by:
Charlie Harrison <csharrison@chromium.org> Cr-Commit-Position: refs/heads/master@{#542132}
-
Aaron Leventhal authored
When switching to a tab that has the Omnibox or other UI focused, screen readers do not read the title of the document being navigated to. For the case where a tab is switched, and the UI is about to be focused, fire an extra focus event on the widget's root view so that the document title is read before the UI focus. TBR: tsepez@chromium.org Bug: 791757 Cq-Include-Trybots: master.tryserver.chromium.linux:closure_compilation Change-Id: I19a91eaac1c24dfd04ab3458a41aaa0c512a8739 Reviewed-on: https://chromium-review.googlesource.com/938778 Commit-Queue: Dominic Mazzoni <dmazzoni@chromium.org> Reviewed-by:
Scott Violet <sky@chromium.org> Reviewed-by:
Dominic Mazzoni <dmazzoni@chromium.org> Cr-Commit-Position: refs/heads/master@{#542131}
-
Fady Samuel authored
In device emulation mode, the top level renderer devtools lies to Blink about the size and scale factor by constructing an emulated ScreenInfo object. Prior to this patch, we propagated that ScreenInfo down to OOPIFs and BrowserPlugins. This caused double scaling of the compositor viewport size in OOPIFs because the top level renderer had already scaled down the surface embedded. This CL instead plumbs down the original ScreenInfo which fixes the device emulation problem. Bug: 819903, 672962 Change-Id: Ie32c2798edcbe8738c463edf86d45c86f33d0d5b Reviewed-on: https://chromium-review.googlesource.com/956554 Commit-Queue: Fady Samuel <fsamuel@chromium.org> Reviewed-by:
Dmitry Gozman <dgozman@chromium.org> Reviewed-by:
Lucas Gadani <lfg@chromium.org> Cr-Commit-Position: refs/heads/master@{#542130}
-
Amirhossein Simjour authored
DropdownPopupWindow uses AnchoredPopupWindow instead of using ListPopupWindow. This should potentially fix the positioning problem of DropdownPopupWindow. BUG=809758 Change-Id: I246aca46f28b0c71cccadee3d7d12208f6f58268 Reviewed-on: https://chromium-review.googlesource.com/923426 Commit-Queue: Amirhossein Simjour <asimjour@chromium.org> Reviewed-by:
Theresa <twellington@chromium.org> Cr-Commit-Position: refs/heads/master@{#542129}
-
Tibor Goldschwendt authored
Bug: 811987 Cq-Include-Trybots: master.tryserver.chromium.android:android_optional_gpu_tests_rel;master.tryserver.chromium.linux:linux_optional_gpu_tests_rel;master.tryserver.chromium.linux:linux_vr;master.tryserver.chromium.mac:mac_optional_gpu_tests_rel;master.tryserver.chromium.win:win_optional_gpu_tests_rel Change-Id: I5c33b4210727b62357e0706c7b19800b98bdebc4 Reviewed-on: https://chromium-review.googlesource.com/956251Reviewed-by:
Joshua Pawlicki <waffles@chromium.org> Reviewed-by:
Christopher Grant <cjgrant@chromium.org> Commit-Queue: Tibor Goldschwendt <tiborg@chromium.org> Cr-Commit-Position: refs/heads/master@{#542128}
-
Gauthier Ambard authored
This CL changes the creation of the view used to simulate the background of the NTP during stack view animation so it is unvisible at the creation. Bug: 819795 Cq-Include-Trybots: master.tryserver.chromium.mac:ios-simulator-cronet;master.tryserver.chromium.mac:ios-simulator-full-configs Change-Id: Ia59dea3ff4902b21ad1fa097d0bb3c7210d41b93 Reviewed-on: https://chromium-review.googlesource.com/956071Reviewed-by:
Jérôme Lebel <jlebel@chromium.org> Commit-Queue: Gauthier Ambard <gambard@chromium.org> Cr-Commit-Position: refs/heads/master@{#542127}
-
Simeon Anfinrud authored
When using a CastWebContentsFragment, the host activity will determine the audio focus state. Only request audio focus in CastWebContentsActivity so that if the activity on Android TV launches while another app is playing media, the old app will stop playing when CastWebContentsActivity becomes visible. Bug: Internal b/73952032 Test: cast_shell_junit_tests Change-Id: I109ef227706c6e2ed9273b70a57029ddd1d6b248 Reviewed-on: https://chromium-review.googlesource.com/956272Reviewed-by:
Luke Halliwell <halliwell@chromium.org> Commit-Queue: Simeon Anfinrud <sanfin@chromium.org> Cr-Commit-Position: refs/heads/master@{#542126}
-
Geoff Lang authored
BUG=811828 BUG=811794 BUG=811791 BUG=811823 BUG=811871 BUG=811830 BUG=812108 BUG=811867 BUG=811817 BUG=811800 BUG=811826 BUG=811833 BUG=811805 BUG=811824 BUG=811793 BUG=811868 BUG=811783 BUG=811818 BUG=811788 BUG=811784 Change-Id: I191e5d2c0057579b0f9b75dfaf67c01db7d82f61 Reviewed-on: https://chromium-review.googlesource.com/956305 Commit-Queue: Geoff Lang <geofflang@chromium.org> Reviewed-by:
Kenneth Russell <kbr@chromium.org> Reviewed-by:
John Budorick <jbudorick@chromium.org> Cr-Commit-Position: refs/heads/master@{#542125}
-
James Cook authored
Rename the class to TouchExplorationManager, since it's already ash-only and there are no other such managers. Remove the _chromeos suffix from the file, since ash is chromeos only these days. This code probably dates back to when ash ran on Windows. Some of this class could be collapsed with TouchExplorationController. Bug: none Test: ash_unittests Change-Id: Id6e81d60e32ff27bccaf4f3d231e463c4c4eb747 Reviewed-on: https://chromium-review.googlesource.com/956536 Commit-Queue: James Cook <jamescook@chromium.org> Reviewed-by:
David Tseng <dtseng@chromium.org> Cr-Commit-Position: refs/heads/master@{#542124}
-
Gauthier Ambard authored
This CL fixes the navigation history popup for the bottom toolbar, allowing it to be shown when the user long press the arrows. Bug: 818681 Cq-Include-Trybots: master.tryserver.chromium.mac:ios-simulator-cronet;master.tryserver.chromium.mac:ios-simulator-full-configs Change-Id: Ib405bd5ddccefca940bcd3021ee320dc62bc4f09 Reviewed-on: https://chromium-review.googlesource.com/950883 Commit-Queue: Gauthier Ambard <gambard@chromium.org> Reviewed-by:
Mark Cogan <marq@chromium.org> Cr-Commit-Position: refs/heads/master@{#542123}
-
Gauthier Ambard authored
During side swipe, the snapshot of the toolbar cannot have the blur effect. This CL adds a solid grey background on the toolbar during the side swipe. Bug: 807988 Cq-Include-Trybots: master.tryserver.chromium.mac:ios-simulator-cronet;master.tryserver.chromium.mac:ios-simulator-full-configs Change-Id: I6301068a453e70cc1da005858657dd0fdd2f8ea1 Reviewed-on: https://chromium-review.googlesource.com/955422 Commit-Queue: Gauthier Ambard <gambard@chromium.org> Reviewed-by:
Justin Cohen <justincohen@chromium.org> Cr-Commit-Position: refs/heads/master@{#542122}
-
Ken Rockot authored
child.Connect -> invitation.Send TBR=jcivelli@chromium.org Bug: None Change-Id: I4ec473430405e69af6e54dac058d9942a54be0c6 Reviewed-on: https://chromium-review.googlesource.com/956694Reviewed-by:
Ken Rockot <rockot@chromium.org> Cr-Commit-Position: refs/heads/master@{#542121}
-
Ehsan Karamad authored
The OOPIF part was blocked on https://crbug.com/810921 which is now fixed. Bug: 810921 Change-Id: Ieafe7d82d594cad99e990e8964517e7dbfcfc090 Reviewed-on: https://chromium-review.googlesource.com/954335Reviewed-by:
Alex Moshchuk <alexmos@chromium.org> Reviewed-by:
David Bokan <bokan@chromium.org> Commit-Queue: Ehsan Karamad <ekaramad@chromium.org> Cr-Commit-Position: refs/heads/master@{#542120}
-
tzik authored
This CL changes the ownership of base::Clock and base::TickClock from injectee-owned to injecter-owned. Before this CL, these instances are owned by the owner of the injectee or one of the injectees themselves. That makes the ownership handling complex. After this CL, the injectee of clocks never own the clock. Instead, injecters owns a clock for testing, and a shared clock is used on the production code. Bug: 789079 Cq-Include-Trybots: master.tryserver.chromium.android:android_optional_gpu_tests_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 Change-Id: Ic6e68a02228f1678e22569c38257e599d4d4e432 Reviewed-on: https://chromium-review.googlesource.com/937009Reviewed-by:
Xiaohan Wang <xhwang@chromium.org> Commit-Queue: Taiju Tsuiki <tzik@chromium.org> Cr-Commit-Position: refs/heads/master@{#542119}
-
Gary Kacmarcik authored
Remove the "+ Add site exception" button for consistency with other Content Settings based on origin. The current UX for this action assumes that the site exception is host-pattern based, so the exception is not stored/cleared correctly. Bug: 818615, 677565 Change-Id: Id47b07ea704c4957f3bb73c2c99629021bc65bd3 Reviewed-on: https://chromium-review.googlesource.com/952574Reviewed-by:
Raymes Khoury <raymes@chromium.org> Reviewed-by:
Finnur Thorarinsson <finnur@chromium.org> Commit-Queue: Gary Kacmarcik <garykac@chromium.org> Cr-Commit-Position: refs/heads/master@{#542118}
-
Hajime Hoshi authored
This is part of efforts to replace base::ThreadTaskRunnerHandle::Get() and SequencedTaskRunnerHandle::Get() with other appropriate task runners in the renderer. This avoids the ThreadTaskRunnerHandle::Get() usage. We are trying to ban ThreadTaskRunnerHandle::Get() in content/renderer because it usually is not the right task runner to use, as a per-frame runner should be used instead. In this case, it doesn't really matter since frames are not involved, but we still want to remove the calls for the bigger benefit of banning it in content/renderer. Bug: 786332 Change-Id: Ia492ce20737e04f4fc0f031ecfb7a1994756588a Reviewed-on: https://chromium-review.googlesource.com/939210Reviewed-by:
Kinuko Yasuda <kinuko@chromium.org> Reviewed-by:
Alexander Timin <altimin@chromium.org> Commit-Queue: Hajime Hoshi <hajimehoshi@chromium.org> Cr-Commit-Position: refs/heads/master@{#542117}
-
Gauthier Ambard authored
This CL changes the method name of NewTabPageControllerDelegate allowing the NTP to change the color of the toolbar. The previous name of the method was confusing. Also, this method inverse the value of the argument (i.e. to display a NTP-colored background on the toolbar, the value 1 needs to be passed, instead of 0 before). Bug: none Cq-Include-Trybots: master.tryserver.chromium.mac:ios-simulator-cronet;master.tryserver.chromium.mac:ios-simulator-full-configs Change-Id: If19d37a8cb1a2f6761e7902b2a828955b73a7e41 Reviewed-on: https://chromium-review.googlesource.com/957093Reviewed-by:
-
Gayane Petrosyan authored
Change-Id: I662bce08782a154da3971f6046b40570c103fa35 Reviewed-on: https://chromium-review.googlesource.com/886825 Commit-Queue: Gayane Petrosyan <gayane@chromium.org> Reviewed-by:
Taiju Tsuiki <tzik@chromium.org> Cr-Commit-Position: refs/heads/master@{#542115}
-
Tommy C. Li authored
These features have been on 100% Stable for 2 weeks now, so I'd consider the launch to have "stuck". Following standard launch procedure, now we remove the experimental code entirely from trunk. Bug: 755643 Cq-Include-Trybots: master.tryserver.chromium.mac:ios-simulator-cronet;master.tryserver.chromium.mac:ios-simulator-full-configs Change-Id: Ia7b506a3ffca8823dc26b54e2dbd9e79082ab1c8 Reviewed-on: https://chromium-review.googlesource.com/956309 Commit-Queue: Justin Donnelly <jdonnelly@chromium.org> Reviewed-by:
Justin Donnelly <jdonnelly@chromium.org> Cr-Commit-Position: refs/heads/master@{#542114}
-
Fredrik Söderquist authored
Supported for animated PNG (APNG) was added via crbug.com/437662 (commit 7d2b8c45.) The "image/apng" MIME type is included in the "Accept" header for image requests. Add "image/apng" to kSupportedImageTypes to allow things like the <picture> element to work (select an appropriate image based on 'type'.) Add "image/apng" to kPrimaryMappings to allow the MIME type to resolve to a reasonable file extension. Bug: 730588 Change-Id: I070e8b16dc8312552b11dc32e776493b3a5be669 Reviewed-on: https://chromium-review.googlesource.com/957092 Commit-Queue: Fredrik Söderquist <fs@opera.com> Reviewed-by:
Asanka Herath <asanka@chromium.org> Cr-Commit-Position: refs/heads/master@{#542113}
-
Juan Antonio Navarro Perez authored
The device_utils.ListProcesses method already does most of the work required to implement GetPidsToTrack. Use that rather than hand-rolling yet another parse-grepping through the output of `ps`. Bug: catapult:#3215 Bug: catapult:#4103 Change-Id: I3825761ce2efc6cbdd940f41505789b61506bd8b Reviewed-on: https://chromium-review.googlesource.com/955628Reviewed-by:
-
John Budorick authored
Bug: 790124, 790129, 790130, 790135 Change-Id: I1691c96df7e9f681a291200c76e9a883d0784214 Reviewed-on: https://chromium-review.googlesource.com/956806Reviewed-by:
Shenghua Zhang <shenghuazhang@chromium.org> Commit-Queue: John Budorick <jbudorick@chromium.org> Cr-Commit-Position: refs/heads/master@{#542111}
-
Corentin Wallez authored
BUG=820190 Change-Id: I8a84d1bc2d8635b7d76b438c938eb1c70874e8a2 Reviewed-on: https://chromium-review.googlesource.com/956923Reviewed-by:
Andrii Shyshkalov <tandrii@chromium.org> Reviewed-by:
-
Christopher Grant authored
This change wires up button hover and click sounds, and adds them to the asset structure. BUG=812026 R=tiborg Cq-Include-Trybots: master.tryserver.chromium.android:android_optional_gpu_tests_rel;master.tryserver.chromium.linux:linux_optional_gpu_tests_rel;master.tryserver.chromium.linux:linux_vr;master.tryserver.chromium.mac:mac_optional_gpu_tests_rel;master.tryserver.chromium.win:win_optional_gpu_tests_rel Change-Id: Iccf113e74663201c62aa220c7e134990369fd755 Reviewed-on: https://chromium-review.googlesource.com/953962 Commit-Queue: Christopher Grant <cjgrant@chromium.org> Reviewed-by:
Tibor Goldschwendt <tiborg@chromium.org> Reviewed-by:
Ian Vollick <vollick@chromium.org> Cr-Commit-Position: refs/heads/master@{#542109}
-
Daniel Bratell authored
Generating GUID objects in a binary requires careful juggling of includes and the define INITGUID (or the header <Initguid.h>). Jumbo builds are not that careful so the simplest/best solution is to remove the file doing INITGUID from jumbo compilation. This manifested itself in jumbo fyi builders building catalog_viewer_service.exe. TBR=dtapuska@chromium.org Bug: 657623 Change-Id: I1457727a0b3a68f82726a2e73c1ae3dedb06f12d Reviewed-on: https://chromium-review.googlesource.com/955690Reviewed-by:
Daniel Bratell <bratell@opera.com> Commit-Queue: Daniel Bratell <bratell@opera.com> Cr-Commit-Position: refs/heads/master@{#542108}
-
Miguel Casas authored
This CL splits VaapiDrmPicture into a common part named VaapiPictureNativePixmap, and two subclasses, namely: - VaapiPictureNativePixmapOzone, containing the current #if defined(USE_OZONE) sandwiched parts, and - VaapiPixtureNativePixmapEgl with the other parts. So the class tree should look like: +-----------------+ | VaapiPicture | +-----------------+ ^ ^ | +----+ +--------------+ +------------------+ | VaapiPicture | | VaapiPictureTFP | | NativePixmap | | (existing on ToT)| +--------------+ +------------------+ ^ +---------------+ | | +-------+ +-------+ | VPNP | | VPNP | | Ozone | | Egl | +-------+ +-------+ The point here is to reduce the amount of if-defs to one single point, the VaapiPictureFactory. BUILD.gn is changed to reflect that VaapiDrmPicture is egl-specific (see bug, |use_egl| definition in [1]), while VaapiDrmPictureOzone is |use_ozone| [2] specific. Note that |use_egl| can coexist with |use_x11| [3]. This CL just moves code around, no new code introduced in either vaapi_picture*.cc. [1] https://cs.chromium.org/chromium/src/ui/gl/features.gni?q=ui/gl/features.gni&sq=package:chromium&dr&l=16 [2] https://cs.chromium.org/chromium/src/build/config/ui.gni?q=build/config/ui.gni&sq=package:chromium&dr&l=26 [3] https://cs.chromium.org/chromium/src/build/config/ui.gni?type=cs&sq=package:chromium&l=47 Bug: 785201 Cq-Include-Trybots: master.tryserver.chromium.android:android_optional_gpu_tests_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 Change-Id: I67f6f4d994322ef7f23a8ed255405963a4be0152 Reviewed-on: https://chromium-review.googlesource.com/951928Reviewed-by:
Julien Isorce <julien.isorce@chromium.org> Commit-Queue: Miguel Casas <mcasas@chromium.org> Cr-Commit-Position: refs/heads/master@{#542107}
-
Benoit Lize authored
This CL adds two types of tests: - An End-to-End (Java + native) test that a cookie can be set. - Third party cookie blocking tests, which is not easily doable from native. Bug: 816837 Change-Id: I713b9c00689edae9944bf10bb92bb805ffa60746 Reviewed-on: https://chromium-review.googlesource.com/955582Reviewed-by:
Bernhard Bauer <bauerb@chromium.org> Commit-Queue: Benoit L <lizeb@chromium.org> Cr-Commit-Position: refs/heads/master@{#542106}
-
https://chromium.googlesource.com/angle/angle.git/+log/e8a93c6ed6fe..33140409810f $ git log e8a93c6ed..331404098 --date=short --no-merges --format='%ad %ae %s' 2018-03-08 lucferron Vulkan: Fix readPixels to support sub image reads correctly Created with: roll-dep src/third_party/angle The AutoRoll server is located here: https://angle-chromium-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;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=ynovikov@chromium.org Change-Id: Ibb58aa612789e37fdace261ee39ef73b8bbec6c9 Reviewed-on: https://chromium-review.googlesource.com/956279Reviewed-by:
angle-chromium-autoroll <angle-chromium-autoroll@skia-buildbots.google.com.iam.gserviceaccount.com> Commit-Queue: angle-chromium-autoroll <angle-chromium-autoroll@skia-buildbots.google.com.iam.gserviceaccount.com> Cr-Commit-Position: refs/heads/master@{#542105}
-
https://skia.googlesource.com/skia.git/+log/66abdf665896..b537879c7214 $ git log 66abdf665..b537879c7 --date=short --no-merges --format='%ad %ae %s' 2018-03-08 stephana Remove SKQP bots from master 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=fmalita@chromium.org Change-Id: I2f8c7375f630e747ae89b2bf56b34495558785fd Reviewed-on: https://chromium-review.googlesource.com/956278Reviewed-by:
skia-chromium-autoroll <skia-chromium-autoroll@skia-buildbots.google.com.iam.gserviceaccount.com> Commit-Queue: skia-chromium-autoroll <skia-chromium-autoroll@skia-buildbots.google.com.iam.gserviceaccount.com> Cr-Commit-Position: refs/heads/master@{#542104}
-
tzik authored
This CL changes the ownership of base::Clock and base::TickClock from injectee-owned to injecter-owned. Before this CL, these instances are owned by the owner of the injectee or one of the injectees themselves. That makes the ownership handling complex. After this CL, the injectee of clocks never own the clock. Instead, injecters owns a clock for testing, and a shared clock is used on the production code. Tbr: justincohen@chromium.org Bug: 789079 Cq-Include-Trybots: master.tryserver.chromium.mac:ios-simulator-cronet;master.tryserver.chromium.mac:ios-simulator-full-configs Change-Id: If1fec026e9d801f76ab46f4f4f28b31ce3eefe34 Reviewed-on: https://chromium-review.googlesource.com/956168 Commit-Queue: Taiju Tsuiki <tzik@chromium.org> Reviewed-by:
Marc Treib <treib@chromium.org> Cr-Commit-Position: refs/heads/master@{#542103}
-
Noel Gordon authored
SharedBuffer::Data() is deprecated, and was being used to consolidate the buffer into one contiguous piece of memory. This is now redundant since the ShardBuffer is created in ReadFile() by adopting the Vector of image data (which is a contiguous piece of memory under the hood). Remove the SharedBuffer::Data() call, move the related error handling for ReadFile() back into that routine. Bug: 601198 Change-Id: Ic0eab97ead78a967180d1ee6bdcb708e3cc3a4e5 Reviewed-on: https://chromium-review.googlesource.com/956644Reviewed-by:
Leon Scroggins <scroggo@chromium.org> Commit-Queue: Noel Gordon <noel@chromium.org> Cr-Commit-Position: refs/heads/master@{#542102}
-
v8-autoroll authored
Summary of changes available at: https://chromium.googlesource.com/v8/v8/+log/dc2d3bb9..0cf325d6 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: I043d518461c291a17ae29f72b1660f7c36fe30db Reviewed-on: https://chromium-review.googlesource.com/956570Reviewed-by:
v8 autoroll <v8-autoroll@chromium.org> Commit-Queue: Michael Achenbach <machenbach@chromium.org> Cr-Commit-Position: refs/heads/master@{#542101}
-
Finnur Thorarinsson authored
Bug: 734095 Change-Id: I87213656fd5e692650d75e8ca7929931bcb54f2d Reviewed-on: https://chromium-review.googlesource.com/955482 Commit-Queue: Finnur Thorarinsson <finnur@chromium.org> Reviewed-by:
Peter Beverloo <peter@chromium.org> Reviewed-by:
Tommy Martino <tmartino@chromium.org> Reviewed-by:
Xi Cheng <chengx@chromium.org> Cr-Commit-Position: refs/heads/master@{#542100}
-
Nico Weber authored
See bug, comment 8. Bug: 813553 Change-Id: Ie584c5f30ea23e0dfd70093600dd6d9f5d6c4dc7 Reviewed-on: https://chromium-review.googlesource.com/957122Reviewed-by:
Hans Wennborg <hans@chromium.org> Commit-Queue: Nico Weber <thakis@chromium.org> Cr-Commit-Position: refs/heads/master@{#542099}
-
Nico Weber authored
Don't run ChromePluginTest.InstalledPlugins in is_official_build=true is_chrome_branded=false builds. ToTWinLibcxx64 sets is_official_build=true but doesn't have src-internal checked out. The test currently fails there because it needs files from src-internal. The fix is to run the test if is_chrome_branded is set (which requires a src-internal checkout), instead of when is_official_build is set. Bug: 801780 Change-Id: Icbfaf8c978ac7618d67b1d7e0cd1f48be45a8232 Reviewed-on: https://chromium-review.googlesource.com/957123Reviewed-by:
-
Christos Froussios authored
Focus should return to the item which caused the export modal dialog to be opened, once the dialog is done. Bug: 818619 Cq-Include-Trybots: master.tryserver.chromium.linux:closure_compilation Change-Id: I0e67519aef1821e751e7dd90bab42b4321408442 Reviewed-on: https://chromium-review.googlesource.com/950774Reviewed-by:
Hector Carmona <hcarmona@chromium.org> Commit-Queue: Christos Froussios <cfroussios@chromium.org> Cr-Commit-Position: refs/heads/master@{#542097}
-
Kent Tamura authored
Bug: 770603 Change-Id: I6a161d1508353fe0fa2c17fb1580ce5385555353 Reviewed-on: https://chromium-review.googlesource.com/956655 Commit-Queue: Kent Tamura <tkent@chromium.org> Reviewed-by:
Hitoshi Yoshida <peria@chromium.org> Cr-Commit-Position: refs/heads/master@{#542096}
-
Egor Pasko authored
This will make it easy to make a metric like request_start_time. I was going to make it based on netlog, but that appears more difficult (many events in netlog, etc). Also navigation events are generally useful for startup, and not very frequent. Bug: 760498 Change-Id: I7b9e9bb7efa0d4f2db729dafbb2b56e8ca1ec0ed Reviewed-on: https://chromium-review.googlesource.com/955632Reviewed-by:
Juan Antonio Navarro Pérez <perezju@chromium.org> Commit-Queue: Egor Pasko <pasko@chromium.org> Cr-Commit-Position: refs/heads/master@{#542095}
-
Elly Fong-Jones authored
This change: 1) Factors out the scoped feature list pattern used in TestWithBrowserView to a separate helper class; 2) Changes TestWithBrowserView to use the new helper classe; 3) Makes BrowserViewTests use the helper class as well. Bug: 817408 Change-Id: I99c4ac591676d61936a75b38b0de22dbf27137da Reviewed-on: https://chromium-review.googlesource.com/955778Reviewed-by:
-
