- 02 Feb, 2018 40 commits
-
-
Koji Ishii authored
Following bot results are included. 3505 3507 1 lines were removed and 21 lines were deflaked by consecutive results since 3426. TBR=eae@chromium.org NOTRY=true Bug: 591099 Cq-Include-Trybots: master.tryserver.chromium.linux:linux_layout_tests_layout_ng Change-Id: I613a05c32c113147ad454c73db91e9387fea5b9e Reviewed-on: https://chromium-review.googlesource.com/896918 Commit-Queue: Koji Ishii <kojii@chromium.org> Reviewed-by:
Koji Ishii <kojii@chromium.org> Cr-Commit-Position: refs/heads/master@{#534033}
-
Roman Sorokin authored
With 'Admin-facing' string in the desc. BUG=chromium:807218 TEST=none Change-Id: I86f60fc351ce4229e48943ae8f8ed7fac9cb966a Reviewed-on: https://chromium-review.googlesource.com/897537 Commit-Queue: Roman Sorokin <rsorokin@chromium.org> Reviewed-by:
Lutz Justen <ljusten@chromium.org> Cr-Commit-Position: refs/heads/master@{#534032}
-
https://webrtc.googlesource.com/src.git/+log/7dd9d6f54b8d..c22d6a8f9b3e $ git log 7dd9d6f54..c22d6a8f9 --date=short --no-merges --format='%ad %ae %s' Created with: roll-dep src/third_party/webrtc The AutoRoll server is located here: https://webrtc-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.linux:linux_chromium_archive_rel_ng;master.tryserver.chromium.mac:mac_chromium_archive_rel_ng;master.tryserver.chromium.win:win-msvc-dbg TBR=webrtc-chromium-sheriffs-robots@google.com Change-Id: Iefef39cd2dc77ab286d279f5af8a7936edda2a17 Reviewed-on: https://chromium-review.googlesource.com/898501 Commit-Queue: webrtc-chromium-autoroll <webrtc-chromium-autoroll@skia-buildbots.google.com.iam.gserviceaccount.com> Reviewed-by:
webrtc-chromium-autoroll <webrtc-chromium-autoroll@skia-buildbots.google.com.iam.gserviceaccount.com> Cr-Commit-Position: refs/heads/master@{#534031}
-
Shu Chen authored
This is a partial roll back of https://codereview.chromium.org/2597413002/, which didn't consider the switch back from a null engine. The new solution is to use the global observer - ui::IMEBridgeObserver to notify the InputImeAPI, instead of using the engine. Change-Id: I2fd4e93b212d6b0caa8a9a4aab5314753a07447b Reviewed-on: https://chromium-review.googlesource.com/896775Reviewed-by:
Lan Wei <azurewei@chromium.org> Commit-Queue: Shu Chen <shuchen@chromium.org> Cr-Commit-Position: refs/heads/master@{#534030}
-
Javier Fernandez authored
This is a fix for a regression introduced in r515391, where we landed the implementation of alignment for positioned objects in a Grid Layout container. We assumed that items with non-static positions shouldn't honor the CSS Box Alignment properties. This is only true for out-of-flow items, absolute or fixed positioned elements. However, sticky positioned elements are treated as relative positioned items, but they indeed use non-static position to define their behavior. Bug: 806622 Change-Id: Ibaed2c36530d81223d1491def9022e82cf9c3525 Reviewed-on: https://chromium-review.googlesource.com/897949 Commit-Queue: Javier Fernandez <jfernandez@igalia.com> Reviewed-by:
Manuel Rego Casasnovas <rego@igalia.com> Reviewed-by:
Christian Biesinger <cbiesinger@chromium.org> Cr-Commit-Position: refs/heads/master@{#534029}
-
Pramod B S authored
Modified files in peerconnection module. Spec: https://github.com/heycam/webidl/issues/365 Bug: 792432 Change-Id: Ic3c8c895d6d71737b07aca504662ab756b06a9d9 Reviewed-on: https://chromium-review.googlesource.com/896916Reviewed-by:
Kentaro Hara <haraken@chromium.org> Commit-Queue: srirama chandra sekhar <srirama.m@samsung.com> Cr-Commit-Position: refs/heads/master@{#534028}
-
Andreas Haas authored
We used ScriptForbiddenScope::IsScriptForbidded to detect if we can reject a promise when we abort streaming compilation. With this CL we use IsContextDestroyed instead because it is more meaningful and serves the same purpose. R=haraken@chromium.org Change-Id: Ifc10fcffca8c9abdf7c88a9f58562c43c8d2d4a1 Reviewed-on: https://chromium-review.googlesource.com/897428Reviewed-by:
-
Hajime Hoshi authored
This is part of efforts to replace non-per-frame task runners with other appropriate task runners in the renderer. Bug: 804694 Change-Id: Ibcb96b6461fda0d777edcb961bc21294fcc5cf23 Reviewed-on: https://chromium-review.googlesource.com/897308Reviewed-by:
-
https://chromium.googlesource.com/chromium/tools/depot_tools.git/+log/e150d63db9f7..1f067b88df42 $ git log e150d63db..1f067b88d --date=short --no-merges --format='%ad %ae %s' 2018-02-02 machenbach Optimize check for existing files in download_from_google_storage Created with: roll-dep src/third_party/depot_tools The AutoRoll server is located here: https://depot-tools-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. TBR=agable@chromium.org Change-Id: Ib01ccf2fcd5646ced79cb3ea36253113f77af4e3 Reviewed-on: https://chromium-review.googlesource.com/899022Reviewed-by:
depot-tools-chromium-autoroll <depot-tools-chromium-autoroll@skia-buildbots.google.com.iam.gserviceaccount.com> Commit-Queue: depot-tools-chromium-autoroll <depot-tools-chromium-autoroll@skia-buildbots.google.com.iam.gserviceaccount.com> Cr-Commit-Position: refs/heads/master@{#534025}
-
Nicolas Dossou-gbete authored
Setting enable_android_deps_repository to true will let the build use the dependencies defined in //third_party/android_deps, that will be added by https://crrev.com/c/819771. By default this flag is disabled. Bug: 807641 Change-Id: If75f7bbac38d2dc807ae35cdd8de0f229bcf6545 Reviewed-on: https://chromium-review.googlesource.com/895691 Commit-Queue: Nicolas Dossou-Gbété <dgn@chromium.org> Reviewed-by:
agrieve <agrieve@chromium.org> Cr-Commit-Position: refs/heads/master@{#534024}
-
Keishi Hattori authored
Check if table_ exists because the table may be freed after the callback has been registered when incremental marking is enabled. Bug: 757440 Change-Id: I677ae294f5d66f539ea6e68ddab29053aaf6c503 Reviewed-on: https://chromium-review.googlesource.com/896913Reviewed-by:
Michael Lippautz <mlippautz@chromium.org> Reviewed-by:
-
Keishi Hattori authored
rare_data_ is a raw pointer instead of a Member (because it is in a union). Hence it needs a manual write barrier. Bug: 757440 Change-Id: I8852471d3c1d7db3e1db7fc1d6f74fbe3f33f311 Reviewed-on: https://chromium-review.googlesource.com/896909Reviewed-by:
-
Keishi Hattori authored
Fixes the unused variable 'kCanClearUnusedSlotsWithMemset' error. Bug: None Change-Id: Ib1f85353e7e4523b865648f8bbbb660f94585852 Reviewed-on: https://chromium-review.googlesource.com/898906Reviewed-by:
-
Chris Pickel authored
This reverts commit a55fe78f. Reason for revert: Possible source of flakiness in AutomationApiTest.DesktopHitTestIframe Original change's description: > Expose reverse relationships in the automation API. > > AXTree now keeps track of reverse relationships automatically, > so take advantage of that to expose reverse relationships for > some attributes already exposed in the automation API. > Also add two relation attributes that are supported by Blink > but weren't added to automation yet: details, errorMessage. > > The new attributes are: > > detailsFor (reverse of details) > errorMessageFor (reverse of errorMessage) > controlledBy (reverse of controls) > descriptionFor (reverse of describedBy) > flowFrom (reverse of flowTo) > labelFor (reverse of labelledBy) > > Bug: 757707 > Cq-Include-Trybots: master.tryserver.chromium.linux:closure_compilation > Change-Id: I112c283c87274388fc9e7e725bcc126393133e39 > TBR: michaelpg@chromium.org > Reviewed-on: https://chromium-review.googlesource.com/837643 > Commit-Queue: Dominic Mazzoni <dmazzoni@chromium.org> > Reviewed-by: David Tseng <dtseng@chromium.org> > Cr-Commit-Position: refs/heads/master@{#533819} TBR=dmazzoni@chromium.org,dtseng@chromium.org,michaelpg@chromium.org Bug: 757707 Bug: 808343 Change-Id: Ie61c369e0f234eca7435ac80d056a6c10ea75a41 Reviewed-on: https://chromium-review.googlesource.com/897864Reviewed-by:
Chris Pickel <sfiera@chromium.org> Commit-Queue: Chris Pickel <sfiera@chromium.org> Cr-Commit-Position: refs/heads/master@{#534020}
-
https://webrtc.googlesource.com/src.git/+log/c7b964cd71fb..7dd9d6f54b8d $ git log c7b964cd7..7dd9d6f54 --date=short --no-merges --format='%ad %ae %s' Created with: roll-dep src/third_party/webrtc BUG=None The AutoRoll server is located here: https://webrtc-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.linux:linux_chromium_archive_rel_ng;master.tryserver.chromium.mac:mac_chromium_archive_rel_ng;master.tryserver.chromium.win:win-msvc-dbg TBR=webrtc-chromium-sheriffs-robots@google.com Change-Id: I1d4edf15d11ce99bc913b8c9cc9c306ecf2c1933 Reviewed-on: https://chromium-review.googlesource.com/898942 Commit-Queue: webrtc-chromium-autoroll <webrtc-chromium-autoroll@skia-buildbots.google.com.iam.gserviceaccount.com> Reviewed-by:
-
Christos Froussios authored
1. Empty space is introduced between the buttons in the export dialog. 2. The export menu item is the More Actions menu is changed to a different string. Additionally, the string is moved to settings_strings.grdp, next to the rest of the strings. Mocks: https://docs.google.com/presentation/d/1nIm5OmaOnb85ZAwMZPSVqHT0vkFbsRYZhOzbmznWU_c/edit#slide=id.g29b76db41a_0_6 Bug: 789561 Cq-Include-Trybots: master.tryserver.chromium.linux:closure_compilation Change-Id: Id939ebe158a68ff71e12a783919c7baaea1ed00f Reviewed-on: https://chromium-review.googlesource.com/892881 Commit-Queue: Christos Froussios <cfroussios@chromium.org> Reviewed-by:
Hector Carmona <hcarmona@chromium.org> Cr-Commit-Position: refs/heads/master@{#534018}
-
arthursonzogni authored
Content-Security-Policy: The CSP source 'self' is usually the origin of the current document. Immediately after an new window or new frame is created, there are no current document. In this case, the origin used is the one of the opener (in case of a new window) or the parent (in case of a new iframe). For you intention: The frame's CSP are already the one of its opener when there are still no committed document. It makes sense to do the same with 'self'. Several web platform tests are added. Bug: 807206 Change-Id: I2acf66d9b6d63d4efb14370a4d0ff2206c943aeb Reviewed-on: https://chromium-review.googlesource.com/895589 Commit-Queue: Arthur Sonzogni <arthursonzogni@chromium.org> Reviewed-by:
Alex Moshchuk <alexmos@chromium.org> Reviewed-by:
Mike West <mkwst@chromium.org> Cr-Commit-Position: refs/heads/master@{#534017}
-
Tsuyoshi Horo authored
This CL adds the following data files for fuzzer: 1: Same as SignedExchangeCertFetcherTest.OneCert 2: Same as SignedExchangeCertFetcherTest.OneCertWithExtension 3: Same as SignedExchangeCertFetcherTest.TwoCerts 4: The actual cert of www.example.com and the intermediate cert. Bug: 803774 Change-Id: I55182556a39c1ed852f2412df18707eae7a7c2c2 Reviewed-on: https://chromium-review.googlesource.com/896770Reviewed-by:
Kinuko Yasuda <kinuko@chromium.org> Reviewed-by:
Kouhei Ueno <kouhei@chromium.org> Commit-Queue: Tsuyoshi Horo <horo@chromium.org> Cr-Commit-Position: refs/heads/master@{#534016}
-
Kush Sinha authored
Bug: 807568 Change-Id: Id9b249fdbacfcd7b64cc7256a33204778f437450 Reviewed-on: https://chromium-review.googlesource.com/897671 Commit-Queue: Kush Sinha <sinhak@chromium.org> Reviewed-by:
David Roger <droger@chromium.org> Cr-Commit-Position: refs/heads/master@{#534015}
-
Alex Clarke authored
This bug was causing tasks to occasionally run sooner than intended. Cq-Include-Trybots: master.tryserver.chromium.linux:linux_layout_tests_layout_ng Change-Id: I93516407032313c1a0026302f49a714c3bbeae40 Reviewed-on: https://chromium-review.googlesource.com/897645Reviewed-by:
Alexander Timin <altimin@chromium.org> Commit-Queue: Alex Clarke <alexclarke@chromium.org> Cr-Commit-Position: refs/heads/master@{#534014}
-
Kent Tamura authored
Simplify Document::createElementNS() code by using Document::createElement(qname, flags). There were no createElement(qname, kCreatedByCreateElement). Removing |flags != kCreatedByCreateElement| in it is safe. This CL has no behavior changes. Bug: 806641 Change-Id: I79112ccedb35ad3804e84fc62def3b0c395b1a7b Reviewed-on: https://chromium-review.googlesource.com/896910Reviewed-by:
Yoshifumi Inoue <yosin@chromium.org> Commit-Queue: Yoshifumi Inoue <yosin@chromium.org> Cr-Commit-Position: refs/heads/master@{#534013}
-
Takashi Toyoshima authored
This patch moves CORSURLLoaderFactory and unit tests for CORSURLLoader and CORSURLLoaderFactory to services/network. Only Mechanical changes. Since now that this patch remove the last source files in content/network, this patch also removes content/network directory completely. It affects WATCHLISTS, other BUILD.gn, DEPS, and OWNERS files that refer content/network. The patch also includes mechanical string replacements, s/content\/network/services\/network/g. Bug: 807565 Cq-Include-Trybots: master.tryserver.chromium.linux:linux_mojo Change-Id: Iba027a1c625d421e5671dd3faac5d32c1d2136ad Tbr: jam@chromium.org Reviewed-on: https://chromium-review.googlesource.com/895217 Commit-Queue: Takashi Toyoshima <toyoshim@chromium.org> Reviewed-by:
-
Koji Ishii authored
Following bot results are included. 3489 3494 3497 3500 3502 22 lines were removed and 31 lines were deflaked by consecutive results since 3410. TBR=eae@chromium.org NOTRY=true Bug: 591099 Cq-Include-Trybots: master.tryserver.chromium.linux:linux_layout_tests_layout_ng Change-Id: I0ae30c41711a7a3cb76154d61e11229f9c16cfd2 Reviewed-on: https://chromium-review.googlesource.com/898142 Commit-Queue: Koji Ishii <kojii@chromium.org> Reviewed-by:
-
Chandan Padhi authored
This is a reland of 2b33f996. Original change's description: > Migrate MediaStreamDispatcherHost to be per frame > > Currently, there is only one MediaStreamDispatcherHost per renderer > process, but almost all its methods require a frame ID to be supplied > by the caller. This CL migrates this object to be per frame. > > Bug: 807274 > Change-Id: I0b7f93e067b789cea5eba020374eb6da243e4890 > Reviewed-on: https://chromium-review.googlesource.com/894864 > Reviewed-by: Guido Urdaneta <guidou@chromium.org> > Reviewed-by: Avi Drissman <avi@chromium.org> > Reviewed-by: Tom Sepez <tsepez@chromium.org> > Cr-Commit-Position: refs/heads/master@{#533970} TBR=guidou, avi, tsepez Bug: 807274 Change-Id: Idca20b52a27123cabfbbb55fe570c97a582d4e7a Reviewed-on: https://chromium-review.googlesource.com/897321Reviewed-by:
Guido Urdaneta <guidou@chromium.org> Commit-Queue: Guido Urdaneta <guidou@chromium.org> Cr-Commit-Position: refs/heads/master@{#534010}
-
Tien-Ren Chen authored
This CL adds clip-path support to SPv175+ by the following changes: 1. PropertyTreeBuilder now builds a clip node (ClipPathClip) or a effect node (ClipPath), depend on whether path-based or mask-based clip path should be used. 2. A per-fragment clip-path cache that saves the bounding box and (if available) the path of path-based clip path. This cache will maintain the same pointer identity across document lifecycles so that clip node comparison during raster invalidation won't incur expensive by-value comparison for paths. 3. ClipPathClipper now paints path-based clip path in appropriate property state. HTML-based contents apply property state in a bundle by local border box property (unchanged), while SVG-based contents apply property states manually in SVGPaintContext. 4. Implement SPv1 composited mask layer adjustment. The hierarchy of effect tree and clip tree are slightly different because in SPv1 composited clip path are always applied to a mask layer then apply to actual contents indirectly. BUG=769942,792769 Cq-Include-Trybots: master.tryserver.blink:linux_trusty_blink_rel;master.tryserver.chromium.linux:linux_layout_tests_slimming_paint_v2 Change-Id: I8dcd5fad685067dc22efda9dedfd6ab6366a26a7 Reviewed-on: https://chromium-review.googlesource.com/896607 Commit-Queue: Tien-Ren Chen <trchen@chromium.org> Reviewed-by:
Xianzhu Wang <wangxianzhu@chromium.org> Reviewed-by:
Chris Harrelson <chrishtr@chromium.org> Cr-Commit-Position: refs/heads/master@{#534009}
-
Jens Widell authored
In jumbo/unity builds, it conflicts with another constant in bindings/ api_event_listeners.cc (that has a different value.) This particular constant didn't seem to add too much. It was only used in one place; in a function with a comment that talks about the constant's value. It could be argued that the code is easier to follow if the (very simple) function implementation then uses the value directly. (In particular since the "value" is in fact the name of another constant.) Change-Id: I728a34b0a8b43382e742c6b640ace3baadc4d2db Reviewed-on: https://chromium-review.googlesource.com/897489Reviewed-by:
Devlin <rdevlin.cronin@chromium.org> Commit-Queue: Jens Widell <jl@opera.com> Cr-Commit-Position: refs/heads/master@{#534008}
-
Jens Widell authored
They conflict in jumbo/unity builds. Since one read a file as binary data (into a `std::vector<uint8_t>`) and one as text (into a `std::string`), it seems reasonable to change the names to `ReadAndDeleteBinaryFile()` and `ReadAndDeleteTextFile()`. Change-Id: I8d8402c64cfe45132df9640bd5664df28742d6d7 Reviewed-on: https://chromium-review.googlesource.com/897603Reviewed-by:
Ken Rockot <rockot@chromium.org> Commit-Queue: Jens Widell <jl@opera.com> Cr-Commit-Position: refs/heads/master@{#534007}
-
Chris Pickel authored
This reverts commit f0baa069. Reason for revert: "BackgroundTest.ActiveDescendantUpdates" is flaky. Since it appears to have been added in this CL, reverting for dtseng to take another look. Original change's description: > Force updates to related nodes > > Test: chromevox_tests. Prior to the change, > BackgroundTest.ActiveDescendantUpdates fails because 'Selected' is not spoken. > > Compute text content for list items without names > > Chrome doesn't compute descendant name on focus. > > Test: manually on twitter.com using j/k. Verify all text gets read. > Cq-Include-Trybots: master.tryserver.chromium.linux:closure_compilation > Change-Id: Ie75dff41f98b8aa3904e435afa4bd245b9771cce > Reviewed-on: https://chromium-review.googlesource.com/891676 > Commit-Queue: David Tseng <dtseng@chromium.org> > Reviewed-by: Nektarios Paisios <nektar@chromium.org> > Cr-Commit-Position: refs/heads/master@{#533960} TBR=dtseng@chromium.org,nektar@chromium.org Bug: 808337 Change-Id: I4f838f27aa81a607bcb83934bec0dc321d336466 No-Presubmit: true No-Tree-Checks: true No-Try: true Cq-Include-Trybots: master.tryserver.chromium.linux:closure_compilation Reviewed-on: https://chromium-review.googlesource.com/897789Reviewed-by:
-
Jens Widell authored
They conflict in jumbo/unity builds. Change-Id: Ie5eae8aa303f5a163eabb74ff931f696127ff032 Reviewed-on: https://chromium-review.googlesource.com/897602Reviewed-by:
-
Shanmuga Pandi M authored
This patch will show deprecate warning for the extra form data which was added, if "value" attribute present with non-empty value for <input type='image'> Intent to Deprecate and Remove: https://groups.google.com/a/chromium.org/forum/#!topic/blink-dev/hp1_-1tgvDs Bug: 753746 Change-Id: I565132d6ef8c6d4a93ce5447e5d18d7f7f8c1910 Reviewed-on: https://chromium-review.googlesource.com/896740Reviewed-by:
Kent Tamura <tkent@chromium.org> Commit-Queue: Shanmuga Pandi <shanmuga.m@samsung.com> Cr-Commit-Position: refs/heads/master@{#534004}
-
Yoshifumi Inoue authored
This patch makes |ComputeSelectionType()| to take |EphemeralRange| instead of two positions |start| and |end| to make sure we keep invariant of |start| < |end| for improving code health. Change-Id: Ibe2427899f7b60b9a37e23c516172203276ab12c Reviewed-on: https://chromium-review.googlesource.com/898683Reviewed-by:
Yoichi Osato <yoichio@chromium.org> Commit-Queue: Yoshifumi Inoue <yosin@chromium.org> Cr-Commit-Position: refs/heads/master@{#534003}
-
Yuta Kitamura authored
This reverts commit 6d10aad7. Reason for revert: Caused MSAN bot failure. https://ci.chromium.org/buildbot/chromium.memory/Linux%20MSan%20Tests/7639 Original change's description: > [css-typed-om] Refactor CSSStyleImageValue to prepare for gradients. > > Currently CSSStyleImageValue implements the CSSImageValue class in > Typed OM. CSSImageValue can be anything that's an <image> i.e. a <url>, > <gradient>, or <cross-fade>. > > However, the current implementation of CSSStyleImageValue only really > works for <url>s. This patch moves <url> specific logic into > CSSURLImageValue. This prepares for the implementation of <gradient> and > <cross-fade>, which will be subclasses of CSSStyleImageValue. > > We also do some cleanup work (e.g. making things const, moving code in > header to cpp). > > Bug: 803680 > Change-Id: I848b3eab5dbbcb9bbe7f0aea0d377497f63bfda3 > Reviewed-on: https://chromium-review.googlesource.com/882786 > Commit-Queue: Darren Shen <shend@chromium.org> > Reviewed-by: nainar <nainar@chromium.org> > Cr-Commit-Position: refs/heads/master@{#533949} TBR=nainar@chromium.org,shend@chromium.org Change-Id: I9aedca3ac9e3ecb07ed283447febede0628d7aeb No-Presubmit: true No-Tree-Checks: true No-Try: true Bug: 803680 Reviewed-on: https://chromium-review.googlesource.com/896671Reviewed-by:
Yuta Kitamura <yutak@chromium.org> Commit-Queue: Yuta Kitamura <yutak@chromium.org> Cr-Commit-Position: refs/heads/master@{#534002}
-
Marina Ciocea authored
Replace duplicate implementation of AudioDeviceDescription::IsLoopbackDevice with function call. Bug: 783109 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: I896038026359723d8fe1c4366cd3a54ddd51e9c1 Reviewed-on: https://chromium-review.googlesource.com/897534Reviewed-by:
Max Morin <maxmorin@chromium.org> Commit-Queue: Marina Ciocea <marinaciocea@chromium.org> Cr-Commit-Position: refs/heads/master@{#534001}
-
v8-autoroll authored
Summary of changes available at: https://chromium.googlesource.com/v8/v8/+log/09eb87e6..33f6b756 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: Id386ff99db678d09ae255d4b2a066f209b443bf0 Reviewed-on: https://chromium-review.googlesource.com/898783Reviewed-by:
v8 autoroll <v8-autoroll@chromium.org> Commit-Queue: v8 autoroll <v8-autoroll@chromium.org> Cr-Commit-Position: refs/heads/master@{#534000}
-
Jens Widell authored
In jumbo/unity builds, some of the constants conflicted with (identical) declarations in the anonymous namespace in verified_content.cc. Change-Id: I66e9ecb180e99cfd468c914ddce44613a5013242 Reviewed-on: https://chromium-review.googlesource.com/897604Reviewed-by:
-
Yoshifumi Inoue authored
This patch changes "push-down-implicit-styles-mac.html" with utilizing |selection_test()| for ease of maintenance. Because of test cases specify selected range directly instead of using |Selection#modify()|, test cases are independent from platform, e.g. selecting whitespaces after word. Following patch will rename this test file with remove "-mac" suffix to "push-down-implicit-styles.html" since this test file no more depends on MacOS. [1] http://crrev.com/c/896764 Rewrite editing/execCommand/remove-format-multiple-elements-{mac,-win}.html to use selection_test() Bug: 679977 Change-Id: I5c9a77d5d48651a9f3936496ca14b195a5633828 Reviewed-on: https://chromium-review.googlesource.com/898765Reviewed-by:
-
Yoshifumi Inoue authored
This patch changes |CreateVisibleSelection()| to utilize |ComputeRange()| of |SelectionTemplate| to simplify source code for improving code health. Change-Id: I3d19d963cf04acba7f7bb0cb090dd45139f914a4 Reviewed-on: https://chromium-review.googlesource.com/897290Reviewed-by:
-
Yoshifumi Inoue authored
This patch changes "push-down-font-styles{,-mac}.html" into "push-down-font-styles.html" with utilizing |selection_test()| for ease of maintenance. Because of test cases specify selected range directly instead of using |Selection#modify()|, test cases are independent from platform, e.g. selecting whitespaces after word. [1] http://crrev.com/c/896764 Rewrite editing/execCommand/remove-format-multiple-elements-{mac,-win}.html to use selection_test() Bug: 679977 Change-Id: If2a26ed1d3007048d6799ce8f7fb20e12903d9c4 Reviewed-on: https://chromium-review.googlesource.com/898764Reviewed-by: -
Yoshifumi Inoue authored
This patch Introduces printers for |EphemeralRange| and |EphemeralRangeInFlatTree| for ease of debugging and a preparation of the patch[1]. [1] http://crrev.com/c/898683 Make ComputeSelectionType() to take EphemeralRange Change-Id: I2988679b2c9febe50d29a9330ac6e5577972ca20 Reviewed-on: https://chromium-review.googlesource.com/896739Reviewed-by:
-
Yoshifumi Inoue authored
This patch changes "push-down-implicit-styles-around-list-{mac,win}.html" into "push-down-implicit-styles-around-list.html" with utilizing |selection_test()| for ease of maintenance. Because of test cases specify selected range directly instead of using |Selection#modify()|, test cases are independent from platform, e.g. selecting whitespaces after word. [1] http://crrev.com/c/896764 Rewrite editing/execCommand/remove-format-multiple-elements-{mac,-win}.html to use selection_test() Bug: 679977 Change-Id: I21267338d42fe7400be683b102fb85168bb6b01b Reviewed-on: https://chromium-review.googlesource.com/896777Reviewed-by:
-
