- 20 Feb, 2018 40 commits
-
-
https://chromium.googlesource.com/chromium/tools/depot_tools.git/+log/7707c8a10d46..64e33cba1794 $ git log 7707c8a10..64e33cba1 --date=short --no-merges --format='%ad %ae %s' 2018-02-20 jbudorick Fix gclient flattening of CIPD deps. Created with: roll-dep src/third_party/depot_tools The AutoRoll server is located here: https://depot-tools-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. TBR=agable@chromium.org Change-Id: I6fe6380c46a112fed2f67582080791898dc33d72 Reviewed-on: https://chromium-review.googlesource.com/927603 Commit-Queue: depot-tools-chromium-autoroll <depot-tools-chromium-autoroll@skia-buildbots.google.com.iam.gserviceaccount.com> Reviewed-by:
depot-tools-chromium-autoroll <depot-tools-chromium-autoroll@skia-buildbots.google.com.iam.gserviceaccount.com> Cr-Commit-Position: refs/heads/master@{#537933}
-
Krishna Govind authored
R=cmasso@chromium.org Cq-Include-Trybots: master.tryserver.chromium.linux:closure_compilation;master.tryserver.chromium.mac:ios-simulator-cronet;master.tryserver.chromium.mac:ios-simulator-full-configs Change-Id: I7b159991f463ea174c4bad8b93d99442b4c679c3 Reviewed-on: https://chromium-review.googlesource.com/927662Reviewed-by:
Krishna Govind <govind@chromium.org> Cr-Commit-Position: refs/heads/master@{#537932}
-
Adam Langley authored
Change-Id: I58c55695d01e7630977c4e901dea6587989040b0 Bug: none Reviewed-on: https://chromium-review.googlesource.com/920901Reviewed-by:
Balazs Engedy <engedy@chromium.org> Commit-Queue: Adam Langley <agl@chromium.org> Cr-Commit-Position: refs/heads/master@{#537931}
-
Ryan Hamilton authored
Merge internal change: 186297117 Change-Id: I4f4b9409d2760742cbdae5a71023a7626e0185ae Reviewed-on: https://chromium-review.googlesource.com/925686 Commit-Queue: Ryan Hamilton <rch@chromium.org> Reviewed-by:
Zhongyi Shi <zhongyi@chromium.org> Cr-Commit-Position: refs/heads/master@{#537930}
-
Michael Achenbach authored
References from client projects to removed methods have been removed in previous CLs. Bug: 756691 Change-Id: I405ec09301455884f00348aadf82e865e85df721 Reviewed-on: https://chromium-review.googlesource.com/925683Reviewed-by:
Bruce Dawson <brucedawson@chromium.org> Reviewed-by:
Robbie Iannucci <iannucci@chromium.org> Reviewed-by:
Nico Weber <thakis@chromium.org> Reviewed-by:
Dirk Pranke <dpranke@chromium.org> Commit-Queue: Michael Achenbach <machenbach@chromium.org> Cr-Commit-Position: refs/heads/master@{#537929}
-
Peter Boström authored
Highlights the BubbleIconView whenever the corresponding widget is visible. This was previously a one-off done for the bookmark star view. This addresses highlighting the translate icon, bookmark icon, zoom icon and others in a single place. This makes it less error prone and more future proof. BubbleIconView still has to be added as an observer of the bubble widget (but this is already required to remove button-press highlights when the widget closes). Bug: chromium:808273 Change-Id: Ibd4e14b4876be3e79ef8a85a0a9feade65aa5628 Reviewed-on: https://chromium-review.googlesource.com/905408Reviewed-by:
Scott Violet <sky@chromium.org> Reviewed-by:
Bret Sepulveda <bsep@chromium.org> Commit-Queue: Peter Boström <pbos@chromium.org> Cr-Commit-Position: refs/heads/master@{#537928}
-
Nick Harper authored
Change-Id: If7ff9c7f878374335c779456e88c1314005d9f12 Reviewed-on: https://chromium-review.googlesource.com/924684Reviewed-by:
Eric Lawrence <elawrence@chromium.org> Commit-Queue: Nick Harper <nharper@chromium.org> Cr-Commit-Position: refs/heads/master@{#537927}
-
Takumi Fujimoto authored
This CL makes the WiredDisplayMRP observe displays only after MediaRouter requests to query for sinks, so that we are not observing displays during startup and looking up the primary display, which is causing a crash in display-less Chrome OS. Bug: 803881 Change-Id: I751d1cd295fa3db1c1bdee65570ee4ab7ad8cc23 Reviewed-on: https://chromium-review.googlesource.com/924524 Commit-Queue: Takumi Fujimoto <takumif@chromium.org> Reviewed-by:
Derek Cheng <imcheng@chromium.org> Cr-Commit-Position: refs/heads/master@{#537926}
-
Victor Costan authored
https://chromium.googlesource.com/external/github.com/google/snappy.git/+log/b02bfa754ebf..ca37ab7fb9b7 $ git log b02bfa754..ca37ab7fb --date=short --no-merges --format='%ad %ae %s' 2018-02-02 jgorbe Ensure DecompressAllTags starts on a 32-byte boundary + 16 bytes. 2018-01-16 scrubbed Fix an incorrect analysis / comment in the "pattern doubling" code. 2018-01-04 costan Fix Travis CI configuration for OSX. 2017-12-21 chandlerc Rework a very hot, very sensitive part of snappy to reduce the number of instructions, the number of dynamic branches, and avoid a particular loop structure than LLVM has a very hard time optimizing for this particular case. 2017-12-20 costan Fix generated version number in open source release. Created with: roll-dep src/third_party/snappy/src Change-Id: I45f0f9e6aa7c81daef1172d39cef0a16814ccf20 Reviewed-on: https://chromium-review.googlesource.com/924573Reviewed-by:
Chris Mumford <cmumford@chromium.org> Commit-Queue: Victor Costan <pwnall@chromium.org> Cr-Commit-Position: refs/heads/master@{#537925}
-
Bailey Berro authored
This change implements MoveEntry on SmbProviderClient and FakeSmbProviderClient. BUG=chromium:757625 TEST=compiles Change-Id: Icf38fa220dbb993a86f12e8c5464b93b18f06c3d Reviewed-on: https://chromium-review.googlesource.com/923081Reviewed-by:
Zentaro Kavanagh <zentaro@chromium.org> Commit-Queue: Bailey Berro <baileyberro@chromium.org> Cr-Commit-Position: refs/heads/master@{#537924}
-
Tommy C. Li authored
TBR=sky@chromium.org Bug: 813759 Change-Id: I5896d323742aa3811d0a96a972d7ea4e7cd90769 Reviewed-on: https://chromium-review.googlesource.com/927454Reviewed-by:
Tommy Li <tommycli@chromium.org> Commit-Queue: Tommy Li <tommycli@chromium.org> Cr-Commit-Position: refs/heads/master@{#537923}
-
James Wallace-Lee authored
Allow webview to check damage in OnBeginFrame to prevent unnecessary draws. Change-Id: I45269b7ae6223358b756c523197b309d7bc5155e Reviewed-on: https://chromium-review.googlesource.com/868503Reviewed-by:
Bo <boliu@chromium.org> Commit-Queue: James Wallace-Lee <jamwalla@chromium.org> Cr-Commit-Position: refs/heads/master@{#537922}
-
Lambros Lambrou authored
Some limitations were added, because they were believed to mitigate the video-freezing problems with WebRTC connections. Now that the underlying problems have been addressed, these limitations can be removed. Max bitrate is increased to 100Mbps from 20Mbps, undoing crrev.com/a975bd60 Empty frames are sent every 2s instead of 200ms, to address crbug.com/803599. Empty frames were introduced in crrev.com/a6a3f1e8 Bug: 773549, 803599 Change-Id: I5c463d12d8935a63792656c7e79586bec0caed69 Reviewed-on: https://chromium-review.googlesource.com/907591 Commit-Queue: Lambros Lambrou <lambroslambrou@chromium.org> Reviewed-by:
Joe Downing <joedow@chromium.org> Cr-Commit-Position: refs/heads/master@{#537921}
-
spqchan authored
Bug: 794067 Change-Id: Ie2abc3d1b6b139d026508fc7ed2b20e5e57efedc Reviewed-on: https://chromium-review.googlesource.com/927451Reviewed-by:
Elly Fong-Jones <ellyjones@chromium.org> Commit-Queue: Sarah Chan <spqchan@chromium.org> Cr-Commit-Position: refs/heads/master@{#537920}
-
Donn Denman authored
Disables RTCRtpSenderTest.ReplaceTrackSetsTrack on Android and Linux due to flaky behavior. BUG=800465 TBR=sergeyu@chromium.org Change-Id: I1c0f3864d2d60b81b34a9718ead8e1c9f5c94cff Reviewed-on: https://chromium-review.googlesource.com/927535 Commit-Queue: Donn Denman <donnd@chromium.org> Reviewed-by:
Donn Denman <donnd@chromium.org> Cr-Commit-Position: refs/heads/master@{#537919}
-
Morten Stenshorne authored
UpdateOutOfFlowBlockLayout() needs to deal with both the actual containing block size, as specified by CSS (i.e. the padding box size), and the border-box size of the object that is the containing block of an absolutely positioned descendant. Some light cleanup. We always need the border sizes, so just calculate them once, unconditionally. Also, there seems no need for a special path for grid/flex, so I removed it. Some tests start to pass. All but one are still expected to fail in legacy. Cq-Include-Trybots: master.tryserver.chromium.linux:linux_layout_tests_layout_ng Change-Id: I85462d76aa9fa3d0105bf079e59f7c2a7a70f117 Reviewed-on: https://chromium-review.googlesource.com/926004Reviewed-by:
Aleks Totic <atotic@chromium.org> Commit-Queue: Morten Stenshorne <mstensho@chromium.org> Cr-Commit-Position: refs/heads/master@{#537918}
-
Kinuko Yasuda authored
Per https://groups.google.com/a/chromium.org/forum/#!topic/platform-architecture-dev/Ala6ZxFsFPQ TBR=falken@chromium.org,haraken@chromium.org,jam@chromium.org Bug: 809983 Change-Id: I80cd2e3fb8b73f157492ca7f6cf91d16b50c5234 Reviewed-on: https://chromium-review.googlesource.com/926462 Commit-Queue: Kinuko Yasuda <kinuko@chromium.org> Reviewed-by:
Kinuko Yasuda <kinuko@chromium.org> Cr-Commit-Position: refs/heads/master@{#537917}
-
Weiliang Chen authored
Root should only has damage when there is visible damage and has nothing to do with whether there are contributing layers. Damage with 0 quad is a valid case when only drawn layer just got removed. TBR=danakj@chromium.org Cq-Include-Trybots: master.tryserver.blink:linux_trusty_blink_rel;master.tryserver.chromium.android:android_optional_gpu_tests_rel Change-Id: I6c6d81b99bb626dd853cc6b8be0d1379ae3df3c5 Reviewed-on: https://chromium-review.googlesource.com/927349Reviewed-by:
danakj <danakj@chromium.org> Reviewed-by:
weiliangc <weiliangc@chromium.org> Commit-Queue: weiliangc <weiliangc@chromium.org> Cr-Commit-Position: refs/heads/master@{#537916}
-
Dominick Ng authored
This CL extracts the UI-independent logic from AppBannerInfoBarDelegateAndroid, and moves it to a new class, AppBannerUiDelegateAndroid. This will allow a new modal UI surface to be introduced for banners that uses the same app installation logic. BUG=811578 Change-Id: I0998b0086783d4c78f25c966917c40adb9638f7f Reviewed-on: https://chromium-review.googlesource.com/915442 Commit-Queue: Dominick Ng <dominickn@chromium.org> Reviewed-by:
Dominick Ng <dominickn@chromium.org> Reviewed-by:
Ben Wells <benwells@chromium.org> Reviewed-by:
Ted Choc <tedchoc@chromium.org> Cr-Commit-Position: refs/heads/master@{#537915}
-
Mike Dougherty authored
FindElementAtPoint asynchronously returns the DOM element details using postMessage which can be used to communicate with the native app from any document frame. Bug: 228355 Cq-Include-Trybots: master.tryserver.chromium.mac:ios-simulator-cronet;master.tryserver.chromium.mac:ios-simulator-full-configs Change-Id: I52c6e08f168824f0f458b97965e627d75a93ffa6 Reviewed-on: https://chromium-review.googlesource.com/918404 Commit-Queue: Mike Dougherty <michaeldo@chromium.org> Reviewed-by:
Eugene But <eugenebut@chromium.org> Cr-Commit-Position: refs/heads/master@{#537914}
-
Dominick Ng authored
This CL is a minor renaming CL with no functional changes. This prepares for the introduction of a new use for the app layout: app install banners for web and native apps. BUG=811578 Change-Id: Iff4201d160cb93bb2881d24a2f24cadf7fcd94bc Reviewed-on: https://chromium-review.googlesource.com/920125Reviewed-by:
Peter Kotwicz <pkotwicz@chromium.org> Reviewed-by:
Ted Choc <tedchoc@chromium.org> Commit-Queue: Dominick Ng <dominickn@chromium.org> Cr-Commit-Position: refs/heads/master@{#537913}
-
Steven Holte authored
Bug:812729 Change-Id: I113527a6c3c7d6ecc8e71f499b39cd7201e4bb73 Reviewed-on: https://chromium-review.googlesource.com/922687 Commit-Queue: Steven Holte <holte@chromium.org> Reviewed-by:
Robert Kaplow <rkaplow@chromium.org> Cr-Commit-Position: refs/heads/master@{#537912}
-
Dirk Pranke authored
We're close enough to turning off MSVC altogether that this doesn't make enough sense any more. R=jbudorick@chromium.org, hans@chromium.org BUG=777642 Change-Id: I64cdde1cbe777abd58fbdb8ee79e71e2b3e4cce7 Reviewed-on: https://chromium-review.googlesource.com/927456Reviewed-by:
John Budorick <jbudorick@chromium.org> Commit-Queue: Dirk Pranke <dpranke@chromium.org> Cr-Commit-Position: refs/heads/master@{#537911}
-
Fady Samuel authored
With this CL, we no longer kill the renderer when there is a surface invariants violation. Often that state can be recovered from on resize, for example or maybe even on the next frame. Bug: 791154, 672962 Change-Id: I21d0827b54932c636b015c96f3db97a3d2d58dac Reviewed-on: https://chromium-review.googlesource.com/926913Reviewed-by:
Saman Sami <samans@chromium.org> Commit-Queue: Fady Samuel <fsamuel@chromium.org> Cr-Commit-Position: refs/heads/master@{#537910}
-
Christopher Grant authored
This button now resides in the omnibox. BUG= R=ymalik Cq-Include-Trybots: master.tryserver.chromium.android:android_optional_gpu_tests_rel;master.tryserver.chromium.linux:linux_optional_gpu_tests_rel;master.tryserver.chromium.linux:linux_vr;master.tryserver.chromium.mac:mac_optional_gpu_tests_rel;master.tryserver.chromium.win:win_optional_gpu_tests_rel Change-Id: Icb3882f31e35655156374ac442cce03a13ab73cd Reviewed-on: https://chromium-review.googlesource.com/926767 Commit-Queue: Christopher Grant <cjgrant@chromium.org> Reviewed-by:
Ian Vollick <vollick@chromium.org> Reviewed-by:
Yash Malik <ymalik@chromium.org> Cr-Commit-Position: refs/heads/master@{#537909}
-
Andrew Grieve authored
Brings the finalize_apk step 19 seconds -> 4 seconds on my machine (for ChromePublic.apk). Also enables v2 signing of apks, which makes them install faster on N+ devices. Bug: 810890 Change-Id: I9fd52ed26c56472bd2894ace6f0516fc386143d6 Reviewed-on: https://chromium-review.googlesource.com/920651Reviewed-by:
John Budorick <jbudorick@chromium.org> Commit-Queue: agrieve <agrieve@chromium.org> Cr-Commit-Position: refs/heads/master@{#537908}
-
Allen Bauer authored
Bug: 813253 Change-Id: I11c496289a4e693b78595de76e1398f896643348 Reviewed-on: https://chromium-review.googlesource.com/927449 Commit-Queue: Allen Bauer <kylixrd@chromium.org> Reviewed-by:
Peter Kasting <pkasting@chromium.org> Cr-Commit-Position: refs/heads/master@{#537907}
-
Xianzhu Wang authored
The fields are used for SPv2 only, so disable comparison of them for SPv175. Bug: 813577 Cq-Include-Trybots: master.tryserver.blink:linux_trusty_blink_rel;master.tryserver.chromium.linux:linux_layout_tests_slimming_paint_v2 Change-Id: I09f808d7545e49edefdbffcff1bb4385563a47d9 Reviewed-on: https://chromium-review.googlesource.com/926911 Commit-Queue: Xianzhu Wang <wangxianzhu@chromium.org> Reviewed-by:
Stephen Chenney <schenney@chromium.org> Cr-Commit-Position: refs/heads/master@{#537906}
-
Alexander Alekseev authored
Chrome OS removes users when profiles are not loaded, and Chrome does not know the exact profile path until it is mounted. So we need to be able to remove entries from ProfileInfoCache based solely on AccountId. This Cl implements removal of cache entries given the AccountId and engages it on Chrome OS. Bug: 808125 Change-Id: I9f8e3a5f1004d119425b92a4bdc3d8011a526dd5 Reviewed-on: https://chromium-review.googlesource.com/905751Reviewed-by:
anthonyvd <anthonyvd@chromium.org> Reviewed-by:
Bernhard Bauer <bauerb@chromium.org> Commit-Queue: Alexander Alekseev <alemate@chromium.org> Cr-Commit-Position: refs/heads/master@{#537905}
-
Scott Chen authored
Bug: 810515 Change-Id: I18891fc67b1ff12017d6a56ac1e0acf8092bdfa5 Reviewed-on: https://chromium-review.googlesource.com/912549 Commit-Queue: Scott Chen <scottchen@chromium.org> Reviewed-by:
Demetrios Papadopoulos <dpapad@chromium.org> Cr-Commit-Position: refs/heads/master@{#537904}
-
Matt Mueller authored
Change-Id: Iaf9671db3799ddc0d305ef317238c1f45ba70809 Reviewed-on: https://chromium-review.googlesource.com/927527Reviewed-by:
Eric Roman <eroman@chromium.org> Commit-Queue: Matt Mueller <mattm@chromium.org> Cr-Commit-Position: refs/heads/master@{#537903}
-
v8-autoroll authored
Summary of changes available at: https://chromium.googlesource.com/v8/v8/+log/056e8283..6d2e57db 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: Idbc16d832d1f63a4f3aab420d4a0437ba2e106f3 Reviewed-on: https://chromium-review.googlesource.com/926766Reviewed-by:
v8 autoroll <v8-autoroll@chromium.org> Commit-Queue: v8 autoroll <v8-autoroll@chromium.org> Cr-Commit-Position: refs/heads/master@{#537902}
-
Scott Chen authored
This CL gives the sign-in row a more eye-catching banner style, and shrinks to the normal style after 10 impressions. Bug: 800981 Cq-Include-Trybots: master.tryserver.chromium.linux:closure_compilation Change-Id: Id3b55249d0eea22d6f426faf6ef863c2355f7e92 Reviewed-on: https://chromium-review.googlesource.com/912695 Commit-Queue: Scott Chen <scottchen@chromium.org> Reviewed-by:
Demetrios Papadopoulos <dpapad@chromium.org> Cr-Commit-Position: refs/heads/master@{#537901}
-
Ken Rockot authored
Because of some overlap in test coverage, this CL fixes two tangentially related features of the WebRequest API when the Network Service is enabled: - Registered declarative web and net request API rules elicit correct proxy establishment via WebRequestProxyingURLLoader, so both declarative APIs mostly work, modulo some cross-cutting features which affect all WebRequest behavior - onHeadersReceived handlers which modify response headers into a redirect are now handled properly To support the declarative WebRequest changes in particular, all rules registries now have a UI-thread cache. Registries for webviews, which did not have a UI-thread cache before this change, are ephemeral and do not persist in storage. Re-enables a bunch of tests. TBR=benwells@chromium.org Bug: 721414 Cq-Include-Trybots: master.tryserver.chromium.linux:linux_mojo Change-Id: I60aa27a3a44119f6119377b1f7ccac725b0678c7 Reviewed-on: https://chromium-review.googlesource.com/923432 Commit-Queue: Ken Rockot <rockot@chromium.org> Reviewed-by:
Karan Bhatia <karandeepb@chromium.org> Reviewed-by:
Reilly Grant <reillyg@chromium.org> Cr-Commit-Position: refs/heads/master@{#537900}
-
https://skia.googlesource.com/skia.git/+log/300cc5b5c1e1..ee78dbe529d9 $ git log 300cc5b5c..ee78dbe52 --date=short --no-merges --format='%ad %ae %s' 2018-02-20 caryclark update path docs online 2018-02-20 caryclark fix Rect doc 2018-02-20 reed hide complex versions of isOval and isRRect 2018-02-20 bsalomon Introduce GrColorType 2018-02-20 skcms-skia-autoroll Roll skia/third_party/externals/skcms/ 08d4cf2b3..fa14d14de (3 commits) Created with: roll-dep src/third_party/skia The AutoRoll server is located here: https://autoroll.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.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 TBR=djsollen@chromium.org Change-Id: Ie65c2646afb1dcad8daa0d1674e4ab01de911145 Reviewed-on: https://chromium-review.googlesource.com/927225Reviewed-by:
skia-chromium-autoroll <skia-chromium-autoroll@skia-buildbots.google.com.iam.gserviceaccount.com> Commit-Queue: skia-chromium-autoroll <skia-chromium-autoroll@skia-buildbots.google.com.iam.gserviceaccount.com> Cr-Commit-Position: refs/heads/master@{#537899}
-
Yaron Friedman authored
Found while tracking references to |onFinishGetGitmap| Seems like Pedro found some dead code earlier but missed this part BUG=651348 Change-Id: Ic93464abc91ae98dbf8eb9a92a641d162838b90e Reviewed-on: https://chromium-review.googlesource.com/921862 Commit-Queue: Yaron Friedman <yfriedman@chromium.org> Reviewed-by:
David Trainor <dtrainor@chromium.org> Cr-Commit-Position: refs/heads/master@{#537898}
-
Joe Downing authored
Thi change enables iq stanza logging to make debugging signaling issues in the wild easier. BUG= Change-Id: I14ab58fddcf37b350abe5ab5a1c398e43cb26336 Reviewed-on: https://chromium-review.googlesource.com/927348Reviewed-by:
Jamie Walch <jamiewalch@chromium.org> Commit-Queue: Joe Downing <joedow@chromium.org> Cr-Commit-Position: refs/heads/master@{#537897}
-
Albert Chaulk authored
Bug: b/72506886 Test: load browser sample as an overlay Change-Id: I69fd1e69e80f1c624508f77f745327acdb26e19c Reviewed-on: https://chromium-review.googlesource.com/926970Reviewed-by:
Luke Halliwell <halliwell@chromium.org> Commit-Queue: Albert Chaulk <achaulk@chromium.org> Cr-Commit-Position: refs/heads/master@{#537896}
-
Xiangjun Zhang authored
Add a UDP transport that can send/receive UDP packets through Network Service by connecting to a UDPSocket that implements network::mojom::UDPSocket interface. Bug: 734672 Change-Id: Ia2ccf456eea85394e01be919eee645684ba7625a TBR=jam@chromium.org Change-Id: Ia2ccf456eea85394e01be919eee645684ba7625a Reviewed-on: https://chromium-review.googlesource.com/910127 Commit-Queue: Xiangjun Zhang <xjz@chromium.org> Reviewed-by:
Christian Dullweber <dullweber@chromium.org> Reviewed-by:
Yuri Wiitala <miu@chromium.org> Cr-Commit-Position: refs/heads/master@{#537895}
-
Ryan Hamilton authored
It's actually unused if you at all the call sites. The two places that priority() is actually called it's explicitly calling down to the base class impl! (And then the unused methods in Handle) Change-Id: I6eb696636ae9346a0110271aeec3e8144d15e606 Reviewed-on: https://chromium-review.googlesource.com/926503Reviewed-by:
Zhongyi Shi <zhongyi@chromium.org> Commit-Queue: Ryan Hamilton <rch@chromium.org> Cr-Commit-Position: refs/heads/master@{#537894}
-