• Hans Wennborg's avatar
    Reland "Fixed SDK lookup for non C:\ Windows installation." · 2e7257da
    Hans Wennborg authored
    This reverts commit 864136ef.
    
    Reason for revert:
    While it seems this did trigger the cdb failures in crbug.com/846313,
    it was not the root cause and reverting it didn't actually help.
    
    Original change's description:
    > Revert "Fixed SDK lookup for non C:\ Windows installation."
    > 
    > This reverts commit 57fd44c7.
    > 
    > Reason for revert: suspect for messing up symbols on Win7 bot
    >                    @ crbug.com/846313
    > 
    > Original change's description:
    > > Fixed SDK lookup for non C:\ Windows installation.
    > >
    > > Bug: None
    > > Change-Id: Ia0aa186d1d39b1beac8ce0152683f774ad5d2eaf
    > > Reviewed-on: https://chromium-review.googlesource.com/1066065
    > > Reviewed-by: Jochen Eisinger <jochen@chromium.org>
    > > Commit-Queue: Jochen Eisinger <jochen@chromium.org>
    > > Cr-Commit-Position: refs/heads/master@{#561063}
    > 
    > TBR=jochen@chromium.org,yura.yaroshevich@gmail.com
    > 
    > # Not skipping CQ checks because original CL landed > 1 day ago.
    > 
    > Bug: 846313
    > Change-Id: Id93a965aea5555961c539d47e05e79410894eff8
    > Reviewed-on: https://chromium-review.googlesource.com/1072307
    > Commit-Queue: Gabriel Charette <gab@chromium.org>
    > Reviewed-by: Gabriel Charette <gab@chromium.org>
    > Cr-Commit-Position: refs/heads/master@{#561609}
    
    TBR=gab@chromium.org,jochen@chromium.org,yura.yaroshevich@gmail.com
    
    Bug: 846313
    Change-Id: I9955a47bbe8a81a0dcf7befebfc422560f8a1cc4
    Reviewed-on: https://chromium-review.googlesource.com/1073314Reviewed-by: default avatarHans Wennborg <hans@chromium.org>
    Commit-Queue: Hans Wennborg <hans@chromium.org>
    Cr-Commit-Position: refs/heads/master@{#561873}
    2e7257da
AUTHORS 36.7 KB