Implement ScreenOrientationDelegate on ChromeOS

content/public/browser/screen_orientation_provider.cc

Index: content/public/browser/screen_orientation_provider.cc
diff --git a/content/public/browser/screen_orientation_provider.cc b/content/public/browser/screen_orientation_provider.cc
index 07c4c9323e5b3e417e97bd06d7175594a88eebe6..35a07eeffde67319a00fca785cc50b72d335372c 100644
--- a/content/public/browser/screen_orientation_provider.cc
+++ b/content/public/browser/screen_orientation_provider.cc
@@ -47,17 +47,19 @@ void ScreenOrientationProvider::LockOrientation(int request_id,
     if (!rvh) {
       dispatcher_->NotifyLockError(request_id,
                                    blink::WebLockOrientationErrorCanceled);
+      return;
     }
     RenderViewHostDelegate* rvhd = rvh->GetDelegate();
     if (!rvhd) {
       dispatcher_->NotifyLockError(request_id,
                                    blink::WebLockOrientationErrorCanceled);
+      return;
     }
     if (!rvhd->IsFullscreenForCurrentTab()) {
       dispatcher_->NotifyLockError(request_id,
           blink::WebLockOrientationErrorFullScreenRequired);
+      return;

[mlamouri (slow - plz ping), 2014/10/15 15:51:27]

I moved this fix in https://codereview.chromium.org/655273003/ given that you
are OOO for the rest of the week.

     }
-    return;
   }
 
   if (lock_orientation == blink::WebScreenOrientationLockNatural) {