Closed Bug 740188 Opened 12 years ago Closed 12 years ago

Screen Orientation API: implement locking

Categories

(Core :: DOM: Core & HTML, defect)

defect
Not set
normal

Tracking

()

RESOLVED FIXED
mozilla14

People

(Reporter: mounir, Assigned: mounir)

References

Details

(Keywords: dev-doc-complete)

Attachments

(3 files, 3 obsolete files)

Attached patch Part 1 - HAL/DOM boilerplate (obsolete) — Splinter Review
Basically, we want to add a lockOrientation(DOMString orientation) method to window.screen.
Attachment #610332 - Flags: superreview?(jonas)
Attachment #610332 - Flags: review?(bugs)
Attached patch Part 2 - Basic security model (obsolete) — Splinter Review
Basically, regular content will have to be fullscreen and chrome can do whatever they want.
Attachment #610336 - Flags: superreview?(jonas)
Attachment #610336 - Flags: review?(bugs)
Attachment #610337 - Flags: superreview?(jonas)
Attachment #610337 - Flags: review?(bugs)
Depends on: 740190
Whiteboard: [needs review]
Keywords: dev-doc-needed
Attachment #610332 - Flags: review?(bugs) → review-
Comment on attachment 610336 [details] [diff] [review]
Part 2 - Basic security model

Use GetOwner(), no mOwner
Attachment #610336 - Flags: review?(bugs) → review-
Comment on attachment 610337 [details] [diff] [review]
Part 3 - Revert orientation when fullscreen is left

Please no raw pointers which you don't clear ever.
Attachment #610337 - Flags: review?(bugs) → review-
Attachment #610332 - Attachment is obsolete: true
Attachment #610332 - Flags: superreview?(jonas)
Attachment #610415 - Flags: superreview?(jonas)
Attachment #610415 - Flags: review?(bugs)
Attachment #610336 - Attachment is obsolete: true
Attachment #610336 - Flags: superreview?(jonas)
Attachment #610416 - Flags: superreview?(jonas)
Attachment #610416 - Flags: review?(bugs)
Attachment #610337 - Attachment is obsolete: true
Attachment #610337 - Flags: superreview?(jonas)
Attachment #610418 - Flags: superreview?(jonas)
Attachment #610418 - Flags: review?(bugs)
Attachment #610415 - Flags: review?(bugs) → review+
Comment on attachment 610416 [details] [diff] [review]
Part 2 - Basic security model


>-nsScreen::Create(nsPIDOMWindow* aWindow)
>+nsScreen::Create(nsGlobalWindow* aWindow)
Is there now any need for this change?



> {
>+  if (!aWindow) {
>+    return nsnull;
>+  }
MOZ_ASSERT that aWindow is not null


>+  nsCOMPtr<nsIDOMWindow> window = do_QueryInterface(GetOwner());
GetOwner returns nsPIDOMWindow which is-a nsIDOMWindow, so there shouldn't be
need for this QI
;
> }
> 
>+
> NS_IMETHODIMP
Extra newline before NS_IMETHODIMP
Attachment #610416 - Flags: review?(bugs) → review+
Comment on attachment 610418 [details] [diff] [review]
Part 3 - Revert orientation when fullscreen is left


>+  private:
>+    nsIDOMEventTarget* mTarget;

Just use event->GetCurrentTarget();
Attachment #610418 - Flags: review?(bugs) → review+
Attachment #610416 - Flags: superreview?(jonas)
Attachment #610418 - Flags: superreview?(jonas)
Attachment #610415 - Flags: superreview?(jonas) → superreview+
https://hg.mozilla.org/mozilla-central/rev/a1ba687fd66a
https://hg.mozilla.org/mozilla-central/rev/d8aa32fa5531
https://hg.mozilla.org/mozilla-central/rev/13bf530d772d
Status: ASSIGNED → RESOLVED
Closed: 12 years ago
Resolution: --- → FIXED
Whiteboard: [needs review]
Target Milestone: --- → mozilla14
Depends on: 743217
Depends on: 747442
Depends on: 764753
Depends on: 766903
This has been documented, some time ago ;-)
You need to log in before you can comment on or make changes to this bug.