Closed
Bug 655427
Opened 14 years ago
Closed 14 years ago
make sharedIDs used by automation not depend on package name
Categories
(Core Graveyard :: Widget: Android, defect)
Tracking
(firefox5 fixed)
RESOLVED
FIXED
Tracking | Status | |
---|---|---|
firefox5 | --- | fixed |
People
(Reporter: blassey, Assigned: blassey)
References
Details
Attachments
(2 files, 2 obsolete files)
(deleted),
patch
|
Details | Diff | Splinter Review | |
(deleted),
patch
|
mfinkle
:
review+
mfinkle
:
approval-mozilla-aurora+
|
Details | Diff | Splinter Review |
No description provided.
Attachment #530784 -
Flags: review?(mark.finkle)
Comment 1•14 years ago
|
||
This patch is slightly bigger, but has the advantage that it does the right thing if anyone ever builds a non-Fennec Android app on the Mozilla platform. (Same as attachment 530783 [details] [diff] [review] from bug 654838, but updated to include Aurora.)
I'd be fine either way on this, since non-Fennec Android apps are purely theoretical at this point.
Assignee | ||
Comment 3•14 years ago
|
||
this keeps the existing behavior for any app other than fennec
Attachment #530784 -
Attachment is obsolete: true
Attachment #530784 -
Flags: review?(mark.finkle)
Attachment #530788 -
Flags: review?(mark.finkle)
Comment 4•14 years ago
|
||
Comment on attachment 530788 [details] [diff] [review]
patch
>diff --git a/embedding/android/Makefile.in b/embedding/android/Makefile.in
> -DANDROID_VERSION_CODE=$(ANDROID_VERSION_CODE) \
> $(NULL)
>
>+
>+
>+
> GARBAGE += \
No need to add the extra line breaks
r+ with the nit
Attachment #530788 -
Flags: review?(mark.finkle) → review+
Updated•14 years ago
|
OS: Linux → Android
Hardware: x86_64 → All
Assignee | ||
Updated•14 years ago
|
Attachment #530788 -
Flags: approval-mozilla-aurora?
Updated•14 years ago
|
Attachment #530788 -
Flags: approval-mozilla-aurora? → approval-mozilla-aurora+
Assignee | ||
Comment 5•14 years ago
|
||
Assignee | ||
Updated•14 years ago
|
Status: NEW → RESOLVED
Closed: 14 years ago
Resolution: --- → FIXED
Assignee | ||
Updated•14 years ago
|
status-firefox5:
--- → fixed
Updated•13 years ago
|
Assignee: nobody → blassey.bugs
Updated•3 years ago
|
Product: Core → Core Graveyard
You need to log in
before you can comment on or make changes to this bug.
Description
•