- 19 Nov, 2018 40 commits
-
-
Marc Treib authored
The retry task was plumbed back from SyncSchedulerImpl through SyncManagerImpl, SyncBackendHostCore, SyncBackendHostImpl to DataTypeManagerImpl, where it did nothing. Bug: 867425 Change-Id: I1788573635da7c12062ed8aea094b74ef8349b0d Reviewed-on: https://chromium-review.googlesource.com/c/1341532Reviewed-by:
Mohamed Amir Yosef <mamir@chromium.org> Commit-Queue: Marc Treib <treib@chromium.org> Cr-Commit-Position: refs/heads/master@{#609235}
-
Mikel Astiz authored
The SyncableService (SessionsSyncManager), the directory datatype controller and all abstractions required for accomodating two implementations are removed, together with the corresponding about flag and feature toggle (kill switch). Sync integration tests are updated because they also exercised the legacy codepath. TBR=rohitrao@chromium.org,pkasting@chromium.org Bug: 895455 Change-Id: Ifaca1696e97f88b70b0c61ad89fed2372b239e46 Reviewed-on: https://chromium-review.googlesource.com/c/1340814Reviewed-by:
Mikel Astiz <mastiz@chromium.org> Reviewed-by:
Marc Treib <treib@chromium.org> Commit-Queue: Mikel Astiz <mastiz@chromium.org> Cr-Commit-Position: refs/heads/master@{#609234}
-
Clemens Arbesser authored
Bug: 806868 Change-Id: Iee8f1ff42f8c76b7553ffce55d51ca5e54afd95b Reviewed-on: https://chromium-review.googlesource.com/c/1340818Reviewed-by:
Mathias Carlen <mcarlen@chromium.org> Commit-Queue: Clemens Arbesser <arbesser@google.com> Cr-Commit-Position: refs/heads/master@{#609233}
-
Maxim Kolosovskiy authored
This reverts commit 8fcafe0e. Reason for revert: the CL caused test failure https://bugs.chromium.org/p/chromium/issues/detail?id=906577 Original change's description: > Add tests for sharing and managing items in team drives. > > Bug: 903637 > Change-Id: Ic9f49f7de59210e400a578060eb611fafebe193f > Reviewed-on: https://chromium-review.googlesource.com/c/1341287 > Commit-Queue: Sam McNally <sammc@chromium.org> > Reviewed-by: Noel Gordon <noel@chromium.org> > Cr-Commit-Position: refs/heads/master@{#609191} TBR=noel@chromium.org,sammc@chromium.org Change-Id: I6385ca389f4f2e8807ee6a34c4a0a27744c5ac39 No-Presubmit: true No-Tree-Checks: true No-Try: true Bug: 903637 Reviewed-on: https://chromium-review.googlesource.com/c/1341516Reviewed-by:
Maxim Kolosovskiy <kolos@chromium.org> Commit-Queue: Maxim Kolosovskiy <kolos@chromium.org> Cr-Commit-Position: refs/heads/master@{#609232}
-
Kent Tamura authored
- Introduce empty ElementInternals interface - Add HTMLElement.prototype.attachInternals() - Add a flag to ElementRareData about ElementInternals attachment - Promote "ElementInternals" runtime flag to "test" All the change is behind the runtime flag. Some test cases in external/wpt/custom-elements/CustomElementRegistry.html fail by this CL because this CL enables GET() operation for "disabledFeatures", and these test cases doesn't expect it. We should update the testcases later. Bug: 905922 Bug: https://github.com/w3c/webcomponents/issues/758 Change-Id: I7b3dbfd1d422c7eae5c7a6e01ddea50a00134a6e Reviewed-on: https://chromium-review.googlesource.com/c/1341734Reviewed-by:
Hayato Ito <hayato@chromium.org> Commit-Queue: Kent Tamura <tkent@chromium.org> Cr-Commit-Position: refs/heads/master@{#609231}
-
Friedrich Horschig authored
This CL introduces the new style of the keyboard accessory (clean white) to highlight is as part of the Chrome UI. It's also the base for IPH which will explicitly state the origin of suggestions and actions. Because the new design uses autofill suggestions as integral part, it is guarded by the #autofill-keyboard-accessory-view flag which isn't used in other way anymore. To keep the old implementation until the new design is fully approved and fleshed out, the KeyboardAccessoryModernView* classes extend the existing classes and replace them if the appropriate flags are set. For screenshots highlighting the differences, see: go/keyboard-accessory-v2 Bug: 853772, 876750 Change-Id: I44718a4e2269d17b002d7dc890784a847ed840e7 Reviewed-on: https://chromium-review.googlesource.com/c/1338087Reviewed-by:
Vadym Doroshenko <dvadym@chromium.org> Reviewed-by:
Theresa <twellington@chromium.org> Commit-Queue: Friedrich Horschig [CET] <fhorschig@chromium.org> Cr-Commit-Position: refs/heads/master@{#609230}
-
Christian Dullweber authored
Add tests similar to BrowsingDataRemoverBrowserTest to Android. They verify that cookies and other data is counted and deleted correctly by Clear Browsing Data. To perform asynchronous calls to Javascript, this adds support for DomAutomationController.send on Android. Bug: 905612, 562487 Change-Id: I2c5d3929cf4e1eed704b21b55ad9988b5a09c53e Reviewed-on: https://chromium-review.googlesource.com/c/1339864 Commit-Queue: Christian Dullweber <dullweber@chromium.org> Reviewed-by:
David Trainor <dtrainor@chromium.org> Cr-Commit-Position: refs/heads/master@{#609229}
-
Sam McNally authored
When adjusting the search parameters, the data model briefly has no entries. Updating the image-dominant calculation during this interval causes the image-dominant to be disabled, causing a flicker of the file names if the contents is image-dominant. Bug: 483718 Change-Id: I099a013e5f05c36902be67aeffc40edbc718be26 Reviewed-on: https://chromium-review.googlesource.com/c/1341736 Commit-Queue: Sam McNally <sammc@chromium.org> Reviewed-by:
Luciano Pacheco <lucmult@chromium.org> Cr-Commit-Position: refs/heads/master@{#609228}
-
chromium-autoroll authored
https://skia.googlesource.com/skia.git/+log/01979132133e..e7b1a13a1e2a git log 01979132133e..e7b1a13a1e2a --date=short --no-merges --format='%ad %ae %s' 2018-11-19 skia-autoroll@skia-public.iam.gserviceaccount.com Roll third_party/externals/angle2 1c6b26a70f93..fde74c078f77 (1 commits) Created with: gclient setdep -r src/third_party/skia@e7b1a13a1e2a The AutoRoll server is located here: https://autoroll.skia.org/r/skia-autoroll 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=luci.chromium.try:android_optional_gpu_tests_rel;luci.chromium.try:linux-chromeos-compile-dbg;luci.chromium.try:linux_optional_gpu_tests_rel;luci.chromium.try:mac_optional_gpu_tests_rel;luci.chromium.try:win_optional_gpu_tests_rel;master.tryserver.blink:linux_trusty_blink_rel TBR=csmartdalton@chromium.org Change-Id: I46c445d15a8701b0141cb60189c43f22c3b3cb3b Reviewed-on: https://chromium-review.googlesource.com/c/1341567Reviewed-by:
chromium-autoroll <chromium-autoroll@skia-public.iam.gserviceaccount.com> Commit-Queue: chromium-autoroll <chromium-autoroll@skia-public.iam.gserviceaccount.com> Cr-Commit-Position: refs/heads/master@{#609227}
-
Camillo Bruni authored
This CL was created automatically with tools/perf/update_wpr script R=perezju@chromium.org Bug: 878390 Change-Id: Ifc106e9166ef46fbb04a87dee98af72a55da31bb Reviewed-on: https://chromium-review.googlesource.com/c/1329925Reviewed-by:
Juan Antonio Navarro Pérez <perezju@chromium.org> Commit-Queue: Camillo Bruni <cbruni@chromium.org> Cr-Commit-Position: refs/heads/master@{#609226}
-
Vaclav Brozek authored
The old FormData -> PasswordForm parser runs in renderer process, before the data about forms is passed on to the browser process. The new parser runs in browser, after it receives the data from the renderer. For legacy reasons, the FormData (parsing input) is enclosed inside PasswordForm (output) during the inter-process transport. As a result, if the old parser fails to parse completely, the new one has no chance to run at all, because there is no valid PasswordForm to encapsulate its input FormData during the inter-process transport. The important case when this matters is if there are password fields present, but none of them is enabled. The old parser gives up, the new does not care about enabled vs. disabled, so can parse. This CL allows the old parser to produce a minimal PasswordForm if only disabled password fields are present, to allow transporting the FormData to the new parser. In PasswordManager, all these minimal PasswordForms are managed by at most one PasswordFormManager and only used as a fallback, so the functionality of Chrome under the old parser is not affected. Bug: 904908 Change-Id: Idaf569d39884375dc5942dea4349aeec41355c76 Reviewed-on: https://chromium-review.googlesource.com/c/1337497 Commit-Queue: Vaclav Brozek <vabr@chromium.org> Reviewed-by:
-
Maxim Kolosovskiy authored
This reverts commit aee1ee31. Reason for revert: Suspect this CL caused https://bugs.chromium.org/p/monorail/issues/detail?id=4576 Original change's description: > Make EndOfSentence stop at block boundaries > > This patch stops EndOfSentence from crossing block boundaries, since > block boundaries are also sentence boundaries. > > Bug: 778507 > Change-Id: I19ec97de1bbaac50db4c55b38ffb158339003993 > Reviewed-on: https://chromium-review.googlesource.com/c/1337432 > Reviewed-by: Yoichi Osato <yoichio@chromium.org> > Reviewed-by: Yoshifumi Inoue <yosin@chromium.org> > Commit-Queue: Xiaocheng Hu <xiaochengh@chromium.org> > Cr-Commit-Position: refs/heads/master@{#608850} TBR=yosin@chromium.org,yoichio@chromium.org,xiaochengh@chromium.org # Not skipping CQ checks because original CL landed > 1 day ago. Bug: 778507 Change-Id: I7d48146bd7fb27c34427f543ad3b54443ae63715 Reviewed-on: https://chromium-review.googlesource.com/c/1340815Reviewed-by:
-
Ioana Pandele authored
This change separates the current text of the infobar into title and body. The latter contains a link pointing to where the user can find the saved generated password. Screenshot on the bug. Bug: 903257 Change-Id: Iadbfb68298b338d990c4be0b0ab3d11007b4b125 Reviewed-on: https://chromium-review.googlesource.com/c/1156586Reviewed-by:
Rouslan Solomakhin <rouslan@chromium.org> Commit-Queue: Ioana Pandele <ioanap@chromium.org> Cr-Commit-Position: refs/heads/master@{#609223}
-
Justin Cohen authored
Change-Id: Idd933d2bfcff3e136c4b2787a80d70b562068ceb Reviewed-on: https://chromium-review.googlesource.com/c/1341206 Commit-Queue: Gauthier Ambard <gambard@chromium.org> Reviewed-by:
Gauthier Ambard <gambard@chromium.org> Cr-Commit-Position: refs/heads/master@{#609222}
-
Vaclav Brozek authored
The PasswordForm.FillOnLoad UKM aims at measuring the change in behaviour of Chrome with the new password form parser, if an explicit condition for filling on load is changed from A to B. It turned out that even if A is the explicit condition, B still can break fill on load. So the change is not so much A -> B as A && B -> B. This means that the metric's bucket recording the state when A is true and B false needs to be merged into the bucket when A == B, because in both cases the behaviour does not change. Bug: 895781 Change-Id: Ic3ab14d9a1e43c0312580d2dfe5bff34f2cdd20a Reviewed-on: https://chromium-review.googlesource.com/c/1340274Reviewed-by:
-
Hans Wennborg authored
This is part of cleaning up for a new Clang warning that's currently disabled: In file included from ../../ash/app_list/home_launcher_gesture_handler.cc:19: ../../ash/wm/window_transient_descendant_iterator.h:69:43: error: explicitly defaulted copy assignment operator is implicitly deleted [-Werror,-Wdefaulted-function-deleted] WindowTransientDescendantIteratorRange& operator=( ^ ../../ash/wm/window_transient_descendant_iterator.h:72:37: note: copy assignment operator of 'WindowTransientDescendantIteratorRange' is implicitly deleted because field 'begin_' has an inaccessible copy assignment operator WindowTransientDescendantIterator begin_; ^ Bug: 890307 Change-Id: Ic47bc989f9fe26e1387f9a6a8c00f19fcf6b91ce Reviewed-on: https://chromium-review.googlesource.com/c/1340331 Commit-Queue: Hans Wennborg <hans@chromium.org> Reviewed-by:Robert Flack <flackr@chromium.org> Cr-Commit-Position: refs/heads/master@{#609220}
-
Yuki Shiino authored
The return type of OnBeforeUnloadEventHandler is DOMString?, so we should use V8StringResource<kTreatNullAsNullString> instead of V8String<kDefaultMode>. Change-Id: I6df9581cf5dd01ae8215d773c95a3e9dc6a78e08 Reviewed-on: https://chromium-review.googlesource.com/c/1341779Reviewed-by:
Hitoshi Yoshida <peria@chromium.org> Reviewed-by:
Kentaro Hara <haraken@chromium.org> Commit-Queue: Yuki Shiino <yukishiino@chromium.org> Cr-Commit-Position: refs/heads/master@{#609219}
-
Daniel Bratell authored
Change-Id: I88b641bf356044ca94552273081af22c0fde1795 Reviewed-on: https://chromium-review.googlesource.com/c/1341530Reviewed-by:
Fredrik Söderquist <fs@opera.com> Commit-Queue: Daniel Bratell <bratell@opera.com> Cr-Commit-Position: refs/heads/master@{#609218}
-
Yuki Shiino authored
This patch fixes a regression at https://crrev.com/c/1215512 , which doesn't run 'beforeunload' event listeners when one of the related execution contexts is paused. This patch adds an exceptional handling for 'beforeunload' event so that 'beforeunload' event listeners run regardless of pause. Bug: 903297 Change-Id: I92bb24a8fd16ee9f81a5082f5c561c97c1dd5348 Reviewed-on: https://chromium-review.googlesource.com/c/1333548 Commit-Queue: Yuki Shiino <yukishiino@chromium.org> Reviewed-by:
-
Mario Sanchez Prada authored
This will remove the need to know about specific instances of SigninManager, AccountTrackerService, ProfileOAuth2TokenService and GaiaCookieManagerService from the calling places for those methods. Also move the SigninManagerForTest definition in identity_test_environment.h one level up, to allow sharing it among |IdentityManagerDependenciesOwner| and |IdentityTestEnvironment| without relying on such definition being also available from identity_test_utils.h (from where it's now being removed due to no longer being required by those test utilities). Bug: 904404 Change-Id: If96b0c7dea763030dc7372d58922f82a31eb5733 Reviewed-on: https://chromium-review.googlesource.com/c/1336127 Commit-Queue: Mario Sanchez Prada <mario@igalia.com> Reviewed-by:
Toni Baržić <tbarzic@chromium.org> Reviewed-by:
David Roger <droger@chromium.org> Reviewed-by:
Peter Kasting <pkasting@chromium.org> Reviewed-by:
Colin Blundell <blundell@chromium.org> Cr-Commit-Position: refs/heads/master@{#609216}
-
chromium-internal-autoroll authored
https://chrome-internal.googlesource.com/chrome/src-internal.git/+log/ecb09be23157..7e385fd60000 Created with: gclient setdep -r src-internal@7e385fd60000 The AutoRoll server is located here: https://autoroll-internal.skia.org/r/src-internal-chromium-autoroll 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=mmoss@chromium.org Change-Id: Ifdfe2ed136d3a4440cc92cee9b8706b50e8c77ab Reviewed-on: https://chromium-review.googlesource.com/c/1341566Reviewed-by:
chromium-internal-autoroll <chromium-internal-autoroll@skia-corp.google.com.iam.gserviceaccount.com> Commit-Queue: chromium-internal-autoroll <chromium-internal-autoroll@skia-corp.google.com.iam.gserviceaccount.com> Cr-Commit-Position: refs/heads/master@{#609215}
-
Julian Pastarmov authored
Some ph tags were incorrectly using the $1 placeholder and were replaced with the string Google Chrome instead of their content. BUG=none Change-Id: Idd9540861a44c124ad8ac5e9ccca6e588e98b74f Reviewed-on: https://chromium-review.googlesource.com/c/1337624Reviewed-by:
Pavol Marko <pmarko@chromium.org> Commit-Queue: Julian Pastarmov <pastarmovj@chromium.org> Cr-Commit-Position: refs/heads/master@{#609214}
-
Yutaka Hirano authored
It was introduced for out-of-blink CORS, but the check doesn't work with extensions. Until it's fixes let's rely on the renderer side check and disable the check. Bug: 895999 Change-Id: I59aaafcb9185e1937372cf8967ef45d3728dff64 Reviewed-on: https://chromium-review.googlesource.com/c/1335078 Commit-Queue: Yutaka Hirano <yhirano@chromium.org> Reviewed-by:
John Abd-El-Malek <jam@chromium.org> Reviewed-by:
Takashi Toyoshima <toyoshim@chromium.org> Reviewed-by:
Marijn Kruisselbrink <mek@chromium.org> Cr-Commit-Position: refs/heads/master@{#609213}
-
Hayato Ito authored
Current implementation of flat tree traversal is not cheap when traversing host's children. See http://crbug.com/906494 for details. There was an attempt to make it faster in v1, https://crrev.com/c/1328622, however, this idea was already implemented in v0, and this attempt actually caused a perf regression in some cases (https://crbug.com/905197). This CL makes flat tree traversal for host's children much faster in a different approach. The basic idea is: 1. Let NodeRareData has additional pointers, as FlatTreeNodeData, and set pointers at the timing of slot assignment recalc. 2. At flat tree traversal, use these pointers to skip hashmap lookup. In this CL, |next| and |previous| pointers for assigned nodes are used in flat tree traversal. Thanks to that, flat tree traversal no longer lookup node-to-index hashmap, nor do a liner search there. These additional pointers are only set for host's children. There shouldn't be additional cost for other cases. In a follow-up CL, I'll use FlatTreeNodeData's |assigned_slot| to make flat tree traversal and other operations much faster again. Since this task might be complicated, that can be done in another CL. BUG: 906494 Change-Id: I617da70371d6733050b7dc754d3b40c88c090c79 Reviewed-on: https://chromium-review.googlesource.com/c/1337225 Commit-Queue: Hayato Ito <hayato@chromium.org> Reviewed-by:
Kent Tamura <tkent@chromium.org> Cr-Commit-Position: refs/heads/master@{#609212}
-
Dean Liao authored
Use VideoFrameLayout to store negotiated input/output format, coded_size and buffer layout obtained from S_FMT command. BUG=b:110815424,b:73752373 TEST=Run VEA VDA unittest on devices (peach_pit and elm) VEA: video_encode_accelerator_unittest \ --test_stream_data=bear-320x180.yuv:320:180:1:bear.mp4:100000:30 \ --disable_flush --single-process-tests -v=0 VDA: video_decode_accelerator_unittest \ --test_stream_data=/usr/local/video/test-25fps.h264:320:240:250:258:\ 35:150:1 -v=0 --disable_flush --single-process-tests --ozone-platform=gbm Change-Id: If708cb66c5f33cb16ad8599339999a3aac9fcd47 Reviewed-on: https://chromium-review.googlesource.com/c/1340440 Commit-Queue: Shuo-Peng Liao <deanliao@google.com> Reviewed-by:
Alexandre Courbot <acourbot@chromium.org> Reviewed-by:
Hirokazu Honda <hiroh@chromium.org> Cr-Commit-Position: refs/heads/master@{#609211}
-
Kent Tamura authored
Implement the second paragraph of [1]. - Rename HTMLFormElement::ConstructFormDataSet() to ConstructEntryList() The algorithm name in the specification was renamed. - Add HTMLFormElement::is_constructing_entry_list_. It is set to 'true' in ConstructFormEntryList(). - FormData::Create() takes additional ExceptionState& argument The behavior change by this CL is behind a runtime flag. [1] https://docs.google.com/document/d/1JO8puctCSpW-ZYGU8lF-h4FWRIDQNDVexzHoOQ2iQmY/edit?pli=1#heading=h.veshh4ug726u Bug: 825684 Change-Id: I28fc29c2b8e969a5fbe1b6f955e9d8b65df0d8ec Reviewed-on: https://chromium-review.googlesource.com/c/1340079Reviewed-by:
Yoshifumi Inoue <yosin@chromium.org> Commit-Queue: Kent Tamura <tkent@chromium.org> Cr-Commit-Position: refs/heads/master@{#609210}
-
Vlad Tsyrklevich authored
fdc1883a13ac fuchsia: Disable ProcessReaderFuchsia.ChildThreads, flaking 656fa55c74a8 Refactor ProcessReaderWin to use ProcessMemoryWin 1c4c1277e870 Add a ProcessSnapshot::Memory method ad788280e4df fuchsia: Disable HTTPS tests to avoid (unavailable) dependency on BoringSSL f42a579b8c84 Add ProcessSnapshotMinidump::CustomMinidumpStreams() 083fd52cc3c1 linux: Fix build failure on ARM/MIPS Change-Id: If912a1d3a287ebbf9f34d77f84dc8ff517456383 Reviewed-on: https://chromium-review.googlesource.com/c/1341242Reviewed-by:Mark Mentovai <mark@chromium.org> Commit-Queue: Vlad Tsyrklevich <vtsyrklevich@chromium.org> Cr-Commit-Position: refs/heads/master@{#609209}
-
Yutaka Hirano authored
They are no longer leaky. Bug: 672740 Change-Id: I327d3f5c073008ec092dc3f94bb00701d05cd183 Reviewed-on: https://chromium-review.googlesource.com/c/1341735Reviewed-by:
Hiroki Nakagawa <nhiroki@chromium.org> Commit-Queue: Yutaka Hirano <yhirano@chromium.org> Cr-Commit-Position: refs/heads/master@{#609208}
-
David 'Digit' Turner authored
This patch fixed the base::trace_event::TraceEvent::Reset() method to ensure that it never creates dangling pointers. This can happen in the following case: - TraceEvent::Initialize() is called on an instance, with TRACE_EVENT_FLAG_COPY set in the |flags| argument. This will copy argument names, copyable string values, as well as the name and scope into a single heap allocated buffer backed by |parameter_copy_storage_|, and will also adjust all internal pointer fields to point to it. - TraceEvent::Reset() is called on the same instance, this frees the storage area, but before this CL did not update the internal pointers, who were now dangling into heap-free memory! - Later, some code will iterate over the arguments with a loop like: for (int i = 0; i < kTraceMaxNumArgs && arg_names_[i] != nullptr; ++i) { ... } The assumption being that an arg_names_[i] value of nullptr indicates the end of list. Unfortunately, in the case above, this will read completely invalid values from memory. + Fix TraceEvent::MoveFrom() to call other->Reset() to ensure that the source instance is left in consistent state. I believe this is the source of flakiness on many tests related to TraceEvent, and hope this fixes it. BUG=905624,899813 R=oystene@chromium.org,primiano@chromium.org,alexilin@chromium.org,pkl@chromium.org Change-Id: I63cbadc728130cddc68b8c92b28e1e3f584793f4 Reviewed-on: https://chromium-review.googlesource.com/c/1340308 Commit-Queue: David Turner <digit@chromium.org> Reviewed-by:Peter Lee <pkl@chromium.org> Reviewed-by:
oysteine <oysteine@chromium.org> Cr-Commit-Position: refs/heads/master@{#609207}
-
chromium-internal-autoroll authored
https://chrome-internal.googlesource.com/chrome/src-internal.git/+log/6340200508f8..ecb09be23157 Created with: gclient setdep -r src-internal@ecb09be23157 The AutoRoll server is located here: https://autoroll-internal.skia.org/r/src-internal-chromium-autoroll 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=mmoss@chromium.org Change-Id: Ib075e2ef3d8f0d384cc1d1c4c2d1265cf9bb0124 Reviewed-on: https://chromium-review.googlesource.com/c/1341564Reviewed-by:
-
Koji Ishii authored
Following bot results are included. 11841 11844 11850 11855 11857 11861 11868 11873 11876 11877 11879 11885 11886 11890 11903 11913 11919 11921 11923 11925 11935 0 lines were removed and 3 lines were deflaked by consecutive results since 11877. TBR=eae@chromium.org, mstensho@chromium.org NOTRY=true Bug: 591099 Change-Id: I42b2db12e91066fb87b5d8c81b4a8b18d6bb3a7f Reviewed-on: https://chromium-review.googlesource.com/c/1337213 Commit-Queue: Koji Ishii <kojii@chromium.org> Reviewed-by:
Koji Ishii <kojii@chromium.org> Cr-Commit-Position: refs/heads/master@{#609205}
-
Christopher Lam authored
This CL adds a WebUI for App Management that replaces chrome://apps when the feature NewAppManagement is enabled. This involves: - adding the feature - adding a new MojoWebUIController - adding a mojom, and its C++ handler - adding a skeleton HTML and Polymer 'app' element - adding a BrowserProxy which connects the Renderer JS to the backend C++. Bug: 906508 Change-Id: Iac5be9ce82dc2cb0b1c77301ca43ccf7a99f318f Reviewed-on: https://chromium-review.googlesource.com/c/1309380 Commit-Queue: calamity <calamity@chromium.org> Reviewed-by:
Dominick Ng <dominickn@chromium.org> Cr-Commit-Position: refs/heads/master@{#609204}
-
Tetsui Ohkubo authored
This CL implements animation for clearing all Notifications. This CL does not accurately implements the animation as the spec. For the UX spec, see the bug. TEST=UnifiedMessageListViewTest BUG=897915 Change-Id: I750b3a3e9d86706bef7c9cd13474dbff986486e9 Reviewed-on: https://chromium-review.googlesource.com/c/1312176Reviewed-by:
Yoshiki Iguchi <yoshiki@chromium.org> Commit-Queue: Tetsui Ohkubo <tetsui@chromium.org> Cr-Commit-Position: refs/heads/master@{#609203}
-
Giovanni Ortuño Urquidi authored
Un-capitalizes all instances of "radio" to make them consistent. TBR=dominickn@chromium.org Bug: 870192 Change-Id: Ibf38e8eae85d2ae05a2f82788193fec880f6a46a Reviewed-on: https://chromium-review.googlesource.com/c/1341773Reviewed-by:
Giovanni Ortuño Urquidi <ortuno@chromium.org> Commit-Queue: Giovanni Ortuño Urquidi <ortuno@chromium.org> Cr-Commit-Position: refs/heads/master@{#609202}
-
Dean Liao authored
We used to get V4L2ImageProcessor instance from constructor and perform initialization later. And Initialize() set several data members which are not changed after Initialize(). With factory method, Create(), we can perform Initialize() before constructing V4L2ImageProcessor instance. And those data members set by Initialize() can be const-qualified. This commit is a leading commit for using VideoFrameLayout to encapsulate input/output format and size. Because VideoFrameLayout is a const object, the input/output VideoFrameLayout variables for creating V4L2ImageProcessor should be determined before calling constructor. Thus factory method becomes a natural design choice. BUG=b:110815424,b:73752373 TEST=Run VEA VDA unittest on devices (peach_pit and elm) VEA: video_encode_accelerator_unittest \ --test_stream_data=bear-320x180.yuv:320:180:1:bear.mp4:100000:30 \ --disable_flush --single-process-tests -v=0 VDA: video_decode_accelerator_unittest \ --test_stream_data=/usr/local/video/test-25fps.h264:320:240:250:258:\ 35:150:1 -v=0 --disable_flush --single-process-tests --ozone-platform=gbm Change-Id: I6909b2c1dbe5eda5845681a51af9e3d1c88d89ec Reviewed-on: https://chromium-review.googlesource.com/c/1339084 Commit-Queue: Shuo-Peng Liao <deanliao@google.com> Reviewed-by:
-
v8-ci-autoroll-builder authored
Summary of changes available at: https://chromium.googlesource.com/v8/v8/+log/bea23e4d..55d0017c 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;luci.chromium.try:linux_optional_gpu_tests_rel;luci.chromium.try:mac_optional_gpu_tests_rel;luci.chromium.try:win_optional_gpu_tests_rel;luci.chromium.try:android_optional_gpu_tests_rel TBR=hablich@chromium.org,v8-waterfall-sheriff@grotations.appspotmail.com Change-Id: I6785c04a89de242ce474ed134405fb167397f522 Reviewed-on: https://chromium-review.googlesource.com/c/1340507Reviewed-by:
v8-ci-autoroll-builder <v8-ci-autoroll-builder@chops-service-accounts.iam.gserviceaccount.com> Commit-Queue: v8-ci-autoroll-builder <v8-ci-autoroll-builder@chops-service-accounts.iam.gserviceaccount.com> Cr-Commit-Position: refs/heads/master@{#609200}
-
Owen Min authored
chrome/browser/policy/browser_dm_token_storage* chrome/browser/policy/machine_level_user_cloud_policy* chrome/browser/policy/cloud/machine_level_user_cloud_policy* components/policy/core/common/cloud/machine_level_user_cloud_policy* Including: Change-Id: If7c4e8c68591db7c227748ab156475dc280e7212 Reviewed-on: https://chromium-review.googlesource.com/c/1340365Reviewed-by:
Georges Khalil <georgesak@chromium.org> Reviewed-by:
Roger Tawa <rogerta@chromium.org> Reviewed-by:
Drew Wilson <atwilson@chromium.org> Commit-Queue: Drew Wilson <atwilson@chromium.org> Cr-Commit-Position: refs/heads/master@{#609199}
-
Yuki Shiino authored
This patch introduces a new counter part of BindingSecurity (in core/ component) as BindingSecurityForPlatform (in platform/ component). BindingSecurityForPlatform is a simple collection of trampolines to BindingSecurity. Change-Id: I41195111ff2f288ee3e314048ac92848adf6be0b Reviewed-on: https://chromium-review.googlesource.com/c/1340439Reviewed-by:
-
Dean Liao authored
According to https://chromium.googlesource.com/chromium/src/+/master/styleguide/c++/c++.md#object-ownership-and-calling-conventions scoped_refptr should be passed by value, not by reference. This CL focus on image_processor. More than 200+ lines need to be fixed in media/gpu: ~/chrome/src/media/gpu$ grep -r "const scoped_refptr<.*>&" | wc 226 1068 21515 BUG=chromium:906479 TEST=Run VEA VDA unittest on devices (peach_pit and elm) VEA: video_encode_accelerator_unittest \ --test_stream_data=bear-320x180.yuv:320:180:1:bear.mp4:100000:30 \ --disable_flush --single-process-tests -v=0 VDA: video_decode_accelerator_unittest \ --test_stream_data=/usr/local/video/test-25fps.h264:320:240:250:258:\ 35:150:1 -v=0 --disable_flush --single-process-tests --ozone-platform=gbm Change-Id: I963501af745124e565eee4a3febf5a0957d53e10 Reviewed-on: https://chromium-review.googlesource.com/c/1341288Reviewed-by:
-
chromium-internal-autoroll authored
https://chrome-internal.googlesource.com/chrome/src-internal.git/+log/42fbb1521607..6340200508f8 Created with: gclient setdep -r src-internal@6340200508f8 The AutoRoll server is located here: https://autoroll-internal.skia.org/r/src-internal-chromium-autoroll 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=mmoss@chromium.org Change-Id: I59ddb3b867746c03f5034fbc4d0495ac60c1be48 Reviewed-on: https://chromium-review.googlesource.com/c/1341563Reviewed-by:
-
