Delete MaskImageChanged, VideoChanged, and FullScreenChanged

Delete MaskImageChanged, VideoChanged, and FullScreenChanged

* MaskImageChanged: No one did anything based on this notification.
* VideoChanged: There's only want caller and it's simpler to just
  setNeedsCompositingUpdate, which does the same thing.
* FullScreenChanged: No one ever sent this notification.

Committed: https://src.chromium.org/viewvc/blink?view=rev&revision=175972

[ojan, 2014-06-11 18:22:15]

https://codereview.chromium.org/334463002/diff/1/Source/core/rendering/Render...
File Source/core/rendering/RenderVideo.cpp (right):

https://codereview.chromium.org/334463002/diff/1/Source/core/rendering/Render...
Source/core/rendering/RenderVideo.cpp:213:
videoElement()->setNeedsCompositingUpdate();
This loses the m_compositedLayerMapping->contentChanged(changeType) call. Is
that OK?

[ojan, 2014-06-11 18:22:57]

On 2014/06/11 at 18:22:15, ojan wrote:
>
https://codereview.chromium.org/334463002/diff/1/Source/core/rendering/Render...
> File Source/core/rendering/RenderVideo.cpp (right):
> 
>
https://codereview.chromium.org/334463002/diff/1/Source/core/rendering/Render...
> Source/core/rendering/RenderVideo.cpp:213:
videoElement()->setNeedsCompositingUpdate();
> This loses the m_compositedLayerMapping->contentChanged(changeType) call. Is
that OK?

Ah. It looks like that does nothing for this change type.

[ojan, 2014-06-11 18:23:00]

lgtm

[commit-bot: I haz the power, 2014-06-11 18:23:57]

CQ is trying da patch. Follow status at
 https://chromium-status.appspot.com/cq/abarth@chromium.org/334463002/1

[commit-bot: I haz the power, 2014-06-11 19:01:27]

Change committed as 175972

[abarth-chromium, 2014-06-11 19:21:37]

On 2014/06/11 at 18:22:57, ojan wrote:
> Ah. It looks like that does nothing for this change type.

Correct!