- 05 Oct, 2017 40 commits
-
-
Justin Okamoto authored
This is a reland of 1dd22ad0 Original change's description: > Android: Allow per-target lint suppression configuration > > This change allows for Android lint suppression configurations > to be defined on a per-target basis (assuming said target is > "Android lint capable") rather than having one global config. > This will allow embedders to keep upstream's lint config for > Chromium targets (w/o patching), while using a separate config > for the embedding application. > > Bug: 737897 > Change-Id: Ieccb95767e0ce541522d7a53edbe734c5bbfe6dc > Reviewed-on: https://chromium-review.googlesource.com/698297 > Reviewed-by: Andrew Grieve <agrieve@chromium.org> > Commit-Queue: Andrew Grieve <agrieve@chromium.org> > Cr-Commit-Position: refs/heads/master@{#506393} Bug: 737897 Change-Id: I4d1aecea7d4893f8f525d53a13b4044f27a17221 Reviewed-on: https://chromium-review.googlesource.com/701317Reviewed-by:
Andrew Grieve <agrieve@chromium.org> Commit-Queue: Andrew Grieve <agrieve@chromium.org> Cr-Commit-Position: refs/heads/master@{#506862}
-
Xiaocheng Hu authored
RenderedPosition caches two |InlineBox*|, and uses 1 as the magic value indicating the uncached value, which is a hacky approach. This patch changes them to Optional<InlineBox*> to make the caching not hacky. Bug: 771398 Change-Id: If8bef20755fcc75b04093de6c2211138d336600e Reviewed-on: https://chromium-review.googlesource.com/702924 Commit-Queue: Xiaocheng Hu <xiaochengh@chromium.org> Commit-Queue: Emil A Eklund <eae@chromium.org> Reviewed-by:
Emil A Eklund <eae@chromium.org> Cr-Commit-Position: refs/heads/master@{#506861}
-
Aleks Totic authored
Bug: 772087 Change-Id: I66bf95d43bd7a32718c6e0ad97e179be0db78cc7 Reviewed-on: https://chromium-review.googlesource.com/702816Reviewed-by:
Xianzhu Wang <wangxianzhu@chromium.org> Commit-Queue: Aleks Totic <atotic@chromium.org> Cr-Commit-Position: refs/heads/master@{#506860}
-
Jingkui Wang authored
The units of touch_major and abs_mt_position_x could be different. We have to scale the values from touch_major/touch_minor before we map the value to radius (pixels). Bug: 771374 Change-Id: Ic5ca2424ff72e475e7146ff7a42725445d2fb250 Reviewed-on: https://chromium-review.googlesource.com/702758Reviewed-by:
Michael Spang <spang@chromium.org> Commit-Queue: Jingkui Wang <jkwang@google.com> Cr-Commit-Position: refs/heads/master@{#506859}
-
Nicolas Pena authored
This CL adds methods to allow the MainThreadTaskQueue to determine whether its associated frame is main or not. This will be used to split EQT by frame type. Bug: chromium:747504 Change-Id: I478f2c1f20ce93c6de0837fdde90a495bc4409d7 Reviewed-on: https://chromium-review.googlesource.com/656110 Commit-Queue: Nicolás Peña Moreno <npm@chromium.org> Reviewed-by:
Daniel Cheng <dcheng@chromium.org> Reviewed-by:
Sami Kyöstilä <skyostil@chromium.org> Reviewed-by:
Nate Chapin <japhet@chromium.org> Reviewed-by:
Alexander Timin <altimin@chromium.org> Cr-Commit-Position: refs/heads/master@{#506858}
-
pdfium-deps-roller@chromium.org authored
https://pdfium.googlesource.com/pdfium.git/+log/4ce4f5f8ab0b..480ca10f7a20 $ git log 4ce4f5f8a..480ca10f7 --date=short --no-merges --format='%ad %ae %s' 2017-10-05 dsinclair Remove unused CPVT_SecProps 2017-10-05 dsinclair Remove more unused params 2017-10-05 dsinclair Remove unused parameters 2017-10-05 rharrison Add ObservedPtr to catch Widget being killed by JS 2017-10-05 rharrison Add ObservedPtrs to catch issues in SaveData 2017-10-05 npm Create FreeType roll script 2017-10-04 rharrison Make GIF decoder more standards complaint 2017-10-05 dsinclair Move CPDF_RenderOptions members to private 2017-10-05 dsinclair Remove friend from CPDF_ApSettings 2017-10-05 npm Roll FT to ae7dc1f62d826083d418e86cce3f66a76dff038a 2017-10-05 dsinclair Remove friends from form code 2017-10-05 dsinclair Remove friends from CPDF_VariableText Created with: roll-dep src/third_party/pdfium BUG=771979,756427,770337 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=dsinclair@chromium.org Change-Id: I875e810e00b7d619534e2dc24519c27088423e15 Reviewed-on: https://chromium-review.googlesource.com/703198 Reviewed-by: <pdfium-deps-roller@chromium.org> Commit-Queue: <pdfium-deps-roller@chromium.org> Cr-Commit-Position: refs/heads/master@{#506857}
-
Dirk Pranke authored
This reverts commit 570b9f37. Reason for revert: Turns out we weren't ready for this yet. Original change's description: > Stop expecting 10.9 builds for the layout tests. > > We are dropping layout test support on 10.9; the first step is to > stop expecting the bots to be running and generating baselines. > > R=qyearsley@chromium.org > BUG=736256 > > Change-Id: I2d5c647cfb2d68df3bf2d078332c504cfccfa8d7 > Reviewed-on: https://chromium-review.googlesource.com/699559 > Reviewed-by: Quinten Yearsley <qyearsley@chromium.org> > Commit-Queue: Dirk Pranke <dpranke@chromium.org> > Cr-Commit-Position: refs/heads/master@{#506417} TBR=qyearsley@chromium.org,dpranke@chromium.org # Not skipping CQ checks because original CL landed > 1 day ago. Bug: 736256 Change-Id: I9633bbe19af74e9a7ba54cfa52f89baa82fcde47 Reviewed-on: https://chromium-review.googlesource.com/703375Reviewed-by:
Dirk Pranke <dpranke@chromium.org> Commit-Queue: Dirk Pranke <dpranke@chromium.org> Cr-Commit-Position: refs/heads/master@{#506856}
-
Steven Valdez authored
https://boringssl.googlesource.com/boringssl/+log/e9c7b1c8ae85e5625e9d24d9e20ccedeaaeb0d0a..12fdd08a401d2ef562612638abd016f8b5794cc7 This removes the RecordType and NoSessionID TLS 1.3 variants. BUG=none Change-Id: I5edc8c6e0e67f581095efa326e4fdc3d339c7204 Reviewed-on: https://chromium-review.googlesource.com/698624 Commit-Queue: Steven Valdez <svaldez@chromium.org> Reviewed-by:
Alexei Svitkine <asvitkine@chromium.org> Reviewed-by:
David Benjamin <davidben@chromium.org> Cr-Commit-Position: refs/heads/master@{#506855}
-
Rune Lillesveen authored
We tried to update the fragment anchor position immediately when all script blocking sheets are removed. As part of that update we do a style and layout update. The last script blocking sheet may be removed as part of removing a style element from the DOM. Updating style and layout in the middle of a DOM operation is not safe. Instead, update the fragment anchor position during the next safe lifecycle update. The crash was caused by Document still pointing to a documentElement being removed while updating the style and layout tree, but its parent pointer had already been set to null. Bug: 771088 Change-Id: Iaaaeece23d795c2e41d31b9fb97fc795eb2b8305 Reviewed-on: https://chromium-review.googlesource.com/698305 Commit-Queue: Rune Lillesveen <rune@opera.com> Reviewed-by:
Steve Kobes <skobes@chromium.org> Cr-Commit-Position: refs/heads/master@{#506854}
-
Corentin Wallez authored
BUG=602737 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: Icd8e5a4c6d1c7ccddbbd591ef22326d2719084f5 Reviewed-on: https://chromium-review.googlesource.com/700989 Commit-Queue: Corentin Wallez <cwallez@chromium.org> Reviewed-by:
Antoine Labour <piman@chromium.org> Cr-Commit-Position: refs/heads/master@{#506853}
-
Yoshifumi Inoue authored
This patch introduces |FrameSelection::SelectWordAroundCaret()| to move code to "FrameSelection.cpp" from "WebLocalFrameImpl.cpp" as preparation of removing usage of |VisiblePositon| and |VisibleSelection| outside "editing/" for improving code health. Bug: 657237 Change-Id: I75c92513e4fc577bb6336553256fdd69a594277d Reviewed-on: https://chromium-review.googlesource.com/702214Reviewed-by:
Koji Ishii <kojii@chromium.org> Commit-Queue: Koji Ishii <kojii@chromium.org> Cr-Commit-Position: refs/heads/master@{#506852}
-
kristipark authored
Bug: 772067 Change-Id: Ia18e7aaa8c22a3c401611bc9952d1ab9fa3e01d9 Reviewed-on: https://chromium-review.googlesource.com/703497 Commit-Queue: Kristi Park <kristipark@chromium.org> Commit-Queue: Daniel Murphy <dmurph@chromium.org> Reviewed-by:
Daniel Murphy <dmurph@chromium.org> Reviewed-by:
Joshua Bell <jsbell@chromium.org> Cr-Commit-Position: refs/heads/master@{#506851}
-
Will Harris authored
NULL -> nullptr TRUE -> true FALSE -> false Changes that were not purely mechanical: - Move a few functions to anonymous namespace. - Replace some unneeded typedef with declfunc. - CreateEvent returns nullptr for invalid handle not INVALID_HANDLE_VALUE. - CreateFile returns INVALID_HANDLE_VALUE not nullptr. - Add enum for is_broker in filesystem interception. Bug: Cq-Include-Trybots: master.tryserver.chromium.win:win10_chromium_x64_rel_ng Change-Id: Iff1beaf7bd7433a32ccb15d09298a3244c6700d1 Reviewed-on: https://chromium-review.googlesource.com/698375 Commit-Queue: Will Harris <wfh@chromium.org> Reviewed-by:
Penny MacNeil <pennymac@chromium.org> Cr-Commit-Position: refs/heads/master@{#506850}
-
Weidong Guo authored
Changes: 1. Handle arrow up/down in SearchResultTileItemListView, so that focus could traverse vertically. 2. Move focus to search box when it is moved outside SearchResultPageView during vertical focus traversal. Design Doc: go/applist-focus BUG=766810 TEST=AppListViewFocusTest.* Change-Id: Idcd54e7d7c4f2cd6fe425e80218310581c7bcaaa Reviewed-on: https://chromium-review.googlesource.com/701578 Commit-Queue: Weidong Guo <weidongg@chromium.org> Reviewed-by:
Xiyuan Xia <xiyuan@chromium.org> Cr-Commit-Position: refs/heads/master@{#506849}
-
Devlin Cronin authored
crrev.com/1f2b58d7 removed the old settings page code, but missed cleaning up the strings. Remove some that are no longer used. This covers: IDS_ABOUT_BUTTON -> IDS_CERT_MANAGER_CERT_PARSE_ERROR Bug: 771795 Change-Id: I8b1ad50dcfc2d45e346945a36b62eb6807dac4e1 Reviewed-on: https://chromium-review.googlesource.com/702921Reviewed-by:
Steven Bennetts <stevenjb@chromium.org> Commit-Queue: Devlin <rdevlin.cronin@chromium.org> Cr-Commit-Position: refs/heads/master@{#506848}
-
Danyao Wang authored
restore_session.html is a magic file included in the app bundle that helps WKBasedNavigationManager implement restore. The navigation manager serializes and encodes the session history in the query parameter, and loads restore_session.html to inject the history into the web view. Bug: 734150 Change-Id: Ideaa7239eb3239322e3e72c5599736962962b465 Reviewed-on: https://chromium-review.googlesource.com/698465Reviewed-by:
Kurt Horimoto <kkhorimoto@chromium.org> Commit-Queue: Kurt Horimoto <kkhorimoto@chromium.org> Cr-Commit-Position: refs/heads/master@{#506847}
-
David Trainor authored
Add the following logs to help us have an idea of what the service behavior is: - Download.Service.Entry.Event - Logs event actions taken on an Entry like start, resume, retry, and suspend. - Download.Service.Entry.RetryCount - At the time of a retry, logs which retry attempt this is. - Download.Service.Entry.ResumptionCount - At the time of resumption, logs which resumption attempt this is. Bug: 769426 Change-Id: I11e5974521c6ba610d9d58940b4dae61ae8c1fbb Reviewed-on: https://chromium-review.googlesource.com/701835 Commit-Queue: David Trainor <dtrainor@chromium.org> Reviewed-by:
Shakti Sahu <shaktisahu@chromium.org> Reviewed-by:
Ilya Sherman <isherman@chromium.org> Reviewed-by:
Xing Liu <xingliu@chromium.org> Cr-Commit-Position: refs/heads/master@{#506846}
-
Devlin Cronin authored
ExtensionService::extension_registrar_ is constructed at ExtensionService construction time, and is never reset. There's no reason to have it as a std::unique_ptr<> instead of a stack-allocated member variable. Make it the latter. Bug: 770879 Change-Id: I68001b9214b6176fc4703e3c6d2f1cbdcece9dff Reviewed-on: https://chromium-review.googlesource.com/701398Reviewed-by:
Michael Giuffrida <michaelpg@chromium.org> Commit-Queue: Devlin <rdevlin.cronin@chromium.org> Cr-Commit-Position: refs/heads/master@{#506845}
-
Morten Stenshorne authored
We already did ignore borders and padding, but forgot about scrollbars. This was quite catastrophic for multicol, if a horizontal scrollbar was actually taller than the columns. We'd end up pushing the scrollbar in front of us (in block direction), column after column. Infinite loop (until OOM). Cq-Include-Trybots: master.tryserver.chromium.linux:linux_layout_tests_layout_ng Change-Id: I5c3038406ccf30eeb0e107b31c6bb783c7089ffd Reviewed-on: https://chromium-review.googlesource.com/702299Reviewed-by:
Ian Kilpatrick <ikilpatrick@chromium.org> Commit-Queue: Morten Stenshorne <mstensho@opera.com> Cr-Commit-Position: refs/heads/master@{#506844}
-
Steven Holte authored
Bug: 754892 Change-Id: I3498e985e9382aa5149406fcad51a8314f42c43d Reviewed-on: https://chromium-review.googlesource.com/701257Reviewed-by:
Alexei Svitkine <asvitkine@chromium.org> Commit-Queue: Steven Holte <holte@chromium.org> Cr-Commit-Position: refs/heads/master@{#506843}
-
Gregory Chatzinoff authored
The prior app store link does not work in iOS11, so use the new link instead. Bug: 771211, 761200 Change-Id: I5b3daff63f4e0f9f78ebe6721aa6497282339eee Reviewed-on: https://chromium-review.googlesource.com/702686 Commit-Queue: Gregory Chatzinoff <gchatz@chromium.org> Reviewed-by:
Rohit Rao (ping after 24h) <rohitrao@chromium.org> Reviewed-by:
Peter Lee <pkl@chromium.org> Cr-Commit-Position: refs/heads/master@{#506842}
-
catapult-deps-roller@chromium.org authored
https://chromium.googlesource.com/catapult.git/+log/09fca0a3600a..400dbc4b2b00 $ git log 09fca0a36..400dbc4b2 --date=short --no-merges --format='%ad %ae %s' 2017-10-05 dtu [pinpoint] Set `updated` on Run(), not using `auto_now` Created with: roll-dep src/third_party/catapult 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.android:android_optional_gpu_tests_rel TBR=sullivan@chromium.org Change-Id: Ic4eec5029d0ad354c0bd4967525973ddd9c36c5b Reviewed-on: https://chromium-review.googlesource.com/702761 Reviewed-by: <catapult-deps-roller@chromium.org> Commit-Queue: <catapult-deps-roller@chromium.org> Cr-Commit-Position: refs/heads/master@{#506841}
-
Ken Rockot authored
This reverts commit 12234368. Reason for revert: Looks like a source of these build failures: https://build.chromium.org/p/chromium.android/builders/Android%20Cronet%20ARM64%20Builder%20%28dbg%29/builds/69753 Original change's description: > Android: Turn on errorprone by default > > BUG=383646 > > Change-Id: Ia87d2dff787ffc5e4a59aca1af81a0ac9b727b4a > Reviewed-on: https://chromium-review.googlesource.com/702574 > Reviewed-by: Andrew Grieve <agrieve@chromium.org> > Commit-Queue: Peter Wen <wnwen@chromium.org> > Cr-Commit-Position: refs/heads/master@{#506831} TBR=wnwen@chromium.org,agrieve@chromium.org Change-Id: Ie933459d048c090df574bf45b6e2b644de3e66e4 No-Presubmit: true No-Tree-Checks: true No-Try: true Bug: 383646 Reviewed-on: https://chromium-review.googlesource.com/703395Reviewed-by:
Ken Rockot <rockot@chromium.org> Commit-Queue: Ken Rockot <rockot@chromium.org> Cr-Commit-Position: refs/heads/master@{#506840}
-
Jian Li authored
We used to disable all form control elements when saving a page as MHTML since the form in the MHTML document cannot be worked on due to form submission and script execution disabled. As suggested by Jeffrey Yasskin, we switched to disable form control elements in the loaded MHTML document. Bug: 771786 Change-Id: I44b9c324192be052dd47db3d126a911ab5dd34e3 Reviewed-on: https://chromium-review.googlesource.com/701368 Commit-Queue: Jian Li <jianli@chromium.org> Reviewed-by:
Daniel Cheng <dcheng@chromium.org> Cr-Commit-Position: refs/heads/master@{#506839}
-
MinChen authored
Bug: 763240 Change-Id: Ibe0ce1f0d44dd947154d231769b53d0eda35dc09 Reviewed-on: https://chromium-review.googlesource.com/702722Reviewed-by:
Jay Civelli <jcivelli@chromium.org> Commit-Queue: min c <minch@chromium.org> Cr-Commit-Position: refs/heads/master@{#506838}
-
Daniel Bratell authored
"using namespace" in a global scope breaks jumbo builds (and are not allowed in the code style guide). This patch replaces them with explicit namespaces. Bug: 771153 Change-Id: If9c4de9e115c5531708cf06f64cb0088daca2d57 Reviewed-on: https://chromium-review.googlesource.com/697370Reviewed-by:
Vadym Doroshenko <dvadym@chromium.org> Reviewed-by:
Daniel Cheng <dcheng@chromium.org> Commit-Queue: Daniel Bratell <bratell@opera.com> Cr-Commit-Position: refs/heads/master@{#506837}
-
Devlin Cronin authored
chrome_user_metrics_extension.proto imports extension_install.proto, but doesn't need to. This was introduced in https://crrev.com/e94d484908fa0f9472c580e7fb9c4ec9c9c1bf78. Remove the unneeded import. Bug: None Change-Id: I6cb3490cbb3ba6da31420588994664dd123823fc Reviewed-on: https://chromium-review.googlesource.com/699058Reviewed-by:
Steven Holte <holte@chromium.org> Commit-Queue: Devlin <rdevlin.cronin@chromium.org> Cr-Commit-Position: refs/heads/master@{#506836}
-
James Cook authored
We have not shipped Windows Ash for years. There is a 1:1 mapping between use_ash and is_chromeos, so consolidate them. Bug: 673826 Test: compiles, gn check for Chrome OS, Linux, Linux Ozone Change-Id: Idc459cbca6b8447ce0f7e7aa9772e38835492234 Reviewed-on: https://chromium-review.googlesource.com/702756Reviewed-by:
Lei Zhang <thestig@chromium.org> Commit-Queue: James Cook <jamescook@chromium.org> Cr-Commit-Position: refs/heads/master@{#506835}
-
Ehsan Chiniforooshan authored
It is flaky. Error: [FATAL:profile_destroyer.cc(56)] Check failed: hosts.empty() || profile->IsOffTheRecord() || content::RenderProcessHost::run_renderer_in_process(). Profile still has 1 hosts 0x7f035e2968ec base::debug::StackTrace::StackTrace() 0x7f035e2bc41c logging::LogMessage::~LogMessage() 0x0000025be6d8 ProfileDestroyer::DestroyProfileWhenAppropriate() 0x0000023da2a0 std::__1::__tree<>::destroy() 0x0000023da1e9 ProfileManager::~ProfileManager() 0x0000023d1ace ProfileManager::~ProfileManager() 0x000002559112 BrowserProcessImpl::StartTearDown() 0x0000022ad6f2 ChromeBrowserMainParts::PostMainMessageLoopRun() 0x000001e0a813 chromeos::ChromeBrowserMainPartsChromeos::PostMainMessageLoopRun() 0x7f035a63837a content::BrowserMainLoop::ShutdownThreadsAndCleanUp() ... Bug=772072 Bug: Change-Id: I91391979088148a9857728f5c7b30d397b8335f2 Reviewed-on: https://chromium-review.googlesource.com/703034Reviewed-by:
Xiyuan Xia <xiyuan@chromium.org> Commit-Queue: Ehsan Chiniforooshan <chiniforooshan@chromium.org> Cr-Commit-Position: refs/heads/master@{#506834}
-
Adithya Srinivasan authored
ScriptWrappableVisitor knows when wrapper tracing is in progress, so when an isolate is provided we can look it up from the visitor directly without trying to access the current ThreadState. Bug: Change-Id: I79ab86ee7ff3774748ba9295b2604de53eaf8db4 Reviewed-on: https://chromium-review.googlesource.com/699067 Commit-Queue: Adithya Srinivasan <adithyas@chromium.org> Reviewed-by:
Kentaro Hara <haraken@chromium.org> Reviewed-by:
Michael Lippautz <mlippautz@chromium.org> Reviewed-by:
Jeremy Roman <jbroman@chromium.org> Cr-Commit-Position: refs/heads/master@{#506833}
-
Xianzhu Wang authored
The switch enables blink feature SlimmingPaintV175 which is an intermediate step between SlimmingPaintV1 and V2. The command line switch is a short form of --enable-blink-features=SlimmingPaintV175, which will ease flag specific layout test expectations and field trials. Bug: 771643 Change-Id: Iedb050d43380a8a1ab548a649d7f526a8f302066 Reviewed-on: https://chromium-review.googlesource.com/698370 Commit-Queue: Xianzhu Wang <wangxianzhu@chromium.org> Reviewed-by:
Jochen Eisinger <jochen@chromium.org> Cr-Commit-Position: refs/heads/master@{#506832}
-
Peter Wen authored
BUG=383646 Change-Id: Ia87d2dff787ffc5e4a59aca1af81a0ac9b727b4a Reviewed-on: https://chromium-review.googlesource.com/702574Reviewed-by:
Andrew Grieve <agrieve@chromium.org> Commit-Queue: Peter Wen <wnwen@chromium.org> Cr-Commit-Position: refs/heads/master@{#506831}
-
Ken Rockot authored
The original correct baseline got lost in the shuffle of a series of colliding reverts. This restores the file to its state prior to the (now reverted) r506693. TBR=drott@chromium.org TBR=eae@chromium.org Bug: 707713 Change-Id: I0f15bc1c4b01ad5972d30e86f3d47b13183fffaa Reviewed-on: https://chromium-review.googlesource.com/703354Reviewed-by:
Ken Rockot <rockot@chromium.org> Cr-Commit-Position: refs/heads/master@{#506830}
-
Hidehiko Abe authored
BUG=739622 TEST=Build. Change-Id: I196aea8c5d66c020a22c6eceb899783da3b2cd3a Reviewed-on: https://chromium-review.googlesource.com/697047 Commit-Queue: Hidehiko Abe <hidehiko@chromium.org> Reviewed-by:
Ryo Hashimoto <hashimoto@chromium.org> Reviewed-by:
Steven Bennetts <stevenjb@chromium.org> Cr-Commit-Position: refs/heads/master@{#506829}
-
Sean O'Brien authored
Extends the wl_pointer protocol using the wayland upstream zwp_pointer_gestures protocol to pass information about pinch gestures. BUG=b:25668308 Change-Id: I9f90ebebec3f8476181e0426401827a3cf1064e4 Reviewed-on: https://chromium-review.googlesource.com/699017 Commit-Queue: Sean O'Brien <seobrien@chromium.org> Reviewed-by:
David Reveman <reveman@chromium.org> Cr-Commit-Position: refs/heads/master@{#506828}
-
Victor Costan authored
This reverts commit 9f9f66ee. Reason for revert: Broke Windows build - https://build.chromium.org/p/chromium/builders/Win/builds/59636 Original change's description: > Roll src/third_party/leveldatabase/src/ 09a3c8e74..4a7e7f50d (11 commits) > > This CL also #defines DeleteFile as DeleteFileW on Windows for all > compilation units that depend on leveldb. This definition impacts > leveldb's API (leveldb::Env::DeleteFile), and is in effect in most of > Chromium, because it is introduced by <windows.h>. After this CL, the > definition is guaranteed to be in effect everywhere, so we don't have to > reason about #include side-effects. > > https://chromium.googlesource.com/external/leveldb.git/+log/09a3c8e74175..4a7e7f50dcf6 > > $ git log 09a3c8e74..4a7e7f50d --date=short --no-merges --format='%ad %ae %s' > 2017-10-04 cmumford Add LEVELDB_EXPORT macro to export public symbols. > 2017-10-04 costan leveldb: Include <algorithm> in util/env_test.cc. > 2017-10-03 costan Fix FD leak in POSIX Env. > 2017-10-03 costan leveldb: Add tests for CL 170769101. > 2017-10-03 costan Remove handling for unused LRUHandle representation special case. > 2017-10-02 sanjay Fix file writing bug in CL 170738066. > 2017-10-02 cmumford Fix use of uninitialized value in LRUHandle. > 2017-10-02 sanjay Fix issue 474: a race between the f*_unlocked() STDIO calls in env_posix.cc and concurrent application calls to fflush(NULL). > 2017-09-26 costan Use portable CRC32C from google/crc32c. > 2017-09-01 costan Remove confusing and unnecessary if. > 2017-09-01 costan Simplify Table::Open() flow and remove a delete call. > > > Created with: > roll-dep src/third_party/leveldatabase/src > > Bug: > Change-Id: I0096e6b863616d14c20994d6cb9696ebc263f138 > Reviewed-on: https://chromium-review.googlesource.com/699018 > Reviewed-by: Chris Mumford <cmumford@chromium.org> > Commit-Queue: Victor Costan <pwnall@chromium.org> > Cr-Commit-Position: refs/heads/master@{#506815} TBR=cmumford@chromium.org,pwnall@chromium.org Change-Id: Ifb074413f4f02289cee7c2535fde1ed516da9ad6 No-Presubmit: true No-Tree-Checks: true No-Try: true Reviewed-on: https://chromium-review.googlesource.com/703316Reviewed-by:
Victor Costan <pwnall@chromium.org> Commit-Queue: Victor Costan <pwnall@chromium.org> Cr-Commit-Position: refs/heads/master@{#506827}
-
Lei Zhang authored
ResourceBundle became ui::ResourceBundle in r72038 but many callers still refer to it without the namespace. Change-Id: I920e21319041cd0e2a2fa671db709a9d8483fb2e Reviewed-on: https://chromium-review.googlesource.com/701662Reviewed-by:
Luke Halliwell <halliwell@chromium.org> Reviewed-by:
Eugene But <eugenebut@chromium.org> Reviewed-by:
Richard Coles <torne@chromium.org> Reviewed-by:
Scott Violet <sky@chromium.org> Commit-Queue: Lei Zhang <thestig@chromium.org> Cr-Commit-Position: refs/heads/master@{#506826}
-
Dmitry Skiba authored
This is a follow-up to crrev.com/c/667936 which moved MAIN/LAUNCHER handling into separate .LauncherMain activity alias. That CL caused existing Chrome app shortcuts to disappear on update. Turned out that Android launcher creates {MAIN/LAUNCHER, <component>} intents for app shortcuts (i.e. it bakes in the component), and then checks that after each update MAIN/LAUNCHER intent for the app resolves to the shortcut's component. If that check fails, launcher removes the shortcut. This means that MAIN/LAUNCHER intents must continue to be handled by .Main activity alias. This CL does two renamings: 1. Renames .Main activity alias to .IntentDispatcher alias 2. Renames .LauncherMain alias to .Main I.e. MAIN/LAUNCHER intents are now handled by .Main alias, making sure shortcuts won't be removed. Additionally this CL moves more MAIN categories from .IntentDispatcher into .Main alias, so that almost all MAIN intents are handled by .Main (one exception is NOTIFICATION_PREFERENCES, as it needs dispatching). Bug: 761606 Change-Id: I6cb926c4677758190f743260e11a85129c8863bb Reviewed-on: https://chromium-review.googlesource.com/700803 Commit-Queue: Dmitry Skiba <dskiba@chromium.org> Reviewed-by:Ted Choc <tedchoc@chromium.org> Cr-Commit-Position: refs/heads/master@{#506825}
-
Jamie Madill authored
Explicitly specifying the GPU driver version in the config will allow us to safely test new driver versions. Because the tests will pick machines of a particular driver version specified in the json, we can ask infra to upgrade a few machines, then run a try job using the new driver version. This modifies the generate script to allow use of wildcards in test dimension set specifiers. Wildcards are only supported as the last character in the dimension. This allows us to more easily match ranges of driver versions. Re-land fixes the driver version for Windows 10 bots. The current NVIDIA driver version used by the bots is 372.54 for the Windows 7 bots and 376.19 for Windows 10. R=kbr@chromium.org BUG=771654 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: Ie06cedefa0e4c4afc598ab7148d192c3f60eca34 Reviewed-on: https://chromium-review.googlesource.com/702855Reviewed-by:
Kenneth Russell <kbr@chromium.org> Commit-Queue: Jamie Madill <jmadill@chromium.org> Cr-Commit-Position: refs/heads/master@{#506824}
-
Irmak Kavasoglu authored
Other possible passwords variable of the password form is currently holding all possible passwords. See this cl: https://chromium-review.googlesource.com/c/chromium/src/+/660077 This cl renames the variable accordingly. Bug: 753806 Change-Id: Id6e19b876392039704d1f23a8b42d1ca18e79ea6 Reviewed-on: https://chromium-review.googlesource.com/693014 Commit-Queue: Maxim Kolosovskiy <kolos@chromium.org> Reviewed-by:
Mike West <mkwst@chromium.org> Reviewed-by:
Vasilii Sukhanov <vasilii@chromium.org> Reviewed-by:
Maxim Kolosovskiy <kolos@chromium.org> Cr-Commit-Position: refs/heads/master@{#506823}
-