Closed Bug 1419247 Opened 3 years ago Closed 3 years ago

Allow Stringify() to support nsSize.

Categories

(Core :: Layout, enhancement)

enhancement
Not set
normal

Tracking

()

RESOLVED FIXED
mozilla59
Tracking Status
firefox59 --- fixed

People

(Reporter: kuoe0.tw, Assigned: kuoe0.tw)

Details

Attachments

(1 file)

Allow us to use Stringify() with nsSize.
Attachment #8930331 - Flags: review?(mtseng)
Assignee: nobody → kuoe0
Comment on attachment 8930331 [details]
Bug 1419247 - Allow Stringify() to support nsSize.

https://reviewboard.mozilla.org/r/201470/#review206616
Attachment #8930331 - Flags: review?(mtseng) → review+
Comment on attachment 8930331 [details]
Bug 1419247 - Allow Stringify() to support nsSize.

https://reviewboard.mozilla.org/r/201470/#review206622

::: gfx/layers/LayersLogging.cpp:113
(Diff revision 2)
> +  aStream << pfx;
> +  aStream << nsPrintfCString(
> +    "(w=%d, h=%d)",
> +    sz.width, sz.height).get();
> +  aStream << sfx;
> +}

Nit: A blank line here, please.
Comment on attachment 8930331 [details]
Bug 1419247 - Allow Stringify() to support nsSize.

https://reviewboard.mozilla.org/r/201470/#review206628

carry r+
Attachment #8930331 - Flags: review+
Pushed by tokuo@mozilla.com:
https://hg.mozilla.org/integration/autoland/rev/476e4e86e9df
Allow Stringify() to support nsSize. r=mtseng
https://hg.mozilla.org/mozilla-central/rev/476e4e86e9df
Status: NEW → RESOLVED
Closed: 3 years ago
Resolution: --- → FIXED
Target Milestone: --- → mozilla59
You need to log in before you can comment on or make changes to this bug.