• danakj's avatar
    Move display_mode and renderer_wide_named_frame_lookup_ to constructor · 73dd303e
    danakj authored
    Pass the DisplayMode to the RenderWidget constructor instead of setting
    it through set_display_mode() in RenderViewImpl::Initialize(). This
    makes the chosen DisplayMode explicit for all creators of RenderWidget
    instead of having a default value.
    
    Also store the renderer_wide_named_frame_lookup_ in RenderViewImpl
    directly in the constructor instead of in Initialize() since it does
    not need to be there (does not make use of the vtable or anything
    funky. Make the member const as a result.
    
    R=ajwong@chromium.org, avi@chromium.org
    
    Bug: 419087
    Change-Id: I8d4354d770df8abf5273935412a3d28a98017434
    Reviewed-on: https://chromium-review.googlesource.com/1153592
    Commit-Queue: Daniel Cheng <dcheng@chromium.org>
    Reviewed-by: default avatarAvi Drissman <avi@chromium.org>
    Reviewed-by: default avatarDaniel Cheng <dcheng@chromium.org>
    Cr-Commit-Position: refs/heads/master@{#578930}
    73dd303e
render_widget_fullscreen_pepper.cc 14.2 KB