Closed
Bug 796084
Opened 11 years ago
Closed 11 years ago
Clean up PZC code
Categories
(Firefox for Android Graveyard :: Toolbar, defect)
Tracking
(Not tracked)
RESOLVED
FIXED
Firefox 18
People
(Reporter: BenWa, Assigned: BenWa)
Details
Attachments
(3 files, 1 obsolete file)
5.34 KB,
patch
|
blassey
:
review+
|
Details | Diff | Splinter Review |
25.05 KB,
patch
|
blassey
:
review+
|
Details | Diff | Splinter Review |
1.90 KB,
patch
|
cwiiis
:
review+
|
Details | Diff | Splinter Review |
Catch all bugs for trivial code clean up.
Assignee | ||
Comment 1•11 years ago
|
||
Attachment #666655 -
Flags: review?(blassey.bugs)
Assignee | ||
Comment 2•11 years ago
|
||
Attachment #666661 -
Flags: review?(blassey.bugs)
Updated•11 years ago
|
Attachment #666655 -
Flags: review?(blassey.bugs) → review+
Updated•11 years ago
|
Attachment #666661 -
Flags: review?(blassey.bugs) → review+
Assignee | ||
Comment 3•11 years ago
|
||
https://hg.mozilla.org/integration/mozilla-inbound/rev/3d9f6bb75bb0 https://hg.mozilla.org/integration/mozilla-inbound/rev/1b0fd0ddbfec
Target Milestone: --- → Firefox 18
Comment 4•11 years ago
|
||
Comment on attachment 666661 [details] [diff] [review] s/MOZ_JAVA_COMPOSITOR/MOZ_ANDROID_OMTC/g Review of attachment 666661 [details] [diff] [review]: ----------------------------------------------------------------- ::: configure.in @@ +5039,5 @@ > fi > > > dnl ======================================================== > dnl = Build with the Android Java compositor Drive-by - shouldn't this comment also be changed? ::: gfx/layers/opengl/LayerManagerOGL.cpp @@ +920,5 @@ > } > > mGLContext->fEnable(LOCAL_GL_SCISSOR_TEST); > > // If the Java compositor is being used, this clear will be done in Same here.
Assignee | ||
Comment 5•11 years ago
|
||
Argg, a configure change :( Alright patch coming.
Assignee | ||
Comment 6•11 years ago
|
||
Comment 7•11 years ago
|
||
Comment on attachment 666971 [details] [diff] [review] follow up Review of attachment 666971 [details] [diff] [review]: ----------------------------------------------------------------- Is this the right patch? If so, I don't think I'm the right reviewer for this.
Assignee | ||
Comment 8•11 years ago
|
||
right patch
Attachment #666971 -
Attachment is obsolete: true
Attachment #666971 -
Flags: review?(chrislord.net)
Attachment #666986 -
Flags: review?(chrislord.net)
Comment 9•11 years ago
|
||
Comment on attachment 666986 [details] [diff] [review] follow up Review of attachment 666986 [details] [diff] [review]: ----------------------------------------------------------------- Better :)
Attachment #666986 -
Flags: review?(chrislord.net) → review+
Comment 10•11 years ago
|
||
https://hg.mozilla.org/mozilla-central/rev/3d9f6bb75bb0 https://hg.mozilla.org/mozilla-central/rev/1b0fd0ddbfec
Status: ASSIGNED → RESOLVED
Closed: 11 years ago
Resolution: --- → FIXED
Assignee | ||
Comment 11•11 years ago
|
||
https://hg.mozilla.org/integration/mozilla-inbound/rev/5a5dabf022b7
Status: RESOLVED → REOPENED
Resolution: FIXED → ---
Comment 12•11 years ago
|
||
https://hg.mozilla.org/mozilla-central/rev/5a5dabf022b7
Status: REOPENED → RESOLVED
Closed: 11 years ago → 11 years ago
Flags: in-testsuite-
Resolution: --- → FIXED
Updated•2 years ago
|
Product: Firefox for Android → Firefox for Android Graveyard
You need to log in
before you can comment on or make changes to this bug.
Description
•