The default bug view has changed. See this FAQ.

Status

()

Core
IPC
RESOLVED FIXED
6 years ago
6 years ago

People

(Reporter: (dormant account), Assigned: (dormant account))

Tracking

Firefox Tracking Flags

(Not tracked)

Details

Attachments

(3 attachments)

(Assignee)

Description

6 years ago
Created attachment 527132 [details] [diff] [review]
Copy new histogram.* files

Update histogram classes to chromium trunk
Attachment #527132 - Flags: review?(jones.chris.g)
(Assignee)

Comment 1

6 years ago
Created attachment 527133 [details] [diff] [review]
hacks to keep things compiling

This is the minimal set of changes to get newer chrome code in. This is my first code-borrowing attempt, so I probably did it wrong.
Attachment #527133 - Flags: review?(jones.chris.g)
(Assignee)

Updated

6 years ago
Blocks: 649502
No longer blocks: 585196
I don't mind rubber-stamping these patches, but it doesn't make a lot of sense for us to build a dependency on code that's not meant to be used externally.  Please don't be shy about forking.  (We may be forced to do that anyway if there's a pending rewrite on the chromium side.)
Comment on attachment 527132 [details] [diff] [review]
Copy new histogram.* files

rs=me
Attachment #527132 - Flags: review?(jones.chris.g) → review+
Comment on attachment 527133 [details] [diff] [review]
hacks to keep things compiling

rs=me
Attachment #527133 - Flags: review?(jones.chris.g) → review+
(Assignee)

Comment 5

6 years ago
(In reply to comment #2)
> I don't mind rubber-stamping these patches, but it doesn't make a lot of sense
> for us to build a dependency on code that's not meant to be used externally. 
> Please don't be shy about forking.  (We may be forced to do that anyway if
> there's a pending rewrite on the chromium side.)

Thanks. I'd like to leverage the painful lessons that they learned for as long as I can. We can fork it once we know what works for us.
Certainly, that's been the m.o. with the IPC stuff.
(Assignee)

Comment 7

6 years ago
Created attachment 528396 [details] [diff] [review]
fix bustage on windows

This is a small compilation fix on windows. Had this attached to the wrong bug before.
Attachment #528396 - Flags: review?(jones.chris.g)
Attachment #528396 - Flags: review?(jones.chris.g) → review+
(Assignee)

Updated

6 years ago
Keywords: checkin-needed
(Assignee)

Comment 8

6 years ago
http://hg.mozilla.org/mozilla-central/rev/28ddb101c4ac
http://hg.mozilla.org/mozilla-central/rev/ecd8499c7828
http://hg.mozilla.org/mozilla-central/rev/00431dc35e33
Status: NEW → RESOLVED
Last Resolved: 6 years ago
Resolution: --- → FIXED
(Assignee)

Updated

6 years ago
Keywords: checkin-needed

Updated

6 years ago
Assignee: nobody → tglek
You need to log in before you can comment on or make changes to this bug.