• Daniel Bratell's avatar
    Remove unused OptionMenuModel from components/translate · 7fcfeb7c
    Daniel Bratell authored
    It was linked into some jumbo build experiments where it triggered
    link errors.
    
    If you include an object file, for instance options_menu_model.o, in
    an archive, for instance libtranslate.a, and nothing references that
    object file, then the linker won't care if options_menu_model.o is
    referencing all kinds of undefined symbols. It's a feature or quirk
    depending on who you asks.
    
    With jumbo compilation this options_menu_model.o would be combined
    with 7+ other .o files into translate_jumbo_1.o and since the .o file
    is no longer completely unused, the linker will get upset about the
    dangling references.
    
    So something that used to be "just" dead code compilation with
    dangling references becomes a linking error when compiling with jumbo.
    
    Change-Id: Id15f7e15804efe2fdf959c4db7fdade061477102
    Reviewed-on: https://chromium-review.googlesource.com/c/1256685
    Commit-Queue: Daniel Bratell <bratell@opera.com>
    Reviewed-by: default avatarTakashi Toyoshima <toyoshim@chromium.org>
    Cr-Commit-Position: refs/heads/master@{#596168}
    7fcfeb7c
BUILD.gn 3.67 KB