Hi Giovanny! You are right. We will have to wait until Android M is released. Feel free to attach a patch to this bug as soon as it has been released and I'll assign the bug to you. I'll try to ping you here if I see anything being released. :)

Hi Sebastian,
Thanks for the alert, I attached a patch with the changes. Let me know if it is OK.
Curious but I was convinced since the beginning that Marshmallow will going to be the name for this Android release. :)
Marshmallows for all the Mozilla Android/Mobile team \o/
http://goo.gl/iAMEui

Comment on attachment 8670098[details][diff][review]
removed the line break (friedger2.patch)
Review of attachment 8670098[details][diff][review]:
-----------------------------------------------------------------
Thanks! Can you create a single, combined patch? :)
::: mobile/android/base/AppConstants.java.in
@@ +63,5 @@
> * If our MIN_SDK_VERSION is 14 or higher, we must be an ICS device.
> * If our MAX_SDK_VERSION is lower than ICS, we must not be an ICS device.
> * Otherwise, we need a range check.
> */
> + public static final boolean preM = MAX_SDK_VERSION < 23 || (MIN_SDK_VERSION < 23 && Build.VERSION.SDK_INT < 23);
It looks like there's a mix of tabs and regular whitespaces at the beginning of this line. Just use four spaces for indentation. :)

(In reply to Mouaad from comment #11)
> Hello :)
> I'am new here ! I was looking for some first simple bugs to start, I don't
> get what is left to do about this bug ?
Hi Mouaad! Basically just combining the two patches. Or rewriting the patch because the code could have changed in the meantime.

(In reply to Sebastian Kaspari (:sebastian) from comment #12)
> Hi Mouaad! Basically just combining the two patches. Or rewriting the patch
> because the code could have changed in the meantime.
Hello :) ! Alright then, can i give it a shot and try do it ? I succeeded to compile the project and run it on my device.

(In reply to Mouaad from comment #13)
> Hello :) ! Alright then, can i give it a shot and try do it ? I succeeded
> to compile the project and run it on my device.
Yeah, go ahead. Let me know if you need any help.

Is this still open to fix? I was thinking of picking this up. I figure that 4 files might need changing. I have a question though, the AppConstants.java file gets regenerated every time I run and hence, changes 'PreMarshmallow' back to 'PreM'. Anything I can do about this?

(In reply to juzerkhambaty from comment #15)
> Is this still open to fix? I was thinking of picking this up. I figure that
> 4 files might need changing. I have a question though, the AppConstants.java
> file gets regenerated every time I run and hence, changes 'PreMarshmallow'
> back to 'PreM'. Anything I can do about this?
Yeah, this is still open. You'll need to edit AppConstants.java.in - this is the template to generate AppConstants.java:
https://dxr.mozilla.org/mozilla-central/source/mobile/android/base/AppConstants.java.in
(Feel free to use the need-info flag so that your question will be answered)

Comment on attachment 8727164[details][diff][review]
Bug_1187382 -_Combined two patches into the latest sources
I have problems applying this patch to the latest tree:
> applying review
> unable to find 'android/base/java/org/mozilla/gecko/tabqueue/TabQueueHelper.java' for patching
> 2 out of 2 hunks FAILED -- saving rejects to file android/base/java/org/mozilla/gecko/tabqueue/TabQueueHelper.java.rej
> unable to find 'android/base/java/org/mozilla/gecko/tabqueue/TabQueueService.java' for patching
> 1 out of 1 hunks FAILED -- saving rejects to file android/base/java/org/mozilla/gecko/tabqueue/TabQueueService.java.rej
> unable to find 'android/base/AppConstants.java.in' for patching
> 1 out of 1 hunks FAILED -- saving rejects to file android/base/AppConstants.java.in.rej
> unable to find 'android/base/java/org/mozilla/gecko/preferences/AndroidImportPreference.java' for patching
> 1 out of 1 hunks FAILED -- saving rejects to file android/base/java/org/mozilla/gecko/preferences/AndroidImportPreference.java.rej
> patch failed, unable to continue (try -v)
> patch failed, rejects left in working directory
> errors during apply, please fix and qrefresh review
Did you create this patch from IntelliJ? It contains those additional lines for IntelliJ and this might be a problem for mercurial (Also there's no commit message in the patch). Try to create the patch on the command line (hg export).

Comment on attachment 8727346[details][diff][review]
Updated patch
Review of attachment 8727346[details][diff][review]:
-----------------------------------------------------------------
This is looking good! I'll push the patch to try.
::: mobile/android/base/AppConstants.java.in
@@ +62,5 @@
> * If our MIN_SDK_VERSION is 14 or higher, we must be an ICS device.
> * If our MAX_SDK_VERSION is lower than ICS, we must not be an ICS device.
> * Otherwise, we need a range check.
> */
> + public static final boolean preMarshmallow = MAX_SDK_VERSION < 23 || (MIN_SDK_VERSION < 23 && Build.VERSION.SDK_INT < 23 && !Build.VERSION.RELEASE.equals("M"));
You can remove !Build.VERSION.RELEASE.equals("M") from here. This was used to exclude M preview builds. No one should be using those anymore.