- 30 Jan, 2018 40 commits
-
-
Theresa authored
BUG=805976 Change-Id: Icdaf76079b16f7664c5dde3b63f429ae5473532f Reviewed-on: https://chromium-review.googlesource.com/893647Reviewed-by:
Ted Choc <tedchoc@chromium.org> Commit-Queue: Theresa <twellington@chromium.org> Cr-Commit-Position: refs/heads/master@{#532999}
-
Friedrich Horschig authored
When under memory pressure, the activity is destroyed and the search result lost. By explicitly saving the query in the instance state bundle, the search results can be recovered even though the activity is recreated. Bug: 807306 Change-Id: Ice12994cdeefc7ee293114207b5faa93cebda5a5 Reviewed-on: https://chromium-review.googlesource.com/893187Reviewed-by:
Theresa <twellington@chromium.org> Commit-Queue: Friedrich Horschig <fhorschig@chromium.org> Cr-Commit-Position: refs/heads/master@{#532998}
-
Steven Bennetts authored
Previously tpm_firmware_update_requested might be undefined. This makes the argument required and ensures that it is passed. Bug: 800103 Cq-Include-Trybots: master.tryserver.chromium.linux:closure_compilation Change-Id: Ia5263d011a58f5f444b64df2e7109083c70b3daf Reviewed-on: https://chromium-review.googlesource.com/887631 Commit-Queue: Steven Bennetts <stevenjb@chromium.org> Reviewed-by:
Demetrios Papadopoulos <dpapad@chromium.org> Cr-Commit-Position: refs/heads/master@{#532997}
-
Yoland Yan authored
Bug: 807251 Change-Id: I49de4e1668fe851ced12fa335e54e5078b81434f Reviewed-on: https://chromium-review.googlesource.com/887792 Commit-Queue: Yoland Yan <yolandyan@chromium.org> Reviewed-by:
Bo <boliu@chromium.org> Cr-Commit-Position: refs/heads/master@{#532996}
-
Victor Costan authored
This CL shrinks the Linux binary by 34KB (138,266,816 to 138,232,320). Code savings aside, this makes sure we don't unintentionally start depending on these features. The following compile-time options are recommended in the SQLite documentation [1]: * SQLITE_LIKE_DOESNT_MATCH_BLOBS - The LIKE and GLOB operators don't work on BLOB columns. This access pattern would be a performance wrench, so it's nice to fail hard here. * SQLITE_OMIT_DEPRECATED - This removes support for deprecated SQLite APIs. Note that SQLite promises to maintain indefinite backwards compatbility for SQL queries (modulo features disabled via compile-time options), so this only applies to the SQLite API. We should never be using deprecated APIs. * SQLITE_OMIT_PROGRESS_CALLBACK - Remove sqlite3_progress_handler(). We don't use this feature, and the documentation says it comes with a small performance penalty. * SQLITE_OMIT_SHARED_CACHE - Using a shared cache sounds nice from a memory consumption standpoint, and we already do that for LevelDB. Unfortunately, Chromium's SQLite databases use a variety of page sizes, which makes cache sharing impossible. * SQLITE_USE_ALLOCA - Use alloca() instead of malloc() for allocating temporary spaces in functions. All of Chrome's platforms support this option, and it results in a slightly smaller binary and less heap churn. The following compile-time options disable features that happen not to be used. * SQLITE_OMIT_AUTOINIT: We initialize SQLite manually in //sql/connection.cc. * SQLITE_OMIT_AUTORESET: We calls sqlite3_reset() correctly to reset prepared statements. * SQLITE_OMIT_GET_TABLE: We don't use sqlite3_{get,free}_table(). * SQLITE_OMIT_LOAD_EXTENSION: We don't use sqlite3_{enable_}load_extension(). Asides from the code savings, there's a tiny security benefit to knowing that extension loading code is definitely not reachable from WebSQL. * SQLITE_OMIT_TCL_VARIABLE: We don't use TCL variables. * SQLITE_OMIT_TRACE: We don't use sqlite3_{profile,trace}(). Bug: 807093 Change-Id: Ie5e59e55dd9b2ed52f7c27682a809c9f7c36d4a3 Reviewed-on: https://chromium-review.googlesource.com/882961 Commit-Queue: Victor Costan <pwnall@chromium.org> Reviewed-by:
Chris Mumford <cmumford@chromium.org> Cr-Commit-Position: refs/heads/master@{#532995}
-
https://chromium.googlesource.com/catapult.git/+log/9ce4426b187b..3f8c43b79487 $ git log 9ce4426b1..3f8c43b79 --date=short --no-merges --format='%ad %ae %s' 2018-01-30 perezju [Devil] Use ListProcesses to count processes in device_monitor Created with: roll-dep src/third_party/catapult The AutoRoll server is located here: https://catapult-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.android:android_optional_gpu_tests_rel TBR=sullivan@chromium.org Change-Id: I03b4c527610e82b303c44b4e7eced40a855f53d9 Reviewed-on: https://chromium-review.googlesource.com/893646 Commit-Queue: catapult-chromium-autoroll <catapult-chromium-autoroll@skia-buildbots.google.com.iam.gserviceaccount.com> Reviewed-by:
catapult-chromium-autoroll <catapult-chromium-autoroll@skia-buildbots.google.com.iam.gserviceaccount.com> Cr-Commit-Position: refs/heads/master@{#532994}
-
https://chromium.googlesource.com/angle/angle.git/+log/6969132c134f..13fd988cf71d $ git log 6969132c1..13fd988cf --date=short --no-merges --format='%ad %ae %s' 2018-01-29 geofflang Vulkan: Add the Vulkan API to gpu_test_expectations. Created with: roll-dep src/third_party/angle The AutoRoll server is located here: https://angle-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.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 TBR=jmadill@chromium.org Change-Id: I1b794d0c516617b2ca8da6b1badeb7c5a7247132 Reviewed-on: https://chromium-review.googlesource.com/893641 Commit-Queue: angle-chromium-autoroll <angle-chromium-autoroll@skia-buildbots.google.com.iam.gserviceaccount.com> Reviewed-by:
angle-chromium-autoroll <angle-chromium-autoroll@skia-buildbots.google.com.iam.gserviceaccount.com> Cr-Commit-Position: refs/heads/master@{#532993}
-
Réda Housni Alaoui authored
Image capture tests were disabled since issue 633242. We think this was caused by the fact that bots were running webcam tests concurrently. Now that concurrency has been disabled for these tests on Windows, they should pass. Bug: 806431 Change-Id: Ic836752abef240d622f13cc7c44883540e033cf7 Reviewed-on: https://chromium-review.googlesource.com/890518Reviewed-by:
Christian Fremerey <chfremer@chromium.org> Commit-Queue: Christian Fremerey <chfremer@chromium.org> Cr-Commit-Position: refs/heads/master@{#532992}
-
Nico Weber authored
Bug: 177475 R=rogerm@chromium.org Change-Id: I4c5c56af1d9ed85a4df6ec9651a189d232180036 Reviewed-on: https://chromium-review.googlesource.com/889761Reviewed-by:
Roger McFarlane <rogerm@chromium.org> Cr-Commit-Position: refs/heads/master@{#532991}
-
Justin Cohen authored
Bug: 805634 Cq-Include-Trybots: master.tryserver.chromium.mac:ios-simulator-cronet;master.tryserver.chromium.mac:ios-simulator-full-configs Change-Id: Id3433625e9c8da612431181aeff11abbea6247aa Reviewed-on: https://chromium-review.googlesource.com/893824Reviewed-by:
Rohit Rao <rohitrao@chromium.org> Commit-Queue: Justin Cohen <justincohen@chromium.org> Cr-Commit-Position: refs/heads/master@{#532990}
-
Scott Graham authored
Causing extra long compiles, and is currently of limited utility on Fuchsia. We might want to investigate the runtime performance delta for this feature once more complex JS is running on Fuchsia. Bug: 764576, 807320 Change-Id: Iefee81a7e9078b42b68766f6891bbed2233a6599 Reviewed-on: https://chromium-review.googlesource.com/893427Reviewed-by:
Wez <wez@chromium.org> Commit-Queue: Scott Graham <scottmg@chromium.org> Cr-Commit-Position: refs/heads/master@{#532989}
-
Troy Hildebrandt authored
This change removes the VoiceSearchTabHelper, and instead allows voice searches to trigger a user activation on the frame. This allows the search results audio to autoplay under the new autoplay policy guidelines. This is achieved by adding an IPC from the RenderFrameHost to the RenderFrame to allow us to call Frame::NotifyUserActivation when a voice search is performed. Bug: 715588 Change-Id: I3440d5482712c21377914a4ccda028b42161bd8f Reviewed-on: https://chromium-review.googlesource.com/820601Reviewed-by:
Alex Moshchuk <alexmos@chromium.org> Reviewed-by:
Michael Wasserman <msw@chromium.org> Reviewed-by:
Kent Tamura <tkent@chromium.org> Reviewed-by:
Maria Khomenko <mariakhomenko@chromium.org> Reviewed-by:
Mounir Lamouri (slow) <mlamouri@chromium.org> Commit-Queue: Troy Hildebrandt <thildebr@chromium.org> Cr-Commit-Position: refs/heads/master@{#532988}
-
Anna Offenwanger authored
Detached the xr test folder from the vr test folder by creating its own set of helper classes. Converted all tests to promise tests where it made sense. As much as possible detached chrome specific code for when we convert tests to WPTs. Bug: 777567 Change-Id: I90e7800cbf86b09f72937033e4a840d1f5719b61 Reviewed-on: https://chromium-review.googlesource.com/889652 Commit-Queue: Anna Offenwanger <offenwanger@chromium.org> Reviewed-by:
Brian Sheedy <bsheedy@chromium.org> Reviewed-by:
Brandon Jones <bajones@chromium.org> Cr-Commit-Position: refs/heads/master@{#532987}
-
Fredrik Hubinette authored
Bug: 754986 Change-Id: Id9b981552b78c59629d2b720717c1ec6e8e4b8a4 Reviewed-on: https://chromium-review.googlesource.com/891650Reviewed-by:
ccameron <ccameron@chromium.org> Commit-Queue: Fredrik Hubinette <hubbe@chromium.org> Cr-Commit-Position: refs/heads/master@{#532986}
-
Ken Rockot authored
Gets rid of the ThrottlingURLLoader::CreateLoaderAndStart override which takes a callback instead of a SharedURLLoaderFactory. Moves LoaderCallback into URLLoaderRequestHandler to clarify its relation to implementations of that class. Introduces SingleRequestURLLoaderFactory as an implementation of SharedURLLoaderFactory that handles a single loader request using a OnceCallback provided at construction time. Adapts NavigationURLLoaderNetworkService to use the new API. Bug: 721414 Change-Id: I1030aef2096edeb4fe525999e6b7d1a626a4110c Reviewed-on: https://chromium-review.googlesource.com/889650 Commit-Queue: Ken Rockot <rockot@chromium.org> Reviewed-by:
Kinuko Yasuda <kinuko@chromium.org> Reviewed-by:
Yuzhu Shen <yzshen@chromium.org> Cr-Commit-Position: refs/heads/master@{#532985}
-
v8-autoroll authored
Summary of changes available at: https://chromium.googlesource.com/v8/v8/+log/3c62a1ec..adcf8626 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: Ib596ed4a556f44b753050fbf2518b3d55ad05d2a Reviewed-on: https://chromium-review.googlesource.com/893280Reviewed-by:
v8 autoroll <v8-autoroll@chromium.org> Commit-Queue: v8 autoroll <v8-autoroll@chromium.org> Cr-Commit-Position: refs/heads/master@{#532984}
-
Charles Zhao authored
Change-Id: I2bef2e12d3a5fa5ac42535da709bfe9d337fdaa1 Reviewed-on: https://chromium-review.googlesource.com/892181Reviewed-by:
Philippe Hamel <hamelphi@chromium.org> Reviewed-by:
Roger McFarlane <rogerm@chromium.org> Commit-Queue: Roger McFarlane <rogerm@chromium.org> Cr-Commit-Position: refs/heads/master@{#532983}
-
Steve Kobes authored
These weren't caught in http://crrev.com/532638, but they are failing on win7_chromium_rel_ng: https://ci.chromium.org/buildbot/tryserver.chromium.win/win7_chromium_rel_ng/91452 Bug: 711468 Change-Id: Iae4b8a5e5a56e96bc835556fe6cc5fcdf11dedac Reviewed-on: https://chromium-review.googlesource.com/893214Reviewed-by:
Stefan Zager <szager@chromium.org> Commit-Queue: Steve Kobes <skobes@chromium.org> Cr-Commit-Position: refs/heads/master@{#532982}
-
gogerald authored
Bug: 763417 Change-Id: Ia02e2e00f2e6a315ae16ee8de852e756a79c0de6 Reviewed-on: https://chromium-review.googlesource.com/891978Reviewed-by:
Rouslan Solomakhin <rouslan@chromium.org> Commit-Queue: Ganggui Tang <gogerald@chromium.org> Cr-Commit-Position: refs/heads/master@{#532981}
-
Daniel Blakemore authored
A new mailto handler provider is being added to Chrome for iOS. This flag will wrap all invocations of the new provider during development. Cq-Include-Trybots: master.tryserver.chromium.mac:ios-simulator-cronet;master.tryserver.chromium.mac:ios-simulator-full-configs Change-Id: I9e24a5f256fc5e220120e173080a67301084b506 Reviewed-on: https://chromium-review.googlesource.com/876729Reviewed-by:
Peter Lee <pkl@chromium.org> Commit-Queue: Daniel Blakemore <danblakemore@google.com> Cr-Commit-Position: refs/heads/master@{#532980}
-
Nico Weber authored
Follow-up to https://chromium-review.googlesource.com/c/889942/ Also update a very old comment that I believe tries to say that the fallthrough is intentional: In https://websvn.kde.org/trunk/kdelibs/khtml/rendering/bidi.cpp?r1=56133&r2=56134&pathrev=65933 the code and comment looked like so: switch(dirCurrent) { ... case QChar::DirEN: if(status.lastStrong != QChar::DirAL) { // if last strong was AL change EN to AN ... } case QChar::DirAN: dirCurrent = QChar::DirAN; ... All the constants have been renamed several times, but the comment was touched just once after being added in https://websvn.kde.org/trunk/kdelibs/khtml/rendering/bidi.cpp?limit_changes=0&view=markup&pathrev=42417 in Mar 2000 (in a commit saying "merge of the new_rendering branch." -- I couldn't find where the new_rendering branch is. Probably in some CVS repo, but I didn't manage to find it) and that's in 56134 linked above, which changed it from // if last strong was AL change EN to AL to // if last strong was AL change EN to AN which looks like a typo fix. This CL was uploaded by git cl split. R=fmalita@chromium.org Bug: 177475 Change-Id: I55de0202f9a67436a4abb630291d1280ee489d4d Reviewed-on: https://chromium-review.googlesource.com/890430 Commit-Queue: Nico Weber <thakis@chromium.org> Reviewed-by:
Florin Malita <fmalita@chromium.org> Cr-Commit-Position: refs/heads/master@{#532979}
-
Dominick Ng authored
This CL removes several different app banner testing files, and replaces them by calling the necessary JavaScript setup code in one test file. For instance, all tests will no longer have the "appinstalled" title overwriting, as that will only be active when the appropriate setup JavaScript is run. This allows future tests to be more easily added, and reduces collisions when manipulating the page title. BUG=806923 Change-Id: Ia28837b8675a2e1fe329cb1df1be4798ddb03d02 Reviewed-on: https://chromium-review.googlesource.com/892182 Commit-Queue: Dominick Ng <dominickn@chromium.org> Reviewed-by:
Matt Giuca <mgiuca@chromium.org> Cr-Commit-Position: refs/heads/master@{#532978}
-
Sigurdur Asgeirsson authored
Bug: 799526 Change-Id: Id1f4e62914232e4cf37d550f6914690334f7ddee Reviewed-on: https://chromium-review.googlesource.com/893207Reviewed-by:
Robert Kaplow <rkaplow@chromium.org> Commit-Queue: Sigurður Ásgeirsson <siggi@chromium.org> Cr-Commit-Position: refs/heads/master@{#532977}
-
Stephen Chenney authored
The ImageBitmap constructor that takes pixel data and a width and height does not check for success of the skia MakeRasterCopy call, that will return null for a host of reasons. Add the check. R=junov@chromium.org BUG=805972 Null check in ImageBitmap:ImageBitmap Change-Id: I4cd8464b21c3b14488ae8a948864b091f17d006a Reviewed-on: https://chromium-review.googlesource.com/887678Reviewed-by:
Florin Malita <fmalita@chromium.org> Commit-Queue: Stephen Chenney <schenney@chromium.org> Cr-Commit-Position: refs/heads/master@{#532976}
-
Ian Clelland authored
The code checks both the runtime flag and the origin trial status, when the origin trial check implicitly checks the runtime flag first. Removing this antipattern will allow us to eventually mark the inappropriate checks as private, and reduce inadvertent bugs. Bug: 799949 Change-Id: I8af29c46a0f5eefacb366e87d378c86b7d4ac71f Reviewed-on: https://chromium-review.googlesource.com/891573Reviewed-by:
Jeremy Roman <jbroman@chromium.org> Commit-Queue: Ian Clelland <iclelland@chromium.org> Cr-Commit-Position: refs/heads/master@{#532975}
-
liberato@chromium.org authored
OnSurfaceDestroyed was unconditionally dereferencing |target_surface_bundle_|, but this can be cleared once the codec enters a terminal state. A null target bundle should just be ignored anyway. Bug: 806630 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: I612c8a0c95ec0e3b918789384f44ad3a01fd2559 Reviewed-on: https://chromium-review.googlesource.com/893638Reviewed-by:
Thomas Guilbert <tguilbert@chromium.org> Commit-Queue: Frank Liberato <liberato@chromium.org> Cr-Commit-Position: refs/heads/master@{#532974}
-
Jialiu Lin authored
For some rare cases, service_ in CheckClientDownloadRequest is already set to null when OnDownloadDestroyed is called, which as a result will make item_ still dangle there, and causes crash at the DCHECK(item_==NULL). Bug: 806785 Change-Id: I384f66ca94d6695d870a632d52b9be26a0f9a3a3 Reviewed-on: https://chromium-review.googlesource.com/891808Reviewed-by:
Jialiu Lin <jialiul@chromium.org> Reviewed-by:
Luke Z <lpz@chromium.org> Commit-Queue: Jialiu Lin <jialiul@chromium.org> Cr-Commit-Position: refs/heads/master@{#532973}
-
Ian Clelland authored
This fixes the problem where Feature Policy can be independently disabled through runtime flags (or chrome://flags), while the features which depend on it, such as FeaturePolicyForPermissions, remain enabled. Bug: 806362 Change-Id: Ied92dc74022a02f22a11c3ce68bd62c1aa98eaae Reviewed-on: https://chromium-review.googlesource.com/893598Reviewed-by:
Jeremy Roman <jbroman@chromium.org> Commit-Queue: Ian Clelland <iclelland@chromium.org> Cr-Commit-Position: refs/heads/master@{#532972}
-
David Bokan authored
It looks like we sometimes recompute the root scroller on an HTMLFrameOwnerElement that's still connected to the DOM tree but doesn't have a ContentFrame. One way this can happen is through layouts caused from unload handlers. This CL adds an early out in ApplyRootScrollerProperties since a detached frame doesn't need it's properties reset. Bug: 805317 Change-Id: I6d285b10a69f74f05604bda5f654fca96fc35238 Reviewed-on: https://chromium-review.googlesource.com/887733 Commit-Queue: David Bokan <bokan@chromium.org> Reviewed-by:
Daniel Cheng <dcheng@chromium.org> Cr-Commit-Position: refs/heads/master@{#532971}
-
Peter Kotwicz authored
This CL: - Replaces calls to ExternalNavigationDelegateImpl#isSpecializedHandlerAvailable() with calls to #countSpecializedHandlers() because #isSpecializedHandlerAvailable() has just one caller - Replaces ChromeBrowserTestRule with NativeLibraryTestRule - Uses a mock context for both ContextUtils#getApplicationContext() and for TabRedirectHandler instead of just for TabRedirectHandler - Makes TestPackageManager#queryIntentActivities() and TestExternalNavigationDelegate#queryIntentActivities() use same implementation. - Changes tests to register relevant http:// and https:// handlers at the beginning of the test. This is a change of behavior. Previously, most URLs would be handled by the "foo" external app - Changes TestExternalNavigationDelegate#mQueryIntentOverride from tristate to boolean. The "true behavior" was nuked but the "false behavior" was preserved. BUG=None Change-Id: If212afde6e4c12d4f08576c4bd26392b8732bc09 Reviewed-on: https://chromium-review.googlesource.com/891744 Commit-Queue: Peter Kotwicz <pkotwicz@chromium.org> Reviewed-by:
Maria Khomenko <mariakhomenko@chromium.org> Cr-Commit-Position: refs/heads/master@{#532970}
-
Victor Costan authored
Our SQLite VFS [1] wrapper implementation unconditionally wraps xCurrentTime(). However, when SQLite is compiled with the SQLITE_OMIT_DEPRECATED option [2], xCurrentTime() is null for the VFS implementations that ship with SQLite, because xCurrentTime() was deprecated in favor of xCurrentTimeInt64(). This CL replaces the unconditional wrapper with a conditional wrapper. This ensures that our VFS wrapper exposes an xCurrentTimeInt64() implementation when used with a modern SQLite library (like the one bundled with Chromium), and an xCurrentTime() implementation when using an older SQLite version (which might be the case when using the library bundled with the operating system). [1] https://www.sqlite.org/c3ref/vfs.html [2] https://www.sqlite.org/compile.html#omit_deprecated Bug: 807093 Change-Id: Id26cc4517a1a23692e7860ed620348c027db990f Reviewed-on: https://chromium-review.googlesource.com/892484Reviewed-by:
Chris Mumford <cmumford@chromium.org> Commit-Queue: Victor Costan <pwnall@chromium.org> Cr-Commit-Position: refs/heads/master@{#532969}
-
Devlin Cronin authored
The i18n custom bindings create v8 values from results for language detection, but do so in an inefficient way (constructing a base::Value and then converting to v8::Value). Update this to use gin::DataObjectBuilder to construct the v8 value directly, and, while we're at it, use some C++11 awesomeness in movable types. Bug: None Change-Id: I80597627b0b3803e50ecbe34009682161d178c7d Reviewed-on: https://chromium-review.googlesource.com/892202Reviewed-by:
Jeremy Roman <jbroman@chromium.org> Commit-Queue: Devlin <rdevlin.cronin@chromium.org> Cr-Commit-Position: refs/heads/master@{#532968}
-
Nate Chapin authored
Change-Id: I26c2a054b9f1e5eb076acd677e1223058825f6d6 Bug: 803416 Test: fast/loader/document-open-iframe-then-detach.html Change-Id: I26c2a054b9f1e5eb076acd677e1223058825f6d6 Reviewed-on: https://chromium-review.googlesource.com/887298Reviewed-by:
Mike West <mkwst@chromium.org> Commit-Queue: Nate Chapin <japhet@chromium.org> Cr-Commit-Position: refs/heads/master@{#532967}
-
Stephen McGruer authored
Change-Id: Idf6dd882d2d50b1ec349f245d76717553ee266b3 Reviewed-on: https://chromium-review.googlesource.com/882002Reviewed-by:
Robert Flack <flackr@chromium.org> Commit-Queue: Stephen McGruer <smcgruer@chromium.org> Cr-Commit-Position: refs/heads/master@{#532966}
-
Hidy Han authored
Bug: 724160 Change-Id: I9ede628f60f2ff32d09bcec9ef365db9246f4c98 Reviewed-on: https://chromium-review.googlesource.com/871046 Commit-Queue: Hidy Han <hidyhan@chromium.org> Reviewed-by:
Lei Zhang <thestig@chromium.org> Reviewed-by:
Mustafa Emre Acer <meacer@chromium.org> Reviewed-by:
Eric Seckler <eseckler@chromium.org> Reviewed-by:
Dmitry Gozman <dgozman@chromium.org> Cr-Commit-Position: refs/heads/master@{#532965}
-
https://webrtc.googlesource.com/src.git/+log/79d331b09102..cf30d8b1ec22 $ git log 79d331b09..cf30d8b1e --date=short --no-merges --format='%ad %ae %s' Created with: roll-dep src/third_party/webrtc BUG=NONE,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: I30491d79540eafe636e555734277e417854399fb Reviewed-on: https://chromium-review.googlesource.com/893525Reviewed-by:
webrtc-chromium-autoroll <webrtc-chromium-autoroll@skia-buildbots.google.com.iam.gserviceaccount.com> Commit-Queue: webrtc-chromium-autoroll <webrtc-chromium-autoroll@skia-buildbots.google.com.iam.gserviceaccount.com> Cr-Commit-Position: refs/heads/master@{#532964}
-
Steven Bennetts authored
There is an edge case where SortNetworkList may destroy the default network, causing a crash. This CL fixes that. Bug: 805322 Change-Id: I3d880dacc567babea08d7b9f21c5c8759e290f83 Reviewed-on: https://chromium-review.googlesource.com/883971 Commit-Queue: Steven Bennetts <stevenjb@chromium.org> Reviewed-by:
Toni Barzic <tbarzic@chromium.org> Cr-Commit-Position: refs/heads/master@{#532963}
-
Robert Ogden authored
Bug: 807049 Change-Id: Ia223724629db4f04a676a023a4dbda365a0dbba3 Reviewed-on: https://chromium-review.googlesource.com/893719Reviewed-by:
Tarun Bansal <tbansal@chromium.org> Commit-Queue: Robert Ogden <robertogden@google.com> Cr-Commit-Position: refs/heads/master@{#532962}
-
https://pdfium.googlesource.com/pdfium.git/+log/af9f83656f98..1917cdd8c90b $ git log af9f83656..1917cdd8c --date=short --no-merges --format='%ad %ae %s' 2018-01-30 dsinclair Cleanup some CPDFSDK_Widget XFA code. Created with: roll-dep src/third_party/pdfium The AutoRoll server is located here: https://pdfium-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=dsinclair@chromium.org Change-Id: Ic7870559e0596ac43919bf0732a649998e138926 Reviewed-on: https://chromium-review.googlesource.com/893599Reviewed-by:
pdfium-chromium-autoroll <pdfium-chromium-autoroll@skia-buildbots.google.com.iam.gserviceaccount.com> Commit-Queue: pdfium-chromium-autoroll <pdfium-chromium-autoroll@skia-buildbots.google.com.iam.gserviceaccount.com> Cr-Commit-Position: refs/heads/master@{#532961}
-
Helen Li authored
The file is not named correctly. This CL uses the correct name suffix. Bug: 721401 Change-Id: I9c0976bc802cee9f3c13141e6edb0b4ba81de97a Reviewed-on: https://chromium-review.googlesource.com/891440Reviewed-by:
Daniel Cheng <dcheng@chromium.org> Commit-Queue: Helen Li <xunjieli@chromium.org> Cr-Commit-Position: refs/heads/master@{#532960}
-