Closed
Bug 1084521
Opened 10 years ago
Closed 10 years ago
did you mean to use @+id instead of @+android:id?
Categories
(Android Background Services Graveyard :: Firefox Accounts, defect)
Tracking
(firefox34 fixed, firefox35 fixed, firefox36 fixed)
RESOLVED
FIXED
Firefox 36
People
(Reporter: rnewman, Assigned: rnewman)
References
Details
Attachments
(1 file)
3.48 KB,
patch
|
lucasr
:
review+
lsblakk
:
approval-mozilla-aurora+
lsblakk
:
approval-mozilla-beta+
|
Details | Diff | Splinter Review |
21:27.88 /Users/rnewman/moz/hg/fx-team/mobile/android/base/resources/layout/fxaccount_custom_server_view.xml:37: note: did you mean to use @+id instead of @+android:id?
21:27.88 /Users/rnewman/moz/hg/fx-team/mobile/android/base/resources/layout/fxaccount_custom_server_view.xml:48: note: did you mean to use @+id instead of @+android:id?
21:27.88 /Users/rnewman/moz/hg/fx-team/mobile/android/base/resources/layout/fxaccount_custom_server_view.xml:90: note: did you mean to use @+id instead of @+android:id?
21:27.88 /Users/rnewman/moz/hg/fx-team/mobile/android/base/resources/layout/fxaccount_custom_server_view.xml:101: note: did you mean to use @+id instead of @+android:id?
Assignee | ||
Comment 1•10 years ago
|
||
Probably causes this, too:
21:28.09 /Users/rnewman/moz/hg/fx-team/mobile/android/base/resources/layout/fxaccount_custom_server_view.xml:37: error: creating resource for external package android: id/account_server_title.
21:28.09 /Users/rnewman/moz/hg/fx-team/mobile/android/base/resources/layout/fxaccount_custom_server_view.xml:37: error: Error: No resource found that matches the given name (at 'id' with value '@+android:id/account_server_title').
21:28.09
21:28.09 /Users/rnewman/moz/hg/fx-team/mobile/android/base/resources/layout/fxaccount_custom_server_view.xml:48: error: creating resource for external package android: id/account_server_summary.
21:28.09 /Users/rnewman/moz/hg/fx-team/mobile/android/base/resources/layout/fxaccount_custom_server_view.xml:48: error: Error: No resource found that matches the given name (at 'id' with value '@+android:id/account_server_summary').
21:28.09
21:28.09 /Users/rnewman/moz/hg/fx-team/mobile/android/base/resources/layout/fxaccount_custom_server_view.xml:90: error: creating resource for external package android: id/sync_server_title.
21:28.10 /Users/rnewman/moz/hg/fx-team/mobile/android/base/resources/layout/fxaccount_custom_server_view.xml:90: error: Error: No resource found that matches the given name (at 'id' with value '@+android:id/sync_server_title').
21:28.10
21:28.10 /Users/rnewman/moz/hg/fx-team/mobile/android/base/resources/layout/fxaccount_custom_server_view.xml:101: error: creating resource for external package android: id/sync_server_summary.
21:28.10 /Users/rnewman/moz/hg/fx-team/mobile/android/base/resources/layout/fxaccount_custom_server_view.xml:101: error: Error: No resource found that matches the given name (at 'id' with value '@+android:id/sync_server_summary').
Assignee | ||
Comment 2•10 years ago
|
||
This looks like an over-zealous s//.
Uplift needed.
Attachment #8507228 -
Flags: review?(nalexander)
Assignee | ||
Updated•10 years ago
|
Assignee: nobody → rnewman
Status: NEW → ASSIGNED
Assignee | ||
Comment 3•10 years ago
|
||
Comment 4•10 years ago
|
||
Comment on attachment 8507228 [details] [diff] [review]
Use +id not +android:id. v1
Review of attachment 8507228 [details] [diff] [review]:
-----------------------------------------------------------------
Yep.
Attachment #8507228 -
Flags: review?(nalexander) → review+
Assignee | ||
Comment 5•10 years ago
|
||
Assignee | ||
Comment 6•10 years ago
|
||
Status: ASSIGNED → RESOLVED
Closed: 10 years ago
Resolution: --- → FIXED
Target Milestone: --- → Firefox 36
Comment 8•10 years ago
|
||
This patch applies cleanly to (at least) the beta branch (firefox-34) and fixes the same issue that occurs when building against that branch as well. I haven't tested if the patch applies cleanly to aurora (firefox-35), but I would assume that it does.
Comment 9•10 years ago
|
||
Comment on attachment 8507228 [details] [diff] [review]
Use +id not +android:id. v1
Approval Request Comment
[Feature/regressing bug #]: no idea.
[User impact if declined]: none; this is build only.
[Describe test coverage new/current, TBPL]: none needed, this is build only.
[Risks and why]: none.
[String/UUID change made/needed]: none.
Attachment #8507228 -
Flags: approval-mozilla-beta?
Attachment #8507228 -
Flags: approval-mozilla-aurora?
Updated•10 years ago
|
Attachment #8507228 -
Flags: approval-mozilla-beta?
Attachment #8507228 -
Flags: approval-mozilla-beta+
Attachment #8507228 -
Flags: approval-mozilla-aurora?
Attachment #8507228 -
Flags: approval-mozilla-aurora+
Comment 10•10 years ago
|
||
You need to log in
before you can comment on or make changes to this bug.
Description
•