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

Unified Diff: content/browser/screen_orientation/screen_orientation_dispatcher_host_impl.h

Issue 2391883006: Mojo-ify implementation of screen orientation locking/unlocking. (Closed)
Patch Set: Code review changes (as to PS7) Created 4 years, 1 month ago
Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.
Jump to:
View side-by-side diff with in-line comments
Download patch
Index: content/browser/screen_orientation/screen_orientation_dispatcher_host_impl.h
diff --git a/content/browser/screen_orientation/screen_orientation_dispatcher_host_impl.h b/content/browser/screen_orientation/screen_orientation_dispatcher_host_impl.h
deleted file mode 100644
index 823afda0de789230ab0f6a6a2e80f07935763157..0000000000000000000000000000000000000000
--- a/content/browser/screen_orientation/screen_orientation_dispatcher_host_impl.h
+++ /dev/null
@@ -1,73 +0,0 @@
-// Copyright 2014 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.
-
-#ifndef CONTENT_BROWSER_SCREEN_ORIENTATION_SCREEN_ORIENTATION_DISPATCHER_HOST_IMPL_H_
-#define CONTENT_BROWSER_SCREEN_ORIENTATION_SCREEN_ORIENTATION_DISPATCHER_HOST_IMPL_H_
-
-#include "base/macros.h"
-#include "content/public/browser/screen_orientation_dispatcher_host.h"
-#include "content/public/browser/web_contents_observer.h"
-#include "third_party/WebKit/public/platform/modules/screen_orientation/WebLockOrientationError.h"
-#include "third_party/WebKit/public/platform/modules/screen_orientation/WebScreenOrientationLockType.h"
-
-namespace content {
-
-class RenderFrameHost;
-class ScreenOrientationProvider;
-class WebContents;
-
-// ScreenOrientationDispatcherHost receives lock and unlock requests from the
-// RenderFrames and dispatch them to the ScreenOrientationProvider. It also
-// make sure that the right RenderFrame get replied for each lock request.
-class CONTENT_EXPORT ScreenOrientationDispatcherHostImpl
- : public ScreenOrientationDispatcherHost,
- public WebContentsObserver {
- public:
- explicit ScreenOrientationDispatcherHostImpl(WebContents* web_contents);
- ~ScreenOrientationDispatcherHostImpl() override;
-
- // ScreenOrientationDispatcherHost:
- void NotifyLockSuccess(int request_id) override;
- void NotifyLockError(int request_id,
- blink::WebLockOrientationError error) override;
- void OnOrientationChange() override;
-
- // WebContentsObserver:
- bool OnMessageReceived(const IPC::Message&,
- RenderFrameHost* render_frame_host) override;
- void DidNavigateMainFrame(const LoadCommittedDetails& details,
- const FrameNavigateParams& params) override;
-
- private:
- void OnLockRequest(RenderFrameHost* render_frame_host,
- blink::WebScreenOrientationLockType orientation,
- int request_id);
- void OnUnlockRequest(RenderFrameHost* render_frame_host);
-
- // Returns a RenderFrameHost if the request_id is still valid and the
- // associated RenderFrameHost still exists. Returns NULL otherwise.
- RenderFrameHost* GetRenderFrameHostForRequestID(int request_id);
-
- void ResetCurrentLock();
- void NotifyLockError(int request_id,
- RenderFrameHost* render_frame_host,
- blink::WebLockOrientationError error);
-
- std::unique_ptr<ScreenOrientationProvider> provider_;
-
- struct LockInformation {
- LockInformation(int request_id, int process_id, int routing_id);
- int request_id;
- int process_id;
- int routing_id;
- };
- // current_lock_ will be NULL if there are no current lock.
- LockInformation* current_lock_;
-
- DISALLOW_COPY_AND_ASSIGN(ScreenOrientationDispatcherHostImpl);
-};
-
-} // namespace content
-
-#endif // CONTENT_BROWSER_SCREEN_ORIENTATION_SCREEN_ORIENTATION_DISPATCHER_HOST_IMPL_H_

Powered by Google App Engine
This is Rietveld 408576698