- 30 Oct, 2017 1 commit
-
-
marcin authored
Bug: Change-Id: If2a9409b0f1db115e67cb63d7adea4634ab574e9 Reviewed-on: https://chromium-review.googlesource.com/736729 Commit-Queue: Nico Weber <thakis@chromium.org> Reviewed-by:
Thomas Anderson <thomasanderson@chromium.org> Reviewed-by:
Dirk Pranke <dpranke@chromium.org> Cr-Commit-Position: refs/heads/master@{#512425}
-
- 29 Oct, 2017 12 commits
-
-
Trent Apted authored
Under MD, the close button takes up more vertical space than the dialog title, so it dominates the title insets. Make the test work for both by removing the close buttons from the layout. Bug: 713030 Change-Id: Ibead5d002dcbe36d6fda65a3a9a8e01695e4f1a8 Reviewed-on: https://chromium-review.googlesource.com/741381 Commit-Queue: Trent Apted <tapted@chromium.org> Reviewed-by:
Bret Sepulveda <bsep@chromium.org> Cr-Commit-Position: refs/heads/master@{#512424}
-
dpapad authored
Bug: 775479 Cq-Include-Trybots: master.tryserver.chromium.linux:closure_compilation Change-Id: I94f5f9d189edcf81f2087f60e5a118824874fad4 Reviewed-on: https://chromium-review.googlesource.com/742650Reviewed-by:
Hector Carmona <hcarmona@chromium.org> Commit-Queue: Demetrios Papadopoulos <dpapad@chromium.org> Cr-Commit-Position: refs/heads/master@{#512423}
-
Matt Menke authored
In particular, switch them to using the EmbeddedTestServer from URLRequestFilter, which required them to be in the same process as the network stack. Bug: 776589 Change-Id: I4fd64fa427adedfd383dee8f86c21c4955d7f3eb Reviewed-on: https://chromium-review.googlesource.com/731523 Commit-Queue: Matt Menke <mmenke@chromium.org> Reviewed-by:
Min Qin <qinmin@chromium.org> Cr-Commit-Position: refs/heads/master@{#512422}
-
Mark Mentovai authored
fbc365fa9ed5 GCC 7.2 support (-Wnoexcept-type) 2f481590112b Make crashpad_util_test build without warnings with clang- cl on Windows b96b9f627ee7 win: Use test::ErrorMessage() in test::ChildLauncher 52d766400da1 linux: ProcessReader can own ProcessMemoryLinux without unique_ptr 9bc5989125b8 crashpad_util_test warning fixes for clang-cl, 64-bit edition 34699d378b82 Create client data structures for typed Annotations 5e9ed4cb9f69 win: Dynamically disable WoW64 tests absent explicit 32-bit build output cb3aa9c4d8f3 DumpAndCrash in extended handler test in favor of debug break c49da9caef30 win: Expect uneven symbolic link support 1dae919b7e5a #include "base/logging.h" in client/annotation.h for DCHECK() ef262d1ee3c2 #include "build/build_config.h" where needed 672c87258912 Upstream crashpad_util_test-in-Chromium changes 692488a254c4 Un-disable WinMultiprocess-based tests in Chromium 3fae8ff07c09 win: Fix -Wsign-compare warnings produced by clang Change-Id: I4b1355afce8be5571af3582f7dd0057b5018d70f Bug: crashpad:204, crashpad:205 Reviewed-on: https://chromium-review.googlesource.com/742284 Commit-Queue: Mark Mentovai <mark@chromium.org> Reviewed-by:
Scott Graham <scottmg@chromium.org> Cr-Commit-Position: refs/heads/master@{#512421}
-
Weidong Guo authored
Changes: Move caps lock key handling from accelerator controller to event_rewriter_chromeos.cc. As an accelerator key, caps lock is always consumed before it is sent to web app (e.g. CRD) or browser tab (keyboard event viewer). So we handle it in event rewriter and do not stop propagating it. BUG=775743 Change-Id: Ib3eb29cb4be023142b0d402d7c56015a7197bc14 Reviewed-on: https://chromium-review.googlesource.com/740090Reviewed-by:
Wez <wez@chromium.org> Reviewed-by:
Shu Chen <shuchen@chromium.org> Reviewed-by:
Ahmed Fakhry <afakhry@chromium.org> Reviewed-by:
Dan Erat <derat@chromium.org> Commit-Queue: Weidong Guo <weidongg@chromium.org> Cr-Commit-Position: refs/heads/master@{#512420}
-
Philip Rogers authored
This patch rebaselines 11 passing tests. These baselines now match the non-RLS path. The updated expectations were generated by copying the non-RLS expectation and adjusting for the scrolling layer differences. Change-Id: Ib2893dc3a16562306cd95ab34ca761a314edab02 Reviewed-on: https://chromium-review.googlesource.com/742455 Commit-Queue: Philip Rogers <pdr@chromium.org> Reviewed-by:
Stefan Zager <szager@chromium.org> Cr-Commit-Position: refs/heads/master@{#512419}
-
skia-deps-roller@chromium.org authored
https://skia.googlesource.com/skia.git/+log/ed6ae56e0ee8..2d8d196318a6 $ git log ed6ae56e0..2d8d19631 --date=short --no-merges --format='%ad %ae %s' 2017-10-29 update-skps Update SKP version Created with: roll-dep src/third_party/skia Documentation for the AutoRoller is here: https://skia.googlesource.com/buildbot/+/master/autoroll/README.md If the roll is causing failures, see: http://www.chromium.org/developers/tree-sheriffs/sheriff-details-chromium#TOC-Failures-due-to-DEPS-rolls 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=stani@chromium.org Change-Id: If9bb4e65bbc2dbfa81b416c4ff4365cc6ba66c09 Reviewed-on: https://chromium-review.googlesource.com/742367Reviewed-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@{#512418}
-
webrtc-autoroll@chromium.org authored
https://webrtc.googlesource.com/src.git/+log/c7b18fef19ff..6bfac0383f34 $ git log c7b18fef1..6bfac0383 --date=short --no-merges --format='%ad %ae %s' Created with: roll-dep src/third_party/webrtc Documentation for the AutoRoller is here: https://skia.googlesource.com/buildbot/+/master/autoroll/README.md If the roll is causing failures, see: http://www.chromium.org/developers/tree-sheriffs/sheriff-details-chromium#TOC-Failures-due-to-DEPS-rolls CQ_INCLUDE_TRYBOTS=master.tryserver.chromium.linux:linux_chromium_archive_rel_ng;master.tryserver.chromium.mac:mac_chromium_archive_rel_ng TBR=webrtc-chromium-sheriffs-robots@google.com Change-Id: I7148eabccc784c9ae22a35a070870cfc0206c432 Reviewed-on: https://chromium-review.googlesource.com/742213Reviewed-by:
WebRTC Roll Bot <webrtc-autoroll@chromium.org> Commit-Queue: WebRTC Roll Bot <webrtc-autoroll@chromium.org> Cr-Commit-Position: refs/heads/master@{#512417}
-
chrome-release-bot authored
TBR=amineer@chromium.org Change-Id: I054ccd393fa6bc7bbe9609f76b6b1d33ca42bd99 Reviewed-on: https://chromium-review.googlesource.com/742477Reviewed-by:
chrome-release-bot@chromium.org <chrome-release-bot@chromium.org> Cr-Commit-Position: refs/heads/master@{#512416}
-
Kinuko Yasuda authored
ServiceWorker wants to pass several params for subresource-loader setup in the renderer that are not URLLoaderFactory, because it needs to create URLLoaderFactory in the renderer. Currently we pass them via a separate Mojo call but it causes racy situation and makes tests flaky. This change introduces SubresourceLoaderParams struct (that has URLLoaderFactoryInfo as a member) and pass it along from navigation request handlers instead of URLLoaderFactoryInfo. This CL also slightly changes how WebUI subresource loader factory is passed; to make it share more code with AppCache etc. Bug: 778898 Cq-Include-Trybots: master.tryserver.chromium.linux:linux_site_isolation Change-Id: Iafe0e2f7bd8f6c500c91bde3c46dff3d4d7dd8c8 Reviewed-on: https://chromium-review.googlesource.com/739621Reviewed-by:
John Abd-El-Malek <jam@chromium.org> Commit-Queue: Kinuko Yasuda <kinuko@chromium.org> Cr-Commit-Position: refs/heads/master@{#512415}
-
Weidong Guo authored
BUG=777029 Change-Id: I691c4ea2e59f39645bb9725d6b59ee90809ae076 Reviewed-on: https://chromium-review.googlesource.com/742207Reviewed-by:
Xiyuan Xia <xiyuan@chromium.org> Commit-Queue: Weidong Guo <weidongg@chromium.org> Cr-Commit-Position: refs/heads/master@{#512414}
-
Lei Tian authored
Add return value for CustomTabBottomBarDelegate.showRemoteViews() so that client can know CCT bottom bar population fails. Bug: 729134 Change-Id: Ie0e8f3522d3ad96a2cf21f30c76579e5bb5a7979 Reviewed-on: https://chromium-review.googlesource.com/707737Reviewed-by:
Yusuf Ozuysal <yusufo@chromium.org> Commit-Queue: Lei Tian <ltian@google.com> Cr-Commit-Position: refs/heads/master@{#512413}
-
- 28 Oct, 2017 27 commits
-
-
Vaclav Brozek authored
This CL adds a reauthentication step before the user is asked to even select a file for exporting passwords to. Bug: 700003 Change-Id: Ie89529a27ce86ed74b0b3c98bf56abefb1a52899 Reviewed-on: https://chromium-review.googlesource.com/739661Reviewed-by:
Christos Froussios <cfroussios@chromium.org> Reviewed-by:
Steven Bennetts <stevenjb@chromium.org> Commit-Queue: Vaclav Brozek <vabr@chromium.org> Cr-Commit-Position: refs/heads/master@{#512412}
-
Manish Jethani authored
Previously @keyframes rules in style sheets injected by extensions were managed in ScopedStyleResolver along with other author rules. With crrev.com/c/641294 we started treating these injected style sheets as user style sheets and moved the management of such rules to StyleEngine. This patch fixes a regression caused by this change whereby StyleResolver is no longer able to find @keyframes rules from these style sheets. StyleEngine now maintains a map of animation names to their corresponding @keyframes rules found in user style sheets. If StyleResolver is unable to find a given @keyframes rule among its author rules, it looks it up in StyleEngine next. BUG=632009,776661 Change-Id: I90462a0721cc75da14ee041c1a70736e4df99fb8 Reviewed-on: https://chromium-review.googlesource.com/735263Reviewed-by:
nainar <nainar@chromium.org> Commit-Queue: nainar <nainar@chromium.org> Cr-Commit-Position: refs/heads/master@{#512411}
-
Vaclav Brozek authored
PasswordsPrivateDelegateImpl is the only client of PasswordManagerPresenter which uses the presenter's reauthentication methods. (PasswordUIViewAndroid bypasses the presenter.) To clarify the structure and exclude the Android usage more naturally than through NOTREACHED() calls, this CL moves the reauthentication methods into PasswordsPrivateDelegateImpl. To preserve testability, PasswordsPrivateDelegateImpl's unit test needs to be extended to be able to observe routed events. This caused a refactoring into a custom-fixture-using test for all the existing tests as well. Bug: 700003 Change-Id: I91b7276cc279473c6e3566c080cfec523965fb0a Reviewed-on: https://chromium-review.googlesource.com/741981Reviewed-by:
Christos Froussios <cfroussios@chromium.org> Reviewed-by:
Steven Bennetts <stevenjb@chromium.org> Commit-Queue: Vaclav Brozek <vabr@chromium.org> Cr-Commit-Position: refs/heads/master@{#512410}
-
Finnur Thorarinsson authored
Bug: 734095 Change-Id: I48d891332bdd25ebc945d64b86c601600369d2d0 Reviewed-on: https://chromium-review.googlesource.com/735862 Commit-Queue: Finnur Thorarinsson <finnur@chromium.org> Reviewed-by:
Peter Beverloo <peter@chromium.org> Cr-Commit-Position: refs/heads/master@{#512409}
-
Xiaohan Wang authored
Based on the failure volume on Canary channel, choose to crash with a 1/10 probablity on this particular error. We also differentiate between the case where encrypted media init data has been reported or not. This is not equivalent to clear/encrypted playback, but should be close enough for bug investigation purposes. This CL also does minor cleanup (reordering) in SourceBufferState. The crash CHECK will be reverted in a few days after we collect enough crash reports. The CL to revert the crash CHECK is prepared at https://chromium-review.googlesource.com/c/chromium/src/+/742229 PSA: If this is causing too many crashes, feel free to CQ the above revert CL at any time! BUG=778363 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: I1b5f1343a0e7ec18261b6e0ad1dac6740339ff83 Reviewed-on: https://chromium-review.googlesource.com/739743 Commit-Queue: Xiaohan Wang <xhwang@chromium.org> Reviewed-by:
Matthew Wolenetz <wolenetz@chromium.org> Cr-Commit-Position: refs/heads/master@{#512408}
-
Vaclav Brozek authored
This CL moves the ownership of PasswordManagerPorter up from PasswordManagerPresenter to the owner of the presenter. It also makes PasswordManagerPresenter forget about the porter and lets PasswordsPrivateDelegateImpl call the PasswordManagerPorter methods directly, bypassing PasswordManagerPresenter. This removes the confusing dependency cycle between PasswordManagerPorter and PasswordManagerPresenter. Bug: 700003 Change-Id: I9ab4f45adc7dba3be083fa0a94ec3daceaa14eef Reviewed-on: https://chromium-review.googlesource.com/739384Reviewed-by:
Steven Bennetts <stevenjb@chromium.org> Reviewed-by:
Christos Froussios <cfroussios@chromium.org> Commit-Queue: Vaclav Brozek <vabr@chromium.org> Cr-Commit-Position: refs/heads/master@{#512407}
-
skia-deps-roller@chromium.org authored
https://skia.googlesource.com/skia.git/+log/c543f3e03f6b..ed6ae56e0ee8 $ git log c543f3e03..ed6ae56e0 --date=short --no-merges --format='%ad %ae %s' 2017-10-28 fmalita Re-land: Stop using fixed point recs for gradient positioning info Created with: roll-dep src/third_party/skia Documentation for the AutoRoller is here: https://skia.googlesource.com/buildbot/+/master/autoroll/README.md If the roll is causing failures, see: http://www.chromium.org/developers/tree-sheriffs/sheriff-details-chromium#TOC-Failures-due-to-DEPS-rolls 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=mtklein@chromium.org Change-Id: I097133e99efa0c67df35c9220458c398e2474842 Reviewed-on: https://chromium-review.googlesource.com/742234Reviewed-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@{#512406}
-
Chris Harrelson authored
It's important for a paint offset root to be set to a LayoutObject when it has any kind of transform. Paint offset roots are tied to the paint offset transform in the code structure, so it is simpler and easier to understand to always allocate it. The optimization this CL removes had an unclear performance benefit as well, which is another reason to be willing to remove it. Bug: 771300 Cq-Include-Trybots: master.tryserver.chromium.linux:linux_layout_tests_slimming_paint_v2 Change-Id: Idd6d32f8d58872f0e7f25040631990fe78277887 Reviewed-on: https://chromium-review.googlesource.com/741038 Commit-Queue: Chris Harrelson <chrishtr@chromium.org> Reviewed-by:
Philip Rogers <pdr@chromium.org> Cr-Commit-Position: refs/heads/master@{#512405}
-
Miguel Casas-Sanchez authored
This CL creates a new va_surface.cc to accompany the existing va_surface.h, and moves into it the existing methods out of vaapi_wrapper.cc which is already large enough. ** No new code is intended except: added a DCHECK and a few consts. ... but just in case: TEST=compile a simplechrome with the patch, deploy to soraka, then navigate to crosvideo2.appspot.com, reproduce e.g. VP9 1080p, and verify in chrome://media-internals that GpuVideoDecoder is used (and, of course, that the reproduction is as good as w/o the patch). Bug: 778093 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: If778645dba01600f63d1e857a5c9c76f761f0fda Reviewed-on: https://chromium-review.googlesource.com/740041Reviewed-by:
Kuang-che Wu <kcwu@chromium.org> Commit-Queue: Miguel Casas <mcasas@chromium.org> Cr-Commit-Position: refs/heads/master@{#512404}
-
Hector Carmona authored
Bug: 764125 Cq-Include-Trybots: master.tryserver.chromium.linux:closure_compilation Change-Id: Ib37d34dc82f7ee8bcdc0e050fbc6e7d5c2993abb Reviewed-on: https://chromium-review.googlesource.com/738635Reviewed-by:
Dave Schuyler <dschuyler@chromium.org> Commit-Queue: Hector Carmona <hcarmona@chromium.org> Cr-Commit-Position: refs/heads/master@{#512403}
-
Lei Zhang authored
The ASAN failure has been fixed. BUG=757243 Change-Id: I64867f85503aab620d51b6bd2b7a911c31c6bd22 Reviewed-on: https://chromium-review.googlesource.com/741543Reviewed-by:
Xiyuan Xia <xiyuan@chromium.org> Commit-Queue: Lei Zhang <thestig@chromium.org> Cr-Commit-Position: refs/heads/master@{#512402}
-
webrtc-autoroll@chromium.org authored
https://webrtc.googlesource.com/src.git/+log/6a5c6f6f5b90..c7b18fef19ff $ git log 6a5c6f6f5..c7b18fef1 --date=short --no-merges --format='%ad %ae %s' Created with: roll-dep src/third_party/webrtc BUG=None,None Documentation for the AutoRoller is here: https://skia.googlesource.com/buildbot/+/master/autoroll/README.md If the roll is causing failures, see: http://www.chromium.org/developers/tree-sheriffs/sheriff-details-chromium#TOC-Failures-due-to-DEPS-rolls CQ_INCLUDE_TRYBOTS=master.tryserver.chromium.linux:linux_chromium_archive_rel_ng;master.tryserver.chromium.mac:mac_chromium_archive_rel_ng TBR=webrtc-chromium-sheriffs-robots@google.com Change-Id: I1fe66dedef42b412f8e0939965ba25dc493e4ee6 Reviewed-on: https://chromium-review.googlesource.com/742748Reviewed-by:
WebRTC Roll Bot <webrtc-autoroll@chromium.org> Commit-Queue: WebRTC Roll Bot <webrtc-autoroll@chromium.org> Cr-Commit-Position: refs/heads/master@{#512401}
-
webrtc-autoroll@chromium.org authored
https://webrtc.googlesource.com/src.git/+log/5d7fd19c209f..6a5c6f6f5b90 $ git log 5d7fd19c2..6a5c6f6f5 --date=short --no-merges --format='%ad %ae %s' Created with: roll-dep src/third_party/webrtc BUG=None Documentation for the AutoRoller is here: https://skia.googlesource.com/buildbot/+/master/autoroll/README.md If the roll is causing failures, see: http://www.chromium.org/developers/tree-sheriffs/sheriff-details-chromium#TOC-Failures-due-to-DEPS-rolls CQ_INCLUDE_TRYBOTS=master.tryserver.chromium.linux:linux_chromium_archive_rel_ng;master.tryserver.chromium.mac:mac_chromium_archive_rel_ng TBR=webrtc-chromium-sheriffs-robots@google.com Change-Id: I5f69213e24219bb632e68e42c31047dc313fc54a Reviewed-on: https://chromium-review.googlesource.com/742352Reviewed-by:
WebRTC Roll Bot <webrtc-autoroll@chromium.org> Commit-Queue: WebRTC Roll Bot <webrtc-autoroll@chromium.org> Cr-Commit-Position: refs/heads/master@{#512400}
-
W. James MacLean authored
GuestViewMessageFilter::OnAttachToEmbedderFrame() up until now assumed the guest was created with the WebContents* that will be the final owner. This works for WebViews created in static HTML or from JS in the embedder, but can fail if window.open is used, and the embedder web contents may be created along with the new window. This function should instead look up the embedder WebContents* from the provided embedder local frame id, and use that to override in the call to GuestViewBase::WillAttach(). Bug: 777839 Change-Id: I4a33ecf3aaab6dbd645835191247670a3dccd1f3 Reviewed-on: https://chromium-review.googlesource.com/738278 Commit-Queue: James MacLean <wjmaclean@chromium.org> Reviewed-by:
Lucas Gadani <lfg@chromium.org> Reviewed-by:
Avi Drissman <avi@chromium.org> Cr-Commit-Position: refs/heads/master@{#512399}
-
Sebastien Marchand authored
This is a follow up on crrev.com/38f2e322 This CL does several things*: - It greatly simplify the logic in SwapThrashingMonitorDelegateWin: As suggested by wez@ the observation window now looks at how many samples are above a certain threshold rather than looking at the average page-fault rate over an interval of time. This average rate was really affected by the small burst in paging and this was causing a lot of unwanted switch to the suspected state. The delta on this CL isn't displayed properly, swap_thrashing_monitor_win.h has been renamed to swap_thrashing_monitor_delegate_win.h and the SampleWindow subclass now replace the PageFaultAverageRate one. - It introduce the monitor that owns this delegate. The monitor is responsible of calling the delegate at regular intervals to sample the state of the system. - It emits some new metrics to record how much time is spent in each state and how often the system transition from one step to another. * Sorry for sending this a one big CL... The timezone difference will really slow down the release of this if I split it into too many smaller CLs (and there's only a few days left for me to work on this). Bug: 771478 TBR: chrisha@chromium.org Change-Id: I8aa5c8fd2c768a3e1ad0c387a67c7205db5da37a Reviewed-on: https://chromium-review.googlesource.com/738072 Commit-Queue: Sébastien Marchand <sebmarchand@chromium.org> Reviewed-by:
Jochen Eisinger <jochen@chromium.org> Reviewed-by:
Ilya Sherman <isherman@chromium.org> Reviewed-by:
Kenichi Ishibashi <bashi@chromium.org> Cr-Commit-Position: refs/heads/master@{#512398}
-
Quinten Yearsley authored
Reason: Currently of the blink_rel try bots, the mac bots are the slowest. Bug: 772335 Change-Id: I087fbd652c6e11d2d884905859f13e1601d0dbe5 Reviewed-on: https://chromium-review.googlesource.com/737415Reviewed-by:
Raphael Kubo da Costa (rakuco) <raphael.kubo.da.costa@intel.com> Reviewed-by:
Dirk Pranke <dpranke@chromium.org> Commit-Queue: Raphael Kubo da Costa (rakuco) <raphael.kubo.da.costa@intel.com> Cr-Commit-Position: refs/heads/master@{#512397}
-
Ramin Halavati authored
To correctly classify images to those suitable for color inversion and those not, we need to detect if some part of the image is static background and ignore it while sampling from the image. This is because frequently there are images with very large static background and some small foreground. Sampling uniformly from the whole image results in very few samples from foreground and most samples from the large background, and this skews the classifier towards inversion, while the decision must be taken based on the foreground area. Bug: 685242 Cq-Include-Trybots: master.tryserver.chromium.linux:linux_layout_tests_slimming_paint_v2 Change-Id: I5b7bd9d2ed1b52913684e7ba2fdc6a2fd0e4a46a Reviewed-on: https://chromium-review.googlesource.com/720375Reviewed-by:
Chris Harrelson <chrishtr@chromium.org> Reviewed-by:
Dominic Mazzoni <dmazzoni@chromium.org> Commit-Queue: Ramin Halavati <rhalavati@chromium.org> Cr-Commit-Position: refs/heads/master@{#512396}
-
Koji Ishii authored
This patch changes NGInlineNode::ComputeOffsetMappingIfNeeded() to return a pointer instead of a ref, in preparation of an NGInlineNode may not always be laid out by LayoutNG. Also makes sure it gets NGInlineNode from NGBlockNode so that it uses the correct LayoutObject for multi-column case. Bug: 636993 Cq-Include-Trybots: master.tryserver.chromium.linux:linux_layout_tests_layout_ng Change-Id: Iaf16e39a30ca79c83db148bfdae40c5df7959e13 Reviewed-on: https://chromium-review.googlesource.com/741102 Commit-Queue: Koji Ishii <kojii@chromium.org> Reviewed-by:
Xiaocheng Hu <xiaochengh@chromium.org> Cr-Commit-Position: refs/heads/master@{#512395}
-
Martijn Croonen authored
This CL adds two checks to the transport security state JSON file parser that ensure the entries have a valid mode ("force-https" or empty) and that the entries don't contain any unknown fields. These checks are intended to catch typo's that would othersize degrade the security of the preloaded domains. Bug: 774246 Change-Id: Ifcb4f4994f982078cc72d7df10e6245da2d9909c Reviewed-on: https://chromium-review.googlesource.com/738117 Commit-Queue: Martijn Croonen <martijnc@chromium.org> Reviewed-by:
Ryan Sleevi <rsleevi@chromium.org> Cr-Commit-Position: refs/heads/master@{#512394}
-
Chromium WPT Sync authored
Using wpt-import in Chromium caf0bdce. Build: https://build.chromium.org/p/chromium.infra.cron/builders/wpt-importer/builds/4005 Note to sheriffs: This CL imports external tests and adds expectations for those tests; if this CL is large and causes a few new failures, please fix the failures by adding new lines to TestExpectations rather than reverting. See: https://chromium.googlesource.com/chromium/src/+/master/docs/testing/web_platform_tests.md TBR=qyearsley@chromium.org No-Export: true Change-Id: Ib9c8022d1505fb910257602994ebc5207b1893a2 Reviewed-on: https://chromium-review.googlesource.com/742523 Commit-Queue: Blink WPT Bot <blink-w3c-test-autoroller@chromium.org> Reviewed-by:
Blink WPT Bot <blink-w3c-test-autoroller@chromium.org> Cr-Commit-Position: refs/heads/master@{#512393}
-
Koji Ishii authored
Following bot results are included. 1933 1934 1935 1938 1939 TBR=eae@chromium.org NOTRY=true Bug: 591099 Change-Id: Ibdc98e8e279f2a7b9e304c073db3f5cfc636a491 Reviewed-on: https://chromium-review.googlesource.com/742821Reviewed-by:
Koji Ishii <kojii@chromium.org> Commit-Queue: Koji Ishii <kojii@chromium.org> Cr-Commit-Position: refs/heads/master@{#512392}
-
Lei Zhang authored
Change-Id: I70a696a5585262668bedf4485f79e8498e649212 Reviewed-on: https://chromium-review.googlesource.com/741080 Commit-Queue: Lei Zhang <thestig@chromium.org> Reviewed-by:
Tom Sepez <tsepez@chromium.org> Cr-Commit-Position: refs/heads/master@{#512391}
-
Chris Cunningham authored
Associates the VideoDecodePerfHistory with the BrowserContext so users can clear just their history via chrome://settings/clearBrowsingData. VideoDecodePerfHistory connected in //content rather than //chrome to facilitate easy use by embedders (e.g. webview). Bug: 695264 Cq-Include-Trybots: master.tryserver.chromium.linux:linux_site_isolation Change-Id: Ic9fec3a7a450a3be49e231c919db227ac837b3c3 Reviewed-on: https://chromium-review.googlesource.com/707840 Commit-Queue: Chrome Cunningham <chcunningham@chromium.org> Reviewed-by:
Dale Curtis <dalecurtis@chromium.org> Reviewed-by:
Charlie Reis <creis@chromium.org> Reviewed-by:
John Abd-El-Malek <jam@chromium.org> Cr-Commit-Position: refs/heads/master@{#512390}
-
Qiang Xu authored
changes: The old logic in https://codereview.chromium.org/2250963003 seems causing problem now, described in crbug.com/777463. We need to reconsider the solution for crbug.com/625222. TBR=xiyuan@chromium.org, jennyz@chromium.org Bug: 777463, 712072 Test: test that crbug.com/777463 is fixed. Change-Id: I7f4a83896febaf05c74a83caf14afeb02b8758ac Reviewed-on: https://chromium-review.googlesource.com/742390 Commit-Queue: Qiang(Joe) Xu <warx@chromium.org> Reviewed-by:
Qiang(Joe) Xu <warx@chromium.org> Cr-Commit-Position: refs/heads/master@{#512389}
-
src-internal-roller@chromium.org authored
https://chrome-internal.googlesource.com/chrome/src-internal.git/+log/cbae27f0a0f7..6107a9e99383 $ git log cbae27f0a..6107a9e99 --date=short --no-merges --format='%ad %ae %s' Created with: roll-dep src-internal BUG=412884 Documentation for the AutoRoller is here: https://skia.googlesource.com/buildbot/+/master/autoroll/README.md If the roll is causing failures, see: http://www.chromium.org/developers/tree-sheriffs/sheriff-details-chromium#TOC-Failures-due-to-DEPS-rolls TBR=phajdan.jr@chromium.org Change-Id: Ie6cf43eaf9903404f9ba059d8fa6bab30686efd2 Reviewed-on: https://chromium-review.googlesource.com/742491Reviewed-by:
src-internal-roller . <src-internal-roller@chromium.org> Commit-Queue: src-internal-roller . <src-internal-roller@chromium.org> Cr-Commit-Position: refs/heads/master@{#512388}
-
Lei Zhang authored
Change-Id: I4a755f8bb007e5945c5b8786eaf69c6b6544c6b4 Reviewed-on: https://chromium-review.googlesource.com/740906Reviewed-by:
John Abd-El-Malek <jam@chromium.org> Commit-Queue: Lei Zhang <thestig@chromium.org> Cr-Commit-Position: refs/heads/master@{#512387}
-
Han Leon authored
This is a follow-up of CL https://chromium-review.googlesource.com/c/chromium/src/+/741522/ git cl format --js chrome/test/data/extensions/api_test/serial/api/background.js BUG=777552 TBR=reillyg@chromium.org Change-Id: Icd4379286f862a1a17cca3b03e1f865591882714 Reviewed-on: https://chromium-review.googlesource.com/742230Reviewed-by:
Han Leon <leon.han@intel.com> Commit-Queue: Han Leon <leon.han@intel.com> Cr-Commit-Position: refs/heads/master@{#512386}
-