New issue
Advanced search Search tips

Issue 647689 link

Starred by 1 user

Issue metadata

Status: WontFix
Owner: ----
Closed: Aug 2017
EstimatedDays: ----
NextAction: ----
OS: ----
Pri: 2
Type: Bug



Sign in to add a comment

Last two lines of a diff are duplicated when showing context

Project Member Reported by avayvod@chromium.org, Sep 16 2016

Issue description

https://codereview.chromium.org/2301823002

Expand MediaControls.cpp, scroll to the bottom of it and click on the last Show context link.

I see:

        m_durationDisplay.get(),
    };
        m_durationDisplay.get(),
    };

Where it doesn't show like this in the raw diff, CL compiles. Should just be:

        m_durationDisplay.get(),
    };

See screenshot attached.
 
duplicatediff.png
51.9 KB View Download
Components: Infra>Codereview>Rietveld
Components: -Infra>Codereview>Rietveld Infra>Codereview>Rietveld>PolymerUI
Status: Available (was: Untriaged)
We are moving to PolyGerrit (https://polygerrit.appspot.com) so this is unlikely to be fixed.
Status: WontFix (was: Available)
Rietveld is deprecated in favor of Gerrit.

Sign in to add a comment