- 29 Jan, 2018 40 commits
-
-
Sylvain Defresne authored
Add variable checkout_ios_internal to conditionally fetch ios_internal. Use pinned revision to ensure build stays hermetic and reproducible. Bug: 803846 Change-Id: Id8ba650400b283645f8c00629dd269739d813195 Reviewed-on: https://chromium-review.googlesource.com/880903 Commit-Queue: Sylvain Defresne <sdefresne@chromium.org> Reviewed-by:
Dirk Pranke <dpranke@chromium.org> Reviewed-by:
Michael Moss <mmoss@chromium.org> Reviewed-by:
John Budorick <jbudorick@chromium.org> Reviewed-by:
Rohit Rao <rohitrao@chromium.org> Cr-Commit-Position: refs/heads/master@{#532437}
-
Robert Ma authored
The two tests rely on CoreText's OpenType feature support. They were previously identified as failing on Mac 10.12. After being tried on Mac 10.13, they still fail. No-Try: True Bug: 774301, 752449 Change-Id: If0cc92a6896e47cadae863a026f2f97770b68cf8 Reviewed-on: https://chromium-review.googlesource.com/881768Reviewed-by:
Dominik Röttsches <drott@chromium.org> Commit-Queue: Robert Ma <robertma@chromium.org> Cr-Commit-Position: refs/heads/master@{#532436}
-
Gauthier Ambard authored
Bug: 806784 Cq-Include-Trybots: master.tryserver.chromium.mac:ios-simulator-cronet;master.tryserver.chromium.mac:ios-simulator-full-configs Change-Id: I687c994bc89696eed93f58f93621740b0fc3c37e Reviewed-on: https://chromium-review.googlesource.com/891234 Commit-Queue: Louis Romero <lpromero@chromium.org> Reviewed-by:
Louis Romero <lpromero@chromium.org> Cr-Commit-Position: refs/heads/master@{#532435}
-
v8-autoroll authored
Summary of changes available at: https://chromium.googlesource.com/v8/v8/+log/ce789d48..c1717655 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: If666036fd7db804f41946419b5418ac3504f1ee0 Reviewed-on: https://chromium-review.googlesource.com/891238Reviewed-by:
v8 autoroll <v8-autoroll@chromium.org> Commit-Queue: v8 autoroll <v8-autoroll@chromium.org> Cr-Commit-Position: refs/heads/master@{#532434}
-
Peter Wen authored
Update suppressions from android_tools upgrades resulting in unused resources (mostly strings) being left behind after their feature is removed. Also update some suppressions as expected. BUG=799070,806060 TBR=nyquist@chromium.org Change-Id: I30d35320ad20b318a921972529f139b3b3e65453 Reviewed-on: https://chromium-review.googlesource.com/886863 Commit-Queue: Peter Wen <wnwen@chromium.org> Reviewed-by:
Peter Wen <wnwen@chromium.org> Reviewed-by:
Theresa <twellington@chromium.org> Cr-Commit-Position: refs/heads/master@{#532433}
-
Ulan Degenbaev authored
The heap snapshot generator will use ScriptWrappableVisitor to capture all DOM nodes that are reachable from V8. The generator needs human-readable textual representation of a node to show the node in UI. This patch adds NameInHeapSnapshot method in TraceWrapperBase. Derived classes can override it to provide specific names. Bug: 749490 Change-Id: Ieea8fb8c4265d9b3441f36e326b7ada0b012b4e8 Reviewed-on: https://chromium-review.googlesource.com/890180Reviewed-by:
Kentaro Hara <haraken@chromium.org> Reviewed-by:
Michael Lippautz <mlippautz@chromium.org> Commit-Queue: Ulan Degenbaev <ulan@chromium.org> Cr-Commit-Position: refs/heads/master@{#532432}
-
Peter E Conn authored
Bug: 806749 Change-Id: If2867aceffe6c3ea9463eb1f5c36715d981ba97b Reviewed-on: https://chromium-review.googlesource.com/891222Reviewed-by:
Bernhard Bauer <bauerb@chromium.org> Commit-Queue: Peter Conn <peconn@chromium.org> Cr-Commit-Position: refs/heads/master@{#532431}
-
Daniel Nicoara authored
If the resolution is non 720p, the display has a device scale factor that needs to be taken into accounts when scaling touch events. Bug: b/72054342 Test: Poke at screen while in 1080p on https://rbyers.github.io/paint.html Change-Id: Ic69714d0acafedc82995cf85178dacff71e40ae7 Reviewed-on: https://chromium-review.googlesource.com/887291Reviewed-by:
Michael Spang <spang@chromium.org> Reviewed-by:
Sergey Volk <servolk@chromium.org> Commit-Queue: Alex Sakhartchouk <alexst@chromium.org> Cr-Commit-Position: refs/heads/master@{#532430}
-
David 'Digit' Turner authored
Completely remove the ability to call r_brk() at runtime for release builds. Normally, this is controlled at runtime by calling crazy_set_debugger_support(false), which is actually performed by the //base/android/linker/legacy_linker.cc. However, due to yet-unsolved reasons, runtime crashes in r_brk() still happen on Intel-based devices, even though this function is now supposed to never be called. This patch is an ugly hack that removes the ability to call r_brk() entirely from the library's code, to unlock the release. A proper fix would understand what's really going on (e.g. passing of dangling pointers in deferred task callbacks?) and address that accordingly. Unfortunately, the issue is really difficult to reproduce locally. BUG=796938 R=agrieve@chromium.org,pasko@chromium.org,rmcilroy@chromium.org Change-Id: I10a08fe3d551be39b6b9919420224b925b8be9a0 Reviewed-on: https://chromium-review.googlesource.com/889757Reviewed-by:
agrieve <agrieve@chromium.org> Commit-Queue: David Turner <digit@chromium.org> Cr-Commit-Position: refs/heads/master@{#532429}
-
Nico Weber authored
Bug: 177475 R=jochen@chromium.org Change-Id: Id6c4b989d652860b5bdbdb04ace4199ab1e1db0f Reviewed-on: https://chromium-review.googlesource.com/889840Reviewed-by:
Jochen Eisinger <jochen@chromium.org> Cr-Commit-Position: refs/heads/master@{#532428}
-
Peter Wen authored
Using aapt to crunch and then avoiding crunching with aapt2's --no-crunch flag crashes aapt2. Just use aapt2's default crunching. BUG=805668 Change-Id: I61a578eb577069e899cd8a830458d54515fc7d21 Reviewed-on: https://chromium-review.googlesource.com/884064 Commit-Queue: Peter Wen <wnwen@chromium.org> Reviewed-by:
-
Sandra Sun authored
Bug: 762054 Change-Id: If11cd4b3bf8cc24a97e4edf90a7dfbde983c0fb5 Reviewed-on: https://chromium-review.googlesource.com/862971Reviewed-by:
Philip Jägenstedt <foolip@chromium.org> Commit-Queue: Sandra Sun <sunyunjia@chromium.org> Cr-Commit-Position: refs/heads/master@{#532426}
-
Tommy Steimel authored
This CL modifies the SingleWebsitePreferences setup to always show the sound setting even when it's not currently set by showing the default setting as selected. Bug: 796873 Change-Id: Id7dec2bb0e75ee138dae95bbd219c569794d2b09 Reviewed-on: https://chromium-review.googlesource.com/889519Reviewed-by:
Finnur Thorarinsson <finnur@chromium.org> Reviewed-by:
Mounir Lamouri <mlamouri@chromium.org> Commit-Queue: Tommy Steimel <steimel@chromium.org> Cr-Commit-Position: refs/heads/master@{#532425}
-
chrome://snippets-internalsJan Krcal authored
The CL reveals the info whether the last fetch was authenticated or not via chrome://snippets-internals. The CL also does a minor polish of the internals page (only to be able to display the new field in a logical place. Bug: 804809 Cq-Include-Trybots: master.tryserver.chromium.linux:closure_compilation Change-Id: I040ad1b6c223e8ab7c2a0893a820dcc014fbcdba Reviewed-on: https://chromium-review.googlesource.com/880862 Commit-Queue: Jan Krcal <jkrcal@chromium.org> Reviewed-by:
Marc Treib <treib@chromium.org> Cr-Commit-Position: refs/heads/master@{#532424}
-
Jonathan authored
Recent bug fixes have lead to more passing tests. Re-enable them. TBR=kylechar@chromium.org Bug: 760320 Change-Id: Iaf8bb1985bda1de31af30c775180f64bdf3d526e Reviewed-on: https://chromium-review.googlesource.com/860277Reviewed-by:
Jonathan Ross <jonross@chromium.org> Reviewed-by:
kylechar <kylechar@chromium.org> Commit-Queue: Jonathan Ross <jonross@chromium.org> Cr-Commit-Position: refs/heads/master@{#532423}
-
Nico Weber authored
R=jochen@chromium.org Change-Id: I9bfb6786d2fb2ef764c91145c488b50cbf76c487 Reviewed-on: https://chromium-review.googlesource.com/889879Reviewed-by:
-
Yang Guo authored
R=mlippautz@chromium.org Bug: v8:7275 Change-Id: Iacfb9b4b3a4237a445caf8bcc99a1c88ac3feabd Reviewed-on: https://chromium-review.googlesource.com/891226Reviewed-by:
-
Allen Bauer authored
Bug: 803815 Change-Id: I32dbf0ed1736339f3420d113a1a5a6440c7b8542 Reviewed-on: https://chromium-review.googlesource.com/887243Reviewed-by:
Bret Sepulveda <bsep@chromium.org> Commit-Queue: Allen Bauer <kylixrd@chromium.org> Cr-Commit-Position: refs/heads/master@{#532420}
-
Michael Giuffrida authored
discard_metrics_util.cc should include discard_metrics_util.h. Change-Id: I542f831ec61202d210c2aa854bfe7276df1f6f5a Reviewed-on: https://chromium-review.googlesource.com/890305Reviewed-by:
François Doray <fdoray@chromium.org> Commit-Queue: François Doray <fdoray@chromium.org> Cr-Commit-Position: refs/heads/master@{#532419}
-
Nico Weber authored
Fixes a bug where many WebFeature::kScrollToFragment* use counters were overcounted. This CL was uploaded by git cl split. R=dcheng@chromium.org Bug: 177475 Change-Id: I09276f288ca751448639dde1f5dbf4c9894155a9 Reviewed-on: https://chromium-review.googlesource.com/890006Reviewed-by:
Daniel Cheng <dcheng@chromium.org> Cr-Commit-Position: refs/heads/master@{#532418}
-
David Bokan authored
EditingStyle: :AddAbsolutePositioningFromElement is unused. Change-Id: I7c5dfc825ca5453c19e6164dfef04fc3cebdd1c6 Reviewed-on: https://chromium-review.googlesource.com/887735Reviewed-by:
Yoshifumi Inoue <yosin@chromium.org> Commit-Queue: David Bokan <bokan@chromium.org> Cr-Commit-Position: refs/heads/master@{#532417}
-
Anita Woodruff authored
R=peter@chromium.org Change-Id: Ia25e9c21b020dfa524c5d2e8ccbf7ba5bf440dcd Reviewed-on: https://chromium-review.googlesource.com/888525Reviewed-by:
Peter Beverloo <peter@chromium.org> Commit-Queue: Anita Woodruff <awdf@chromium.org> Cr-Commit-Position: refs/heads/master@{#532416}
-
Keishi Hattori authored
Epehemeron iteration done callbacks cleanups the Enqueued flag, so it needs to be called on incremental marking abort. This CL moves them to its own stack so they can be invoked separately. Bug: 757440 Change-Id: I815532db4858c2a81ef810891e6569de8e1de002 Reviewed-on: https://chromium-review.googlesource.com/832546 Commit-Queue: Keishi Hattori <keishi@chromium.org> Reviewed-by:
-
Yuki Shiino authored
This patch removes use of filesystem's VoidCallback. The motivation is the same as https://crrev.com/844307 . Bug: 778580 Change-Id: I2f57919461a31a4f64af287b40e0a3a6d1f242cb Reviewed-on: https://chromium-review.googlesource.com/888234 Commit-Queue: Yuki Shiino <yukishiino@chromium.org> Reviewed-by:
Hitoshi Yoshida <peria@chromium.org> Cr-Commit-Position: refs/heads/master@{#532414}
-
Koji Ishii authored
Following bot results are included. 3356 3357 3358 3359 3362 3365 3368 3369 25 lines were removed and 29 lines were deflaked by consecutive results since 3334. TBR=eae@chromium.org NOTRY=true Bug: 591099 Cq-Include-Trybots: master.tryserver.chromium.linux:linux_layout_tests_layout_ng Change-Id: Ic4879340dd6c0f7f555f282bf2bfc18a100c82d4 Reviewed-on: https://chromium-review.googlesource.com/890679 Commit-Queue: Koji Ishii <kojii@chromium.org> Reviewed-by:
Koji Ishii <kojii@chromium.org> Cr-Commit-Position: refs/heads/master@{#532413}
-
David Bokan authored
The stacktrace in the bug revealed a path where we don't check that the LocalFrameView isn't null. No test since I couldn't reproduce the crash. Bug: 805420 Change-Id: Ief57e784e765efadb18b5c9c040e7b5a40ee904c Reviewed-on: https://chromium-review.googlesource.com/887826Reviewed-by:
Dave Tapuska <dtapuska@chromium.org> Commit-Queue: David Bokan <bokan@chromium.org> Cr-Commit-Position: refs/heads/master@{#532412}
-
Elly Fong-Jones authored
This change: 1) Resurrects the 10.6/10.7/10.8 deprecation strings and updates them to 10.9; 2) Adds a feature "Enable109ObsoleteInfobar"; 3) Shows the obsolete system infobar on 10.9 systems when that feature is enabled; 4) Adds a *placeholder* chrome::kMac10_9_ObsoleteURL link for the "learn more" link in the infobar. Bug: 806071 Change-Id: I48ab95e50fd2be7438650bc7f7cbb35fd2c2b02b Reviewed-on: https://chromium-review.googlesource.com/889020 Commit-Queue: Elly Fong-Jones <ellyjones@chromium.org> Reviewed-by:
Mark Mentovai <mark@chromium.org> Cr-Commit-Position: refs/heads/master@{#532411}
-
Sunny Sachanandani authored
We want to merge various smoothness page sets into desktop and mobile page sets for the new rendering benchmark. Since the same urls are duplicated in different page sets e.g. same page with scroll and pinch-zoom or same page with and without gpu raster, it's necessary to have unique names for each instance. This CL allows specifying a name when instantiating these pages. BUG=760553 Change-Id: I7fe789fbda625c6700e089433e75c07cad7d7db8 Reviewed-on: https://chromium-review.googlesource.com/885644Reviewed-by:
Ned Nguyen <nednguyen@google.com> Commit-Queue: Ned Nguyen <nednguyen@google.com> Cr-Commit-Position: refs/heads/master@{#532410}
-
Alexander Timin authored
Split out input task queue from the compositor task queue and use it in content/renderer/input's MainThreadEventQueue. For now scheduler applies the exact same policies to the new queue, so this patch does not affect scheduling policies. It is planned in the future to run input task queue with high priority unconditionally. R=skyostil@chromium.org,haraken@chromium.org,dtapuska@chromium.org BUG=806271 Change-Id: I73204187131d01b936a0a5d5052b8969a62205eb Reviewed-on: https://chromium-review.googlesource.com/888521Reviewed-by:
-
v8-autoroll authored
Summary of changes available at: https://chromium.googlesource.com/v8/v8/+log/b4e468e1..ce789d48 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: Id4572c982100e3f97b43e13f4a8f59f0ae12e6a6 Reviewed-on: https://chromium-review.googlesource.com/891138Reviewed-by:
-
vitaliii authored
Disable ProcessManagerBrowserTest.NestedURLNavigationsToExtensionBlocked because it timeouts constantly on Win. TBR=rdevlin.cronin@chromium.org Bug: 806684 Change-Id: Ic86b6219db299489ec6392b2fd2fe885904bb3d9 Reviewed-on: https://chromium-review.googlesource.com/891022 Commit-Queue: vitaliii <vitaliii@chromium.org> Reviewed-by:
vitaliii <vitaliii@chromium.org> Cr-Commit-Position: refs/heads/master@{#532407}
-
Yuki Shiino authored
This patch removes use of filesystem's MetadataCallback. The motivation is the same as https://crrev.com/844307 . Bug: 778580 Change-Id: I75dda3272deb5cd641aa01da638b9ea5fee98bff Reviewed-on: https://chromium-review.googlesource.com/886921 Commit-Queue: Yuki Shiino <yukishiino@chromium.org> Reviewed-by:
-
Christian Dullweber authored
Add tracing for methods in BrowsingData[Counter]Bridge and each browsing data counter. This will allow easier debugging of performance issues with the Clear Browsing Data dialog. Bug: 804839 Change-Id: I971d08edea77fee4e76e48b901516253e46fcb33 Reviewed-on: https://chromium-review.googlesource.com/883530Reviewed-by:
Martin Šrámek <msramek@chromium.org> Reviewed-by:
-
Anita Woodruff authored
- No longer necessary to wait for NotificationDisplayService mutation now that we are back to waiting for the 'show' event in this test, since f5ae28 Change-Id: I19506c1a8cf1a1f040919a20be131986fff7df44 Reviewed-on: https://chromium-review.googlesource.com/888819Reviewed-by:
-
Greg Thompson authored
BUG=None R=ljusten@chromium.org Change-Id: I0c0f6e061c5d81e0d823d155b84dbf887867bbde Reviewed-on: https://chromium-review.googlesource.com/889499Reviewed-by:
Lutz Justen <ljusten@chromium.org> Commit-Queue: Greg Thompson <grt@chromium.org> Cr-Commit-Position: refs/heads/master@{#532403}
-
Gabriel Charette authored
Add a developer hint to an otherwise cryptic error when accidently registering a second RunLoop::Delegate on a thread. While Runloop::Delegate isn't aware of MessageLoop/ScopedTaskEnvironment: I find useful to add developer hints about the typical developer mistake a DCHECK catches. This avoids multiple developers wasting time figuring out the same thing. Started doing this in a few places in //base and this is another instance recently surfaced by chrisha @ https://groups.google.com/a/google.com/d/topic/lucky-luke-dev/9_eZ6pnvo7Y/discussion R=chrisha@chromium.org Bug: Cryptic error when creating a ScopedTaskEnvironment over an existing MessageLoop. Change-Id: I427e0400b2a2b91ab9b175fa2427014bc8fc4406 Reviewed-on: https://chromium-review.googlesource.com/890174 Commit-Queue: Gabriel Charette <gab@chromium.org> Reviewed-by:
Robert Liao <robliao@chromium.org> Cr-Commit-Position: refs/heads/master@{#532402}
-
vitaliii authored
This reverts commit 4db68ed9. Reason for revert: Seems to cause failures on Mac ASan 64 Tests (see crbug.com/806712). Original change's description: > [Variations] Avoid duplication of the safe seed value stored in prefs. > > When the safe seed and the latest seed have identical contents, use a sentinel > value to avoid storing a duplicate value for the two seeds. This is purely a > storage space optimization; no difference in observed functionality is expected. > > R=asvitkine@chromium.org > > Bug: 727984 > Change-Id: Ica641a479228b0cdfcc4af2ec04ddfc54c36c5eb > Reviewed-on: https://chromium-review.googlesource.com/880482 > Commit-Queue: Ilya Sherman <isherman@chromium.org> > Reviewed-by: Alexei Svitkine <asvitkine@chromium.org> > Cr-Commit-Position: refs/heads/master@{#532027} TBR=isherman@chromium.org,asvitkine@chromium.org # Not skipping CQ checks because original CL landed > 1 day ago. Bug: 727984, 806712 Change-Id: Ic5de33c61bb2acf88d7445ded4f0a1743ab7f456 Reviewed-on: https://chromium-review.googlesource.com/891119Reviewed-by:
-
Javier Fernandez authored
The CSS WG has resolved that it doesn't make sense to define Baseline Content-Alignment in the inline/main axis. https://github.com/w3c/csswg-drafts/issues/1184 The spec has been updated so that <baseline-postion> is not valid for the 'justify-content' property's syntax. This CL updates our parsing logic to match the new spec, including the parsing logic of the place-content shorthand. Additionally, this CL updates the computed value of the <baseline-position> values to match other browsers. We don't implement support for <baseline-position> values in the Content Distribution properties (align-content and justify-content), so I don't expect this change to break content of sites using the CSS Box Alignment feature. Bug: 802248, 803275 Change-Id: I0d3b1b7e7dd28552ee7e131bfe09ef7198565283 Reviewed-on: https://chromium-review.googlesource.com/882362Reviewed-by:
Darren Shen <shend@chromium.org> Commit-Queue: Javier Fernandez <jfernandez@igalia.com> Cr-Commit-Position: refs/heads/master@{#532400}
-
Keishi Hattori authored
In anticipation for incremental marking, disable promptly free for weak collection backings. This is required to avoid callbacks to objects that can be destroyed during the mutator between incremental marking. This CL: - Disable prompty free for weak hash table backings - Adds CheckObjectNotInCallbackStacks which checks that an object doesn't have any callback registered Bug: 757440 Change-Id: Ibeffed9e5a2f15a1b88c934f68e5d86f54c2e2c9 Reviewed-on: https://chromium-review.googlesource.com/836299 Commit-Queue: Keishi Hattori <keishi@chromium.org> Reviewed-by:
-
Christian Dullweber authored
Remove the -tabs suffix from CBD code as the old dialog has been removed and the new dialog isn't special anymore. Also remove some footer related code as the footer has been removed. Bug: 800184 Cq-Include-Trybots: master.tryserver.chromium.linux:closure_compilation Change-Id: I937e2133feb192eff9f043c5bc060d8296806070 Reviewed-on: https://chromium-review.googlesource.com/873911 Commit-Queue: Christian Dullweber <dullweber@chromium.org> Reviewed-by:
Dave Schuyler <dschuyler@chromium.org> Cr-Commit-Position: refs/heads/master@{#532398}
-
