- 24 May, 2018 40 commits
-
-
Moe Ahmadi authored
Bug: 841758 Change-Id: Ice6932221f401e20251926fe2cb41ba6f9061f52 Reviewed-on: https://chromium-review.googlesource.com/1070622Reviewed-by:
Jared Saul <jsaul@google.com> Reviewed-by:
Jesse Doherty <jwd@chromium.org> Commit-Queue: Moe Ahmadi <mahmadi@chromium.org> Cr-Commit-Position: refs/heads/master@{#561534}
-
Dominic Mazzoni authored
Bug: 845143, 843511 Change-Id: I87afc7235692c52502048dcf7ae6036ae8551e51 Reviewed-on: https://chromium-review.googlesource.com/1067491Reviewed-by:
Lei Zhang <thestig@chromium.org> Reviewed-by:
Dominic Mazzoni <dmazzoni@chromium.org> Reviewed-by:
Katie Dektar <katie@chromium.org> Commit-Queue: Dominic Mazzoni <dmazzoni@chromium.org> Cr-Commit-Position: refs/heads/master@{#561533}
-
Moe Ahmadi authored
Bug: 844472 Change-Id: I3a90de7feef9f7f0829e17110f51c5376bc4df1b Reviewed-on: https://chromium-review.googlesource.com/1070514Reviewed-by:
Jared Saul <jsaul@google.com> Reviewed-by:
Jesse Doherty <jwd@chromium.org> Commit-Queue: Moe Ahmadi <mahmadi@chromium.org> Cr-Commit-Position: refs/heads/master@{#561532}
-
v.paturi authored
The change to AttributeModificationParams (http://crrev.com/2623513005) has changed the use of the local variable |value| to params.new_value without noticing that the local variable was shadowing the function parameter. Since AltText() looks at both alt and text attributes, and we don't know which of the two is being parsed in this block, we should compare text->textContent to AltText(), and not params.new_value. The local variable |value| is being renamed to a more appropriate |alt_text_content|.This variable containing AltText() can be used for both comparison and setting which will save one extra function call. Bug: None Change-Id: I7f27ab77f1877c6e405aa41b4e912d826ace0fbf Reviewed-on: https://chromium-review.googlesource.com/1049425 Commit-Queue: Prashant Nevase <prashant.n@samsung.com> Reviewed-by:
Steve Kobes <skobes@chromium.org> Reviewed-by:
Ian Clelland <iclelland@chromium.org> Reviewed-by:
Prashant Nevase <prashant.n@samsung.com> Cr-Commit-Position: refs/heads/master@{#561531}
-
Jérôme Lebel authored
In the unified user consent dialog, if there is no Google account on the device, then the Settings should not be shown since the user cannot sign-in yet. Once an account is added the Settings link should visible and usable. A DCHECK and a comment has been added for -[UnifiedConsentViewController updateIdentityPickerViewWithAvatar:] to make sure -[UnifiedConsentViewController updateIdentityPickerViewWithUserFullName: email:] is always called before to set an avatar to the identity view. With the Settings link: https://drive.google.com/open?id=1NjIsfv47GP_y2yMtxAvQcqVqGF-20USb Without the Settings link: https://drive.google.com/open?id=1pD0QFHNjki1OoC1ztW6CgPfW6iLWOdZf Bug: 827072 Cq-Include-Trybots: luci.chromium.try:ios-simulator-full-configs;master.tryserver.chromium.mac:ios-simulator-cronet Change-Id: Ia4c90c5e85790f0b0b3baa8974a7abc87c1dbf26 Reviewed-on: https://chromium-review.googlesource.com/1070663 Commit-Queue: Jérôme Lebel <jlebel@chromium.org> Reviewed-by:
Gauthier Ambard <gambard@chromium.org> Cr-Commit-Position: refs/heads/master@{#561530}
-
Sigurdur Asgeirsson authored
This allows mocking it for testing the upcoming PerformanceMeasurementManager, which will be a RenderProcessProbe user. Bug: 755840 Change-Id: I7188529a106eb66044d71a514e26a6d6bd0b65ae Reviewed-on: https://chromium-review.googlesource.com/1071725Reviewed-by:
François Doray <fdoray@chromium.org> Commit-Queue: Sigurður Ásgeirsson <siggi@chromium.org> Cr-Commit-Position: refs/heads/master@{#561529}
-
Bence Béky authored
Do not check if callback is null in test-only AsyncSocket subclasses in RunCallback(). If the method was supposed to handle the case of null callback, it should be called MaybeRunCallback() instead. Add DCHECKs on callback at call sites to document expectations. Also, avoid double move when unnecessary. Also, fix GCMConnectionHandlerImplTests. Calling MockTCPClientSocket::Connect() with null callback on ASYNC mock data is not consistent with the async idiom used in //net. Sure it didn't use to blow up, because MockClientSocket::RunCallback() special-cased null callbacks. :( Also, use make_unique instead of new throughout google_apis/gcm/engine/connection_handler_impl_unittest.cc. Bug: 807724 Change-Id: I3adb8d2a09c87fa004b1b4e7a8cf6f0edff064a6 Reviewed-on: https://chromium-review.googlesource.com/1064193 Commit-Queue: Bence Béky <bnc@chromium.org> Reviewed-by:
Eric Roman <eroman@chromium.org> Reviewed-by:
Nicolas Zea <zea@chromium.org> Cr-Commit-Position: refs/heads/master@{#561528}
-
Guido Urdaneta authored
std::function is banned by the style guide. No behavior change intended. Bug: 846327 Change-Id: I7bd3c0bf082147491e5784626a8f74beb1c06719 Reviewed-on: https://chromium-review.googlesource.com/1071517Reviewed-by:
Max Morin <maxmorin@chromium.org> Commit-Queue: Guido Urdaneta <guidou@chromium.org> Cr-Commit-Position: refs/heads/master@{#561527}
-
Chris Nardi authored
A bug was reported where ::first-line broke ::placeholder styling, which seems to have been fixed in https://crrev.com/550589. However, we don't currently have a test for this bug. Add in a test to make sure we don't regress with this bug. Bug: 806247 Change-Id: I5bbcda1ccf5b3ea08a7b66a68071a802f42e9228 Reviewed-on: https://chromium-review.googlesource.com/1071126 Commit-Queue: Chris Nardi <cnardi@chromium.org> Reviewed-by:
Rune Lillesveen <futhark@chromium.org> Cr-Commit-Position: refs/heads/master@{#561526}
-
Sahel Sharify authored
This cl adds new metrics for GSU events that are generated from a touchscreen fling progress. We want to keep the latency of these GSUs separate from GSU events generated from touch move events since the latency for handling the GSU events from fling doesn't include the time needed for handling a touch event. Bug: 249063 Test: RenderWidgetHostLatencyTrackerTest.TestInertialToScrollHistograms Change-Id: Iba4664d19e5218cae4886ee543b709d4c2152fa4 Reviewed-on: https://chromium-review.googlesource.com/1062806Reviewed-by:
Steven Holte <holte@chromium.org> Reviewed-by:
Ken Buchanan <kenrb@chromium.org> Reviewed-by:
Navid Zolghadr <nzolghadr@chromium.org> Reviewed-by:
Timothy Dresser <tdresser@chromium.org> Commit-Queue: Sahel Sharifymoghaddam <sahel@chromium.org> Cr-Commit-Position: refs/heads/master@{#561525}
-
Ethan Kuefner authored
This CL switches the v8.browsing_* benchmarks to the histogram pipeline. NOTRY=true Bug: 744736 Change-Id: Ied3d2bc472251e875e45b2d932d2951ff1cba06b Reviewed-on: https://chromium-review.googlesource.com/1048849 Commit-Queue: Ethan Kuefner <eakuefner@chromium.org> Reviewed-by:
Emily Hanley <eyaich@chromium.org> Cr-Commit-Position: refs/heads/master@{#561524}
-
Elly Fong-Jones authored
This is a new name that seems to alias the existing Turkish layout. Bug: 845389 Change-Id: Ie7cfb42589644ff410f044d1420fbeb196de2289 Reviewed-on: https://chromium-review.googlesource.com/1071727Reviewed-by:
Robert Sesek <rsesek@chromium.org> Commit-Queue: Elly Fong-Jones <ellyjones@chromium.org> Cr-Commit-Position: refs/heads/master@{#561523}
-
Aaron Leventhal authored
This code does not have tests and is likely unnecessary. HandleActiveDecendant is called on the item with @aria-activedescendant, therefore to invalidate something, followingthe forward relation is correct. Using the reverse relation would make sense if something was pointing at the element with @aria-activedescendant that needed updating. TBR=nektar@chromium.org NOTRY=true Bug: None Change-Id: Ifc2e3d71a2089e5fbeb58c1bc6e020811ad9b1d6 Reviewed-on: https://chromium-review.googlesource.com/1070806 Commit-Queue: Aaron Leventhal <aleventhal@chromium.org> Reviewed-by:
Aaron Leventhal <aleventhal@chromium.org> Cr-Commit-Position: refs/heads/master@{#561522}
-
Scott Violet authored
This is needed as KeyEvents are sent directly to the focused window, where as LocatedEvents consider the WindowTargeter of each window. BUG=837692 TEST=covered by tests Change-Id: I935aaa1aaea6a8ea0339e99cc076ee7d459d9034 Reviewed-on: https://chromium-review.googlesource.com/1070857 Commit-Queue: Scott Violet <sky@chromium.org> Reviewed-by:
James Cook <jamescook@chromium.org> Cr-Commit-Position: refs/heads/master@{#561521}
-
Justin Cohen authored
Bug: 846337 Cq-Include-Trybots: luci.chromium.try:ios-simulator-full-configs;master.tryserver.chromium.mac:ios-simulator-cronet Change-Id: I5820f2715e41f9e13430a939a9d61cdca921bb70 Reviewed-on: https://chromium-review.googlesource.com/1071692Reviewed-by:
Stepan Khapugin <stkhapugin@chromium.org> Commit-Queue: Justin Cohen <justincohen@chromium.org> Cr-Commit-Position: refs/heads/master@{#561520}
-
Brian White authored
Bug: 766909 Change-Id: Idd0cbfc8755ee7d0d26d77b6c534ceddc9422bde Reviewed-on: https://chromium-review.googlesource.com/1067610Reviewed-by:
Robert Kaplow <rkaplow@chromium.org> Commit-Queue: Brian White <bcwhite@chromium.org> Cr-Commit-Position: refs/heads/master@{#561519}
-
Arthur Hemery authored
Currently tests that check that a RFH tried to commit in unit tests used MockRenderProcessHost::did_frame_commit_navigation(). This was set by intercepting the interface commit using TestRenderFrameHost::NavigationInterceptor. This is problematic because in the future the RFH will not be the one holding the interface directly, making it harder to use this mechanism. Instead, we just check that the RFH has a navigation request, which is equivalent to it trying to commit. We also remove TestRenderFrameHost::pending_commit() that was never used and now redundant. Bug: 784904 Change-Id: I7ea3c1410d70f9e164ea700a257e167d6ce3ae8c Reviewed-on: https://chromium-review.googlesource.com/1071608Reviewed-by:
Camille Lamy <clamy@chromium.org> Commit-Queue: Arthur Hemery <ahemery@chromium.org> Cr-Commit-Position: refs/heads/master@{#561518}
-
Kyle Horimoto authored
ConnectionMedium represents the medium through which a connection is made, which currently only includes BLE, but will eventually include GCM. ConnectionDetails represents a type of connection to a given device, and includes the device's ID and the ConnectionMedium. ConnectionDetails will be used to track which connections over which mediums are active or pending at any given time. This CL also integrates ConnectionDetail into two classes, ConnectionAttempt and MultiplexedChannel, in place of the IDs that they were previously using. The reason for this change is that clients of these classes need to categorize them via their ConnectionDetails. If we had stayed with an ID instead, these classes would then have to keep an extra map from ID to ConnectionDetails. This change eliminates this extra complexity in those client classes. Bug: 824568, 752273 Change-Id: Ia3532048c2df7c6a39cb231d21656774bc9263ab Reviewed-on: https://chromium-review.googlesource.com/1070710 Commit-Queue: Kyle Horimoto <khorimoto@chromium.org> Reviewed-by:
Ryan Hansberry <hansberry@chromium.org> Cr-Commit-Position: refs/heads/master@{#561517}
-
Rintaro Kuroiwa authored
- DXVAVideoDecodeAccelerator's constructor is changed to take a MediaLog pointer. - Added logs to check whether it is taking the D3D9 or D3D11 path. Test: Verified that the logs show up in media-internals when using MojoVideoDecoder. Cq-Include-Trybots: luci.chromium.try:android_optional_gpu_tests_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 Change-Id: I8894b48594ca4026195b42935f7947331effa29b Reviewed-on: https://chromium-review.googlesource.com/1070755Reviewed-by:
Xiaohan Wang <xhwang@chromium.org> Commit-Queue: Rintaro Kuroiwa <rkuroiwa@chromium.org> Cr-Commit-Position: refs/heads/master@{#561516}
-
Elly Fong-Jones authored
Two suites are Cocoa-only so they are marked as such. PopupBlockerBrowserTest has one broken test because of a known bug; the part of the test that fails is now skipped in Views mode on Mac. TBR=sky@chromium.org Bug: 845389,846329 Change-Id: I5643899196401bec3af23e5edcd28c1c865dbfaf Reviewed-on: https://chromium-review.googlesource.com/1071730 Commit-Queue: Elly Fong-Jones <ellyjones@chromium.org> Reviewed-by:
Elly Fong-Jones <ellyjones@chromium.org> Reviewed-by:
Charlie Harrison <csharrison@chromium.org> Cr-Commit-Position: refs/heads/master@{#561515}
-
Kevin Bailey authored
Fundamentally, change MULTI_VALUE_TYPE to FEATURE_VALUE_TYPE for the tab switch suggestions flag, since we don't need the 'button' option any more. Should be functionally equivalent. Bug: 780835 Change-Id: I56f2d20a577b0755935a5d3de6e9ae6f5c6ba8d9 Reviewed-on: https://chromium-review.googlesource.com/1069805 Commit-Queue: Kevin Bailey <krb@chromium.org> Reviewed-by:
Justin Donnelly <jdonnelly@chromium.org> Cr-Commit-Position: refs/heads/master@{#561514}
-
Daniel Park authored
If enabled: >Removes NTP Button from toolbar when on NTP >Tapping NTP Button leads to NTP Bug: 843639 Change-Id: I5023097f316f1548acef860225d1d7f863d85102 Reviewed-on: https://chromium-review.googlesource.com/1069212 Commit-Queue: Daniel Park <danielpark@chromium.org> Reviewed-by:
David Trainor <dtrainor@chromium.org> Reviewed-by:
Theresa <twellington@chromium.org> Cr-Commit-Position: refs/heads/master@{#561513}
-
Fredrik Söderquist authored
WebContentsObserver is in the 'content' namespace, not in the global. This causes a collision with a 'using' directive for the same class, which breaks Jumbo builds. TBR=avi@chromium.org,siggi@chromium.org Bug: 755840, 846328 Change-Id: I977106611b374c955bf3c5c161c8a52eea927698 Reviewed-on: https://chromium-review.googlesource.com/1071663Reviewed-by:
Fredrik Söderquist <fs@opera.com> Reviewed-by:
Avi Drissman <avi@chromium.org> Reviewed-by:
Sigurður Ásgeirsson <siggi@chromium.org> Commit-Queue: Fredrik Söderquist <fs@opera.com> Cr-Commit-Position: refs/heads/master@{#561512}
-
stkhapugin@chromium.org authored
Adds and uses new button icons for the location bar's trailing button. Bug: 821804 Cq-Include-Trybots: luci.chromium.try:ios-simulator-full-configs;master.tryserver.chromium.mac:ios-simulator-cronet Change-Id: Icf1ca41bb65610940545dee1e94a7e16ca33895f Reviewed-on: https://chromium-review.googlesource.com/1069356Reviewed-by:
Gauthier Ambard <gambard@chromium.org> Commit-Queue: Stepan Khapugin <stkhapugin@chromium.org> Cr-Commit-Position: refs/heads/master@{#561511}
-
Gauthier Ambard authored
Bug: 846302 Cq-Include-Trybots: luci.chromium.try:ios-simulator-full-configs;master.tryserver.chromium.mac:ios-simulator-cronet Change-Id: I959a7b1657531d7170a35df69dd38d210122ea1d Reviewed-on: https://chromium-review.googlesource.com/1071661 Commit-Queue: Gauthier Ambard <gambard@chromium.org> Reviewed-by:
Rohit Rao <rohitrao@chromium.org> Cr-Commit-Position: refs/heads/master@{#561510}
-
Rintaro Kuroiwa authored
- Add H264Parser::GetCurrentSubsamples to get the subsamples entries for the current NALU. Bug: 836557 Change-Id: I18ed7c0bb0f9706a1e5f66d4406c4321e284a363 Reviewed-on: https://chromium-review.googlesource.com/1033500 Commit-Queue: Rintaro Kuroiwa <rkuroiwa@chromium.org> Reviewed-by:
Xiaohan Wang <xhwang@chromium.org> Cr-Commit-Position: refs/heads/master@{#561509}
-
Jinsong Fan authored
Some code was still using the cc namespace in the viz component. This CL fixes that. No change to logic. Bug: 722935 Cq-Include-Trybots: luci.chromium.try:android_optional_gpu_tests_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 Change-Id: I534da76538feb6855aab008965e82d056583eaa4 Reviewed-on: https://chromium-review.googlesource.com/1070167 Commit-Queue: danakj <danakj@chromium.org> Reviewed-by:
danakj <danakj@chromium.org> Cr-Commit-Position: refs/heads/master@{#561508}
-
Ria Jiang authored
Nullcheck |window_->GetHost()| to avoid crashes in RenderWidgetHostViewAura::GetRootFrameSinkId(). Bug: 844447 Change-Id: Ic8d6df563a60be5463f1a90c3386f0f2360f49c4 Reviewed-on: https://chromium-review.googlesource.com/1071276Reviewed-by:
Ken Buchanan <kenrb@chromium.org> Commit-Queue: Ria Jiang <riajiang@chromium.org> Cr-Commit-Position: refs/heads/master@{#561507}
-
Shakti Sahu authored
This CL sets up the test class for the download infobar controller and adds a few tests. More extensive tests will be added in the subsequent CLs. Bug: 845730 Change-Id: I41e64887bd2da1330743ec4a32cdd08766bccb90 Reviewed-on: https://chromium-review.googlesource.com/1069787 Commit-Queue: Shakti Sahu <shaktisahu@chromium.org> Reviewed-by:
David Trainor <dtrainor@chromium.org> Cr-Commit-Position: refs/heads/master@{#561506}
-
Robert Ma authored
This is a top-level GitHub magic file for setting owners of code, which isn't useful to us in Chromium. And it seems to cause weird presubmit errors, so let's skip it. TBR=foolip No-Try: True Change-Id: I0f67ce81c62ac787eed2eda01f456eaa3166e981 Reviewed-on: https://chromium-review.googlesource.com/1071721 Commit-Queue: Robert Ma <robertma@chromium.org> Reviewed-by:
Robert Ma <robertma@chromium.org> Cr-Commit-Position: refs/heads/master@{#561505}
-
Brian Osman authored
I have an APNG with an embedded complex (A2B) profile that previously rendered as garbage, and now renders correctly. Bug: chromium:845931 Cq-Include-Trybots: master.tryserver.blink:linux_trusty_blink_rel;master.tryserver.chromium.linux:linux_layout_tests_slimming_paint_v2 Change-Id: Ia592323db44fba12b45e158fd6b1f3025ed17953 Reviewed-on: https://chromium-review.googlesource.com/1069614Reviewed-by:
Leon Scroggins <scroggo@chromium.org> Reviewed-by:
Florin Malita <fmalita@chromium.org> Commit-Queue: Brian Osman <brianosman@google.com> Cr-Commit-Position: refs/heads/master@{#561504}
-
Vasilii Sukhanov authored
Some PasswordManagerBrowserTestWithConditionalPopupViews are flaky. An additional check may help to detect the problem earlier in the test flow. Bug: 355145 Change-Id: Id0927d943d345b1c9a1d301fee6ee6f352719a52 Reviewed-on: https://chromium-review.googlesource.com/1071668Reviewed-by:
Vadym Doroshenko <dvadym@chromium.org> Commit-Queue: Vasilii Sukhanov <vasilii@chromium.org> Cr-Commit-Position: refs/heads/master@{#561503}
-
James Cook authored
Move ScreenForShutdown cleanup from AshTestBase to AshTestHelper. This makes sure it gets cleaned up after tests that don't use AshTestBase, such as BrowserWithTestWindowTest. This eliminates ~10 flakes from chrome's unit_tests suite, for example the screen_ash.cc DCHECKS in this run: https://chromium-swarm.appspot.com/task?id=3da674dacfa3c110&refresh=10&show_raw=1 Bug: none Test: ash_unittests, unit_tests Change-Id: I307329012599a198629e86ceeb352fb2039b9d32 Reviewed-on: https://chromium-review.googlesource.com/1070319Reviewed-by:
Mitsuru Oshima <oshima@chromium.org> Commit-Queue: James Cook <jamescook@chromium.org> Cr-Commit-Position: refs/heads/master@{#561502}
-
Lukasz Anforowicz authored
Bug: 809261 Change-Id: I5c26d594f09a3cbbf01a4c747968a67d4d078029 Reviewed-on: https://chromium-review.googlesource.com/1066816 Commit-Queue: Łukasz Anforowicz <lukasza@chromium.org> Reviewed-by:
Charlie Reis <creis@chromium.org> Reviewed-by:
Philip Jägenstedt <foolip@chromium.org> Cr-Commit-Position: refs/heads/master@{#561501}
-
Kenichi Ishibashi authored
Bug: N/A Cq-Include-Trybots: master.tryserver.chromium.linux:linux_mojo Change-Id: I91d65b32cbd51de3e9692d47bef9dfe104a208d5 Reviewed-on: https://chromium-review.googlesource.com/1070956Reviewed-by:
Kinuko Yasuda <kinuko@chromium.org> Reviewed-by:
John Abd-El-Malek <jam@chromium.org> Commit-Queue: John Abd-El-Malek <jam@chromium.org> Cr-Commit-Position: refs/heads/master@{#561500}
-
Alexander Alekseev authored
Bug: 823534 Cq-Include-Trybots: master.tryserver.chromium.linux:closure_compilation Change-Id: I289ae4f321e69da48af83d1fe2a727b45fc0b28a Reviewed-on: https://chromium-review.googlesource.com/1069126Reviewed-by:
Xiyuan Xia <xiyuan@chromium.org> Commit-Queue: Alexander Alekseev <alemate@chromium.org> Cr-Commit-Position: refs/heads/master@{#561499}
-
François Beaufort authored
This CL makes sure Picture-in-Picture Web API tests don't require kPictureInPicture feature flag. Bug: 806249 Change-Id: Ic2a95b6ded01813f97373f91f13bdad01ccc419a Reviewed-on: https://chromium-review.googlesource.com/1071569Reviewed-by:
Mounir Lamouri <mlamouri@chromium.org> Commit-Queue: François Beaufort <beaufort.francois@gmail.com> Cr-Commit-Position: refs/heads/master@{#561498}
-
Lutz Justen authored
Part 3 of Active Directory extension policy support. This CL wires ComponentActiveDirectoryPolicyService up with ActiveDirectoryPolicyManager and unlocks extension policy for Active Directory managed Chrome OS devices (Chromad). It is the final missing link to enable extension policy for Chromad. You can now do all the crazy things you've always been dreaming of like using the Certificate Enrollment for Chrome OS extension to mint client certificates in a PKI environment based off ADCS. BUG=chromium:784595 TEST=browser_tests --gtest_filter=ComponentActiveDirectory* Change-Id: I27aaba07aff9460779b18a01cc88cc338a78f381 Reviewed-on: https://chromium-review.googlesource.com/1057621Reviewed-by:
Alexander Alekseev <alemate@chromium.org> Reviewed-by:
Maksim Ivanov <emaxx@chromium.org> Reviewed-by:
Pavol Marko <pmarko@chromium.org> Commit-Queue: Lutz Justen <ljusten@chromium.org> Cr-Commit-Position: refs/heads/master@{#561497}
-
Francois Doray authored
The method returns true if the LifeycleUnit can be frozen. For now, a TabLifecycleUnit can be frozen unless: - It is already frozen. - It is visible. - It is playing audio, has played audio recently, is accessing the microphone, is accessing the camera, is being mirrored. Change-Id: I81157289264a4dbfd97d7a42ce78c5e4c2f19151 Reviewed-on: https://chromium-review.googlesource.com/1070323 Commit-Queue: François Doray <fdoray@chromium.org> Reviewed-by:
Chris Hamilton <chrisha@chromium.org> Cr-Commit-Position: refs/heads/master@{#561496}
-
Sigurdur Asgeirsson authored
Bug: 755840 Change-Id: If82a6287bacecb3e636b8d0d60cae83e5a3d35a3 Reviewed-on: https://chromium-review.googlesource.com/1070226Reviewed-by:
Robert Kaplow <rkaplow@chromium.org> Reviewed-by:
Chris Hamilton <chrisha@chromium.org> Reviewed-by:
Avi Drissman <avi@chromium.org> Commit-Queue: Sigurður Ásgeirsson <siggi@chromium.org> Cr-Commit-Position: refs/heads/master@{#561495}
-