Closed Bug 1362195 Opened 7 years ago Closed 7 years ago

Remove GeckoInterface.setFullScreen

Categories

(GeckoView :: General, enhancement)

50 Branch
Unspecified
Android
enhancement
Not set
normal

Tracking

(firefox55 fixed)

RESOLVED FIXED
mozilla55
Tracking Status
firefox55 --- fixed

People

(Reporter: jchen, Assigned: jchen)

References

(Blocks 1 open bug)

Details

Attachments

(5 files)

      No description provided.
Add an EventDispatcher::Dispatch function for C++ code, so native code
can dispatch events to JS/Java.
Attachment #8866092 - Flags: review?(snorp)
Add the onFullScreen method to GeckoView.ContentListener to handle
entering and exiting full screen mode.
Attachment #8866093 - Flags: review?(snorp)
Send an event from nsWindow to GeckoView when we enter/exit full screen
mode, and call ContentListener.onFullScreen to notify the implementer.
Attachment #8866094 - Flags: review?(snorp)
Use a custom ContentListener for GeckoApp/BrowserApp that implements
onFullScreen to set full screen mode for the Activity.
Attachment #8866095 - Flags: review?(snorp)
Remove GeckoInterface.setFullScreen now that it's no longer used. r=me
for trivial patch.
Attachment #8866096 - Flags: review+
Attachment #8866092 - Flags: review?(snorp) → review+
Attachment #8866093 - Flags: review?(snorp) → review+
Attachment #8866094 - Flags: review?(snorp) → review+
Attachment #8866095 - Flags: review?(snorp) → review+
Pushed by nchen@mozilla.com:
https://hg.mozilla.org/integration/mozilla-inbound/rev/741e175e254c
1. Add EventDispatcher::Dispatch for C++ code; r=snorp
https://hg.mozilla.org/integration/mozilla-inbound/rev/2acf5f4b6943
2. Add onFullScreen to GeckoView.ContentListener; r=snorp
https://hg.mozilla.org/integration/mozilla-inbound/rev/8e8e68c6e3ae
3. Call onFullScreen for full screen changes; r=snorp
https://hg.mozilla.org/integration/mozilla-inbound/rev/5eb0e63fde7b
4. Implement onFullScreen for GeckoApp/BrowserApp; r=snorp
https://hg.mozilla.org/integration/mozilla-inbound/rev/4dda9c43663e
5. Remove GeckoInterface.setFullScreen; r=me
Pushed by nchen@mozilla.com:
https://hg.mozilla.org/integration/mozilla-inbound/rev/6ab9ba92e6a4
6. Implement onFullScreen for geckoview_example; r=me
Assignee: nobody → nchen
Blocks: 1503482
Product: Firefox for Android → GeckoView
Target Milestone: Firefox 55 → mozilla55
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Created:
Updated:
Size: