Location/size change notification when external rendering is enabled

webkit/media/android/webmediaplayer_android.cc

 // Copyright (c) 2012 The Chromium Authors. All rights reserved.
 // Use of this source code is governed by a BSD-style license that can be
 // found in the LICENSE file.
 
 #include "webkit/media/android/webmediaplayer_android.h"
 
 #include "base/command_line.h"
 #include "base/files/file_path.h"
 #include "base/logging.h"
 #include "cc/layers/video_layer.h"
@@ skipping 462 matching lines @@
 }
 
 void WebMediaPlayerAndroid::SetNeedsEstablishPeer(bool needs_establish_peer) {
   needs_establish_peer_ = needs_establish_peer;
 }
 
 void WebMediaPlayerAndroid::UpdatePlayingState(bool is_playing) {
   is_playing_ = is_playing;
 }
 
+bool WebMediaPlayerAndroid::GetGeometryChange(gfx::RectF* rect) {
+  if (!video_weblayer_)
+    return false;
+
+  cc::Layer* layer = video_weblayer_->layer();
+  rect->set_size(layer->bounds());
+  while (layer) {
+    rect->Offset(layer->position().OffsetFromOrigin());
+    layer = layer->parent();
+  }
+
+  if (rect_ == *rect)

[scherkus (not reviewing), 2013/04/08 14:58:23]

based on your usage, |rect| appears to always be (0,0,0,0)

does this condition ever evaluate to true?

[wonsik, 2013/04/09 09:13:22]

|rect| represents the geometry of the video frame (offset and size). |rect_|
will contain the geometry information when it was last computed. It can differ
from (0,0,0,0) in most of cases. Perhaps I confused you with lack of comments..

+    return false;
+
+  rect_ = *rect;
+  return true;
+}
+
 }  // namespace webkit_media