Document SVGImage with a class-level comment

Document SVGImage with a class-level comment

The way SVGImage works is surprising. For example, it is not obvious
that we don't use Skia but instead re-use the existing paint pipeline
in Blink, nor that SVGImage contains a detached Page. This patch
adds some high-level documentation to make these quirks more obvious.

Some minor line-wrapping has also been cleaned up.

BUG=644913
NOTRY=true

Committed: https://crrev.com/6bfa5aace01353b25e5e468e759f48df3e010f50
Cr-Commit-Position: refs/heads/master@{#417665}

[pdr., 2016-09-09 00:37:08]

pdr@chromium.org changed reviewers:
+ davve@opera.com, fmalita@chromium.org, fs@opera.com, schenney@chromium.org

[pdr., 2016-09-09 00:38:31]

This class was badly in need of some high-level documentation.

Please nitpick.

[davve, 2016-09-09 08:29:09]

Describes the quirkiness of SVGImage well. Thanks!

LGTM.

[fs, 2016-09-09 08:39:46]

lgtm

[f(malita), 2016-09-09 13:57:32]

lgtm

[pdr., 2016-09-09 19:03:29]

The CQ bit was checked by pdr@chromium.org

[commit-bot: I haz the power, 2016-09-09 19:03:57]

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, 2016-09-09 19:10:03]

The CQ bit was unchecked by commit-bot@chromium.org

[commit-bot: I haz the power, 2016-09-09 19:10:04]

Try jobs failed on following builders:
  chromium_presubmit on master.tryserver.chromium.linux (JOB_FAILED,
http://build.chromium.org/p/tryserver.chromium.linux/builders/chromium_presub...)

[pdr., 2016-09-09 19:17:53]

The CQ bit was checked by pdr@chromium.org

[pdr., 2016-09-09 19:17:54]

The patchset sent to the CQ was uploaded after l-g-t-m from davve@opera.com,
fmalita@chromium.org, fs@opera.com
Link to the patchset: https://codereview.chromium.org/2325733003/#ps40001
(title: "Rebase with clean tree this time")

[commit-bot: I haz the power, 2016-09-09 19:18:27]

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, 2016-09-09 19:23:19]

Committed patchset #3 (id:40001)

[commit-bot: I haz the power, 2016-09-09 19:26:27]

Description was changed from

==========
Document SVGImage with a class-level comment

The way SVGImage works is surprising. For example, it is not obvious
that we don't use Skia but instead re-use the existing paint pipeline
in Blink, nor that SVGImage contains a detached Page. This patch
adds some high-level documentation to make these quirks more obvious.

Some minor line-wrapping has also been cleaned up.

BUG=644913
NOTRY=true
==========

to

==========
Document SVGImage with a class-level comment

The way SVGImage works is surprising. For example, it is not obvious
that we don't use Skia but instead re-use the existing paint pipeline
in Blink, nor that SVGImage contains a detached Page. This patch
adds some high-level documentation to make these quirks more obvious.

Some minor line-wrapping has also been cleaned up.

BUG=644913
NOTRY=true

Committed: https://crrev.com/6bfa5aace01353b25e5e468e759f48df3e010f50
Cr-Commit-Position: refs/heads/master@{#417665}
==========

[commit-bot: I haz the power, 2016-09-09 19:26:28]

Patchset 3 (id:??) landed as
https://crrev.com/6bfa5aace01353b25e5e468e759f48df3e010f50
Cr-Commit-Position: refs/heads/master@{#417665}