Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(831)

Issue 6688018: Remove old comment. (Closed)

Created:
9 years, 9 months ago by apatrick_chromium
Modified:
9 years, 7 months ago
Reviewers:
Zhenyao Mo
CC:
chromium-reviews, jam
Visibility:
Public.

Description

Remove old comment. TEST=none BUG=none Committed: http://src.chromium.org/viewvc/chrome?view=rev&revision=78629

Patch Set 1 #

Unified diffs Side-by-side diffs Delta from patch set Stats (+0 lines, -2 lines) Patch
M content/renderer/ggl.cc View 1 chunk +0 lines, -2 lines 0 comments Download

Messages

Total messages: 2 (0 generated)
apatrick_chromium
The function has worked for over a year!
9 years, 9 months ago (2011-03-17 23:27:26 UTC) #1
Zhenyao Mo
9 years, 9 months ago (2011-03-17 23:28:57 UTC) #2
LGTM

On Thu, Mar 17, 2011 at 4:27 PM,  <apatrick@chromium.org> wrote:
> Reviewers: Zhenyao Mo,
>
> Message:
> The function has worked for over a year!
>
> Description:
> Remove old comment.
>
> TEST=none
> BUG=none
>
> Please review this at http://codereview.chromium.org/6688018/
>
> SVN Base: svn://chrome-svn/chrome/trunk/src/
>
> Affected files:
>  M     content/renderer/ggl.cc
>
>
> Index: content/renderer/ggl.cc
> ===================================================================
> --- content/renderer/ggl.cc     (revision 78611)
> +++ content/renderer/ggl.cc     (working copy)
> @@ -102,8 +102,6 @@
>   static bool MakeCurrent(Context* context);
>
>   // Display all content rendered since last call to SwapBuffers.
> -  // TODO(apatrick): support rendering to browser window. This function is
> -  // not useful at this point.
>   bool SwapBuffers();
>
>   // Create a hardware accelerated video decoder associated with this
> context.
>
>
>

Powered by Google App Engine
This is Rietveld 408576698