Clarify comments regarding overflow in svg.css

Clarify comments regarding overflow in svg.css

BUG=701654
Review-Url: https://codereview.chromium.org/2782643004
Cr-Commit-Position: refs/heads/master@{#460632}
Committed: https://chromium.googlesource.com/chromium/src/+/d92200ae8377ef1deeee84401f0ac5352c5e7b64

[chrishtr, 2017-03-29 21:12:50]

Description was changed from

==========
none

BUG=
==========

to

==========
Clarify comments regarding overflow in svg.css

BUG=01654
==========

[chrishtr, 2017-03-29 21:13:33]

chrishtr@chromium.org changed reviewers:
+ fs@opera.com

[fs, 2017-03-29 21:35:01]

BUG=... ref missing a digit (7?)

LGTM w/ nits.

https://codereview.chromium.org/2782643004/diff/1/third_party/WebKit/Source/c...
File third_party/WebKit/Source/core/css/svg.css (right):

https://codereview.chromium.org/2782643004/diff/1/third_party/WebKit/Source/c...
third_party/WebKit/Source/core/css/svg.css:44: Note that here "root svg element"
means a frame with purely SVG content, and in which SVG is the root element.
This could also be referred to as "when <svg> is the document element". I.e:

... means when <svg> is the document element. (":root" in CSS parlance.)

https://codereview.chromium.org/2782643004/diff/1/third_party/WebKit/Source/c...
third_party/WebKit/Source/core/css/svg.css:45: LayoutSVGRoot objects embededed
within containing content get overflow: hidden by default.
Nit: s/embededed/embedded/

[chrishtr, 2017-03-29 21:53:33]

Description was changed from

==========
Clarify comments regarding overflow in svg.css

BUG=01654
==========

to

==========
Clarify comments regarding overflow in svg.css

BUG=701654
==========

[chrishtr, 2017-03-30 00:43:13]

The CQ bit was checked by chrishtr@chromium.org

[chrishtr, 2017-03-30 00:43:14]

https://codereview.chromium.org/2782643004/diff/1/third_party/WebKit/Source/c...
File third_party/WebKit/Source/core/css/svg.css (right):

https://codereview.chromium.org/2782643004/diff/1/third_party/WebKit/Source/c...
third_party/WebKit/Source/core/css/svg.css:44: Note that here "root svg element"
means a frame with purely SVG content, and in which SVG is the root element.
On 2017/03/29 at 21:35:00, fs wrote:
> This could also be referred to as "when <svg> is the document element". I.e:
> 
> ... means when <svg> is the document element. (":root" in CSS parlance.)

Done.

https://codereview.chromium.org/2782643004/diff/1/third_party/WebKit/Source/c...
third_party/WebKit/Source/core/css/svg.css:45: LayoutSVGRoot objects embededed
within containing content get overflow: hidden by default.
On 2017/03/29 at 21:35:00, fs wrote:
> Nit: s/embededed/embedded/

Done.

[chrishtr, 2017-03-30 00:43:14]

The patchset sent to the CQ was uploaded after l-g-t-m from fs@opera.com
Link to the patchset: https://codereview.chromium.org/2782643004/#ps20001
(title: "none")

[commit-bot: I haz the power, 2017-03-30 00:44:38]

CQ is trying da patch. Follow status at
 
https://chromium-cq-status.appspot.com/v2/patch-status/codereview.chromium.or...

[commit-bot: I haz the power, 2017-03-30 02:58:16]

CQ is committing da patch.

Bot data: {"patchset_id": 20001, "attempt_start_ts": 1490834593477740,
"parent_rev": "f4cddb4364a978f6ba20cf0b5b4e48c924b121a8", "commit_rev":
"d92200ae8377ef1deeee84401f0ac5352c5e7b64"}

[commit-bot: I haz the power, 2017-03-30 02:59:05]

Description was changed from

==========
Clarify comments regarding overflow in svg.css

BUG=701654
==========

to

==========
Clarify comments regarding overflow in svg.css

BUG=701654

Review-Url: https://codereview.chromium.org/2782643004
Cr-Commit-Position: refs/heads/master@{#460632}
Committed:
https://chromium.googlesource.com/chromium/src/+/d92200ae8377ef1deeee84401f0a...
==========

[commit-bot: I haz the power, 2017-03-30 02:59:06]

Committed patchset #2 (id:20001) as
https://chromium.googlesource.com/chromium/src/+/d92200ae8377ef1deeee84401f0a...