- 09 Mar, 2018 40 commits
-
-
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:
Theresa <twellington@chromium.org> Reviewed-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 <jbudorick@chromium.org> Commit-Queue: Juan Antonio Navarro Pérez <perezju@chromium.org> Cr-Commit-Position: refs/heads/master@{#542112}
-
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:
Scott Violet <sky@chromium.org> Commit-Queue: Elly Fong-Jones <ellyjones@chromium.org> Cr-Commit-Position: refs/heads/master@{#542094}
-
Yang Guo authored
Inline event handlers are compiled with v8::ScriptCompiler::CompileFunctionInContext. Previously, it compiles a wrapper script containing the event handler and executes this wrapper script to object the event handler function. This script execution is observable in debug stepping. An upcoming V8 change skips this wrapper script and creates the event handler function directly. This changes the debug stepping behavior. R=kozyatinskiy@chromium.org, machenbach@chromium.org Bug: v8:7172 Change-Id: I64ee3487f9ca893da305aa41dfc5d2e1e3da4ecc Reviewed-on: https://chromium-review.googlesource.com/955594Reviewed-by:
Michael Achenbach <machenbach@chromium.org> Commit-Queue: Yang Guo <yangguo@chromium.org> Cr-Commit-Position: refs/heads/master@{#542093}
-
Daniel Bratell authored
PostMessage is a macro in Windows and depending on whether Windows headers have been included or not, PostMessage will either be treated as PostMessage or PostMessageW. We use a special header windows_types.h to ensure that the macro is always defined so that there will be no compilation or linking issues missing either PostMessage or PostMessageW objects. Bug: 772713,813749 TBR=leon.han@intel.com Change-Id: I0012774f347ef2e257bc87b4492f58c7be464f78 Reviewed-on: https://chromium-review.googlesource.com/957028Reviewed-by:
-
Nico Weber authored
C++ disallows implicit conversion of function pointer types (such as returned by GetProcAddress()) to void*. MSVC allows it, and clang-cl allows it with a warning. This fixes said warning by adding an explicit cast. It seems conceptually nicer to have GetFunctionPointerFromNativeLibrary() return a fuction pointer type, but for now let's just suppress the warning. Bug: 550065 Change-Id: Ifc633ca5691c2c297db6c646c655b9a85815170a Reviewed-on: https://chromium-review.googlesource.com/955906 Commit-Queue: Daniel Cheng <dcheng@chromium.org> Commit-Queue: Nico Weber <thakis@chromium.org> Reviewed-by:
Daniel Cheng <dcheng@chromium.org> Cr-Commit-Position: refs/heads/master@{#542091}
-
Balazs Engedy authored
This is to avoid duplicating the code in each unit test to implement a call back receiver for each combination of argument types. Bug: 785955 Change-Id: Ic4cd0502fafc666682c29a657e25cab91091db44 Reviewed-on: https://chromium-review.googlesource.com/955566 Commit-Queue: Balazs Engedy <engedy@chromium.org> Reviewed-by:
Jan Wilken Dörrie <jdoerrie@chromium.org> Reviewed-by:
-
Michael Warres authored
Remove reference to headers and crypto stream from the public API of QuicWriteBlockedList in preparation for removing these from the implementation. Merge internal change: 187900607 https://chromium-review.googlesource.com/c/chromium/src/+/956023 Move logic for registering and unregistering QUIC streams with the write blocked list from QuicSpdyStream to QuicStream. Protected by FLAGS_quic_reloadable_flag_quic_register_streams_early Merge internal change: 187702688 https://chromium-review.googlesource.com/c/chromium/src/+/956022 Refactor the code submitted in cl/187642899. Protected by the same flag, FLAGS_quic_reloadable_flag_quic_fix_write_out_of_order_queued_packet_crash. Merge internal change: 187666679 https://chromium-review.googlesource.com/c/chromium/src/+/955724 In QuicConnection, Fixed a use-after-free when writing an out-of-order queued packet. Protected by FLAGS_quic_reloadable_flag_quic_fix_write_out_of_order_queued_packet_crash. Merge internal change: 187642899 https://chromium-review.googlesource.com/c/chromium/src/+/955779 In QUIC, enable fast path to QuicStreamSendBuffer::OnStreamDataAcked. Protected by FLAGS_quic_reloadable_flag_quic_fast_path_on_stream_data_acked. Merge internal change: 187609757 https://chromium-review.googlesource.com/c/chromium/src/+/955902 Make QuicConnection::MaxPacingRate() method virtual. Merge internal change: 187556600 https://chromium-review.googlesource.com/c/chromium/src/+/955777 Make QuicConnection::set_per_packet_options() and QuicClientEpollNetworkHelper::CleanUpUDPSocket() methods public. Merge internal change: 187536653 https://chromium-review.googlesource.com/c/chromium/src/+/955723 Add IETF Path Challenge/Response framer/deframer Merge internal change: 187484592 https://chromium-review.googlesource.com/c/chromium/src/+/954783 Change some vector::push_back into vector::emplace_back to save a temporary object. Not flag protected. Merge internal change: 187482582 https://chromium-review.googlesource.com/c/chromium/src/+/954842 Add a method to the QUIC crypto framer to allow processing truncated messages, which should only be used for debugging. Merge internal change: 187430236 https://chromium-review.googlesource.com/c/chromium/src/+/954724 Pass full QUIC version into the connection debug visitor. Merge internal change: 187423336 https://chromium-review.googlesource.com/c/chromium/src/+/954723 Remove an always-true if condition in QuicConnection::WritePacket. Not flag protected. Merge internal change: 187406845 https://chromium-review.googlesource.com/c/chromium/src/+/954722 Add IETF-Format QUIC Padding Frame support. Merge internal change: 187378942 https://chromium-review.googlesource.com/c/chromium/src/+/954123 Remove redundant get() call on smart pointer. Merge internal change: 187276945 https://chromium-review.googlesource.com/c/chromium/src/+/953306 Deprecate FLAGS_quic_reloadable_flag_quic_inspect_chlo_tags Merge internal change: 187172163 https://chromium-review.googlesource.com/c/chromium/src/+/953802 Add QuicSentPacketManager.initial_congestion_window() method. Merge internal change: 187108946 https://chromium-review.googlesource.com/c/chromium/src/+/953384 In QuicDispatcher, make client_address available in stateless rejector's callback(i.e. QuicDispatcher::OnStatelessRejectorProcessDone). Merge internal change: 187095181 https://chromium-review.googlesource.com/c/chromium/src/+/952487 Move various defintions of kDefaultPriority in QUIC code to QuicStream. Merge internal change: 187082280 https://chromium-review.googlesource.com/c/chromium/src/+/951760 Move priority handling from QuicSpdyStream to QuicStream and from QuicSpdySession to QuicSession. Merge internal change: 187047085 https://chromium-review.googlesource.com/c/chromium/src/+/952485 Fix stale comment. Merge internal change: 187027369 https://chromium-review.googlesource.com/c/chromium/src/+/951759 R=rch@chromium.org Change-Id: I3846cf618363b6e93aaa5aae1fde46fd4300f675 Reviewed-on: https://chromium-review.googlesource.com/956843Reviewed-by:
Ryan Hamilton <rch@chromium.org> Commit-Queue: Michael Warres <mpw@chromium.org> Cr-Commit-Position: refs/heads/master@{#542089}
-
https://chromium.googlesource.com/catapult.git/+log/78d89b42c293..8f3d6b77acab $ git log 78d89b42c..8f3d6b77a --date=short --no-merges --format='%ad %ae %s' 2018-03-07 perezju [Battor] Ignore battor/bin/{linux,mac,win} 2018-03-08 perezju [Telemetry] Add Android_Go disable condition 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: I1a72fce14063a42886f6c85fde6a9ac0f2eede66 Reviewed-on: https://chromium-review.googlesource.com/956112 Commit-Queue: catapult-chromium-autoroll <catapult-chromium-autoroll@skia-buildbots.google.com.iam.gserviceaccount.com> Reviewed-by:
catapult-chromium-autoroll <catapult-chromium-autoroll@skia-buildbots.google.com.iam.gserviceaccount.com> Cr-Commit-Position: refs/heads/master@{#542088}
-
Steve Kobes authored
Hiding browser controls can make the layout viewport larger than the document's layout overflow. Bottom-anchored fixed-position elements were incorrectly clipped with RLS, because the scrolling contents layer was sized to match the overflow rect. Bug: 817714 Cq-Include-Trybots: master.tryserver.blink:linux_trusty_blink_rel;master.tryserver.chromium.linux:linux_layout_tests_slimming_paint_v2 Change-Id: I89c3488370e1bce02a7ed523e930ab9d634f60ac Reviewed-on: https://chromium-review.googlesource.com/954016Reviewed-by:
-
Florent Castelli authored
Add the getParameters() function forwarding all the values that are implemented in WebRTC to the Javascript layer. Other properties that exist will be loaded for future use, but not passed to the Javascript layer. Currently behind RuntimeEnabled=RTCRtpSenderParameters. Bug: 803494 Change-Id: Icd1c1ea58fd1ad3d50a46e97db3026837381c7b6 Reviewed-on: https://chromium-review.googlesource.com/919082Reviewed-by:
Kentaro Hara <haraken@chromium.org> Reviewed-by:
Henrik Boström <hbos@chromium.org> Reviewed-by:
Guido Urdaneta <guidou@chromium.org> Commit-Queue: Florent Castelli <orphis@chromium.org> Cr-Commit-Position: refs/heads/master@{#542086}
-
Friedrich Horschig authored
These are the first steps to port the http-state cookie tests from https://github.com/abarth/http-state/tree/master/tests to the external/wpt/ directory. Accessing <server>/cookies/http-state/all-tests.html will run all tests in cookies/http-state/resources/test-files/. This ensures that tests stay isolated, ignore set cookies and clean up cookies after themselves. By accessing the server directly, single tests can be debugged. e.g. to run test file 0005 in isolation, access this URL: <server>/cookies/http-state/resources/cookie-setter.py?debug=0005 Change-Id: I1fceccf0bc5d080d024998451abd027142d6a16a Reviewed-on: https://chromium-review.googlesource.com/807988 Commit-Queue: Friedrich Horschig <fhorschig@chromium.org> Reviewed-by:
Mike West <mkwst@chromium.org> Cr-Commit-Position: refs/heads/master@{#542085}
-
Vasilii Sukhanov authored
The code is executed once per profile. As it was introduced one year ago, it's reasonable to assume that most users don't need it anymore. It's a manual revert of https://codereview.chromium.org/2714543006/ Bug: 687968 Change-Id: Ic9ef109c27ccccadc8b3fd75d156dec818cfc83c Reviewed-on: https://chromium-review.googlesource.com/955568 Commit-Queue: Vasilii Sukhanov <vasilii@chromium.org> Reviewed-by:
-
