nsICanvasRenderingContextInternal.h currently has this at the end: >135 namespace mozilla { >136 namespace dom { >137 >138 } >139 } This was added in this cset, with something actually inside of that namespace: http://hg.mozilla.org/mozilla-central/diff/9feb8257666c/content/canvas/public/nsICanvasRenderingContextInternal.h ...but that something was removed here, leaving behind an empty namespace block: https://hg.mozilla.org/mozilla-central/diff/ab05eadb67c9/content/canvas/public/nsICanvasRenderingContextInternal.h

While we're at it, this header has a forward-decl for ipc::Shmem, which is no longer needed. That forward-decl was added here: http://hg.mozilla.org/mozilla-central/diff/7d65393c8c73/content/canvas/public/nsICanvasRenderingContextInternal.h ...and its usage was removed here: http://hg.mozilla.org/mozilla-central/diff/8eb26a77a90e/content/canvas/public/nsICanvasRenderingContextInternal.h

Actually, last patch doesn't compile, because we still have a vestigial implementation of the (long-since removed) Shmem-using function "Swap", in WebGLContext.h. (and it relied on the forward-declare that I'm removing) The vestigial "Swap()" implementations should be removed, since they were taken out of the API, as shown in the link at the end of comment 1. This patch updates WebGLContext.h to remove them, and also marks the adjacent function "Redraw" as MOZ_OVERRIDE, so that if it's ever removed from the API, we'll notice that we've got this vestigial version lying around and know to remove it. :)

Sanity-check Try run (builds only): https://tbpl.mozilla.org/?tree=Try&rev=a70b8a933ac1

Landed: https://hg.mozilla.org/integration/mozilla-inbound/rev/fb460ff43b6e