Move MIME related platform files under network/mime

Move MIME related platform files under network/mime

* MIMETypeRegistry.cpp
* MIMETypeFromURL.{cpp,h}
* ContentType.{cpp,h}

Follow-up for crrev.com/2444873002

BUG=659467
Committed: https://crrev.com/97c9fecf4a50dd69ccf162aa1c75c7f40cdbbf58
Cr-Commit-Position: refs/heads/master@{#436977}

[kinuko, 2016-11-05 09:03:57]

Patchset #1 (id:1) has been deleted

[kinuko, 2016-11-05 10:33:40]

The CQ bit was checked by kinuko@chromium.org to run a CQ dry run

[commit-bot: I haz the power, 2016-11-05 10:33:52]

Dry run: 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-11-05 11:50:55]

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

[commit-bot: I haz the power, 2016-11-05 11:50:56]

Dry run: Try jobs failed on following builders:
  linux_android_rel_ng on master.tryserver.chromium.android (JOB_FAILED,
https://build.chromium.org/p/tryserver.chromium.android/builders/linux_androi...)

[kinuko, 2016-11-06 07:10:59]

Description was changed from

==========
Move MIME related platform files under network/mime

Follow-up for crrev.com/2444873002

BUG=659467
==========

to

==========
Move MIME related platform files under network/mime

Follow-up for crrev.com/2444873002

BUG=659467
==========

[kinuko, 2016-11-06 07:10:59]

kinuko@chromium.org changed reviewers:
+ esprehn@chromium.org

[kinuko, 2016-11-06 07:11:25]

Description was changed from

==========
Move MIME related platform files under network/mime

Follow-up for crrev.com/2444873002

BUG=659467
==========

to

==========
Move MIME related platform files under network/mime

* MIMETypeRegistry.cpp
* MIMETypeFromURL.{cpp,h}
* ContentType.{cpp,h}

Follow-up for crrev.com/2444873002

BUG=659467
==========

[kinuko, 2016-11-06 07:11:52]

Moving all the remaining mime files.  Mechanical changes only-- PTL!

[kinuko, 2016-11-11 13:42:26]

Friendly ping-- this is not too exciting change but could you review?

(or please kindly let me know if this disagrees with your plan)

[kinuko, 2016-12-02 05:56:22]

The CQ bit was checked by kinuko@chromium.org to run a CQ dry run

[commit-bot: I haz the power, 2016-12-02 05:56:44]

Dry run: 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-12-02 06:06:41]

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

[commit-bot: I haz the power, 2016-12-02 06:06:42]

Dry run: Try jobs failed on following builders:
  cast_shell_android on master.tryserver.chromium.android (JOB_FAILED,
https://build.chromium.org/p/tryserver.chromium.android/builders/cast_shell_a...)

[kinuko, 2016-12-02 13:49:22]

The CQ bit was checked by kinuko@chromium.org to run a CQ dry run

[commit-bot: I haz the power, 2016-12-02 13:49:38]

Dry run: 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-12-02 14:01:48]

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

[commit-bot: I haz the power, 2016-12-02 14:01:48]

Dry run: Try jobs failed on following builders:
  cast_shell_android on master.tryserver.chromium.android (JOB_FAILED,
https://build.chromium.org/p/tryserver.chromium.android/builders/cast_shell_a...)
  chromeos_amd64-generic_chromium_compile_only_ng on
master.tryserver.chromium.linux (JOB_FAILED,
http://build.chromium.org/p/tryserver.chromium.linux/builders/chromeos_amd64-...)

[kinuko, 2016-12-02 14:30:28]

Patchset #5 (id:90037) has been deleted

[kinuko, 2016-12-02 14:30:34]

The CQ bit was checked by kinuko@chromium.org to run a CQ dry run

[commit-bot: I haz the power, 2016-12-02 14:30:46]

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

[kinuko, 2016-12-02 14:34:32]

Patchset #4 (id:80001) has been deleted

[commit-bot: I haz the power, 2016-12-02 14:40:09]

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

[commit-bot: I haz the power, 2016-12-02 14:40:09]

Dry run: Try jobs failed on following builders:
  blimp_linux_dbg on master.tryserver.chromium.linux (JOB_FAILED,
http://build.chromium.org/p/tryserver.chromium.linux/builders/blimp_linux_dbg...)
  mac_chromium_compile_dbg_ng on master.tryserver.chromium.mac (JOB_FAILED,
http://build.chromium.org/p/tryserver.chromium.mac/builders/mac_chromium_comp...)

[kinuko, 2016-12-02 17:11:20]

The CQ bit was checked by kinuko@chromium.org to run a CQ dry run

[kinuko, 2016-12-02 17:11:25]

Patchset #4 (id:110001) has been deleted

[commit-bot: I haz the power, 2016-12-02 17:11:41]

Dry run: 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-12-02 21:44:21]

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

[commit-bot: I haz the power, 2016-12-02 21:44:22]

Dry run: This issue passed the CQ dry run.

[kinuko, 2016-12-04 15:35:15]

kinuko@chromium.org changed reviewers:
+ mkwst@chromium.org
- esprehn@chromium.org

[kinuko, 2016-12-04 15:39:01]

Mike, do you think you can review this?  The change itself is pretty mechanical,
new location for the mime-related files is already discussed with Elliott in the
previous patch (https://crrev.com/2444873002) and hopefully no need to argue.

[Mike West, 2016-12-06 13:44:35]

LGTM, except for the license changes. If you're just moving the file, my
understanding is that we shouldn't modify the license. *shrug* IANAL, so...

https://codereview.chromium.org/2482653002/diff/130001/third_party/WebKit/Sou...
File third_party/WebKit/Source/platform/network/mime/ContentType.cpp (left):

https://codereview.chromium.org/2482653002/diff/130001/third_party/WebKit/Sou...
third_party/WebKit/Source/platform/network/mime/ContentType.cpp:27: */
Nit: If you're just moving the file, don't change the license.

https://codereview.chromium.org/2482653002/diff/130001/third_party/WebKit/Sou...
File third_party/WebKit/Source/platform/network/mime/MIMETypeRegistry.h (left):

https://codereview.chromium.org/2482653002/diff/130001/third_party/WebKit/Sou...
third_party/WebKit/Source/platform/network/mime/MIMETypeRegistry.h:24: */
Nit: If you're just moving the file, don't change the license.

[tyoshino (SeeGerritForStatus), 2016-12-07 07:20:11]

On 2016/12/06 13:44:35, Mike West (slow) wrote:
> LGTM, except for the license changes. If you're just moving the file, my
> understanding is that we shouldn't modify the license. *shrug* IANAL, so...
> 
>
https://codereview.chromium.org/2482653002/diff/130001/third_party/WebKit/Sou...
> File third_party/WebKit/Source/platform/network/mime/ContentType.cpp (left):
> 
>
https://codereview.chromium.org/2482653002/diff/130001/third_party/WebKit/Sou...
> third_party/WebKit/Source/platform/network/mime/ContentType.cpp:27: */
> Nit: If you're just moving the file, don't change the license.
> 
>
https://codereview.chromium.org/2482653002/diff/130001/third_party/WebKit/Sou...
> File third_party/WebKit/Source/platform/network/mime/MIMETypeRegistry.h
(left):
> 
>
https://codereview.chromium.org/2482653002/diff/130001/third_party/WebKit/Sou...
> third_party/WebKit/Source/platform/network/mime/MIMETypeRegistry.h:24: */
> Nit: If you're just moving the file, don't change the license.

Ditto for the files for which move detection didn't happen.

[tyoshino (SeeGerritForStatus), 2016-12-07 07:20:19]

tyoshino@chromium.org changed reviewers:
+ tyoshino@chromium.org

[tyoshino (SeeGerritForStatus), 2016-12-07 07:20:20]

lgtm

https://codereview.chromium.org/2482653002/diff/130001/third_party/WebKit/Sou...
File third_party/WebKit/Source/core/inspector/InspectorPageAgent.cpp (left):

https://codereview.chromium.org/2482653002/diff/130001/third_party/WebKit/Sou...
third_party/WebKit/Source/core/inspector/InspectorPageAgent.cpp:62: #include
"core/loader/resource/CSSStyleSheetResource.h"
toCSSStyleSheetResource() used in this file is defined in this header using  the
DEFINE_RESOURCE_TYPE_CASTS macro.

https://codereview.chromium.org/2482653002/diff/130001/third_party/WebKit/Sou...
third_party/WebKit/Source/core/inspector/InspectorPageAgent.cpp:64: #include
"core/loader/resource/ScriptResource.h"
ditto about toScriptResource()

[kinuko, 2016-12-07 10:42:01]

The CQ bit was checked by kinuko@chromium.org to run a CQ dry run

[commit-bot: I haz the power, 2016-12-07 10:42:14]

Dry run: 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-12-07 10:55:20]

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

[commit-bot: I haz the power, 2016-12-07 10:55:21]

Dry run: Try jobs failed on following builders:
  linux_chromium_chromeos_ozone_rel_ng on master.tryserver.chromium.linux
(JOB_FAILED,
http://build.chromium.org/p/tryserver.chromium.linux/builders/linux_chromium_...)

[kinuko, 2016-12-07 14:36:09]

The CQ bit was checked by kinuko@chromium.org to run a CQ dry run

[commit-bot: I haz the power, 2016-12-07 14:36:23]

Dry run: 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-12-07 15:27:28]

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

[commit-bot: I haz the power, 2016-12-07 15:27:29]

Dry run: Try jobs failed on following builders:
  android_n5x_swarming_rel on master.tryserver.chromium.android (JOB_FAILED,
https://build.chromium.org/p/tryserver.chromium.android/builders/android_n5x_...)

[kinuko, 2016-12-07 15:33:36]

Thanks! Restored the copyright info in the moved files.

https://codereview.chromium.org/2482653002/diff/130001/third_party/WebKit/Sou...
File third_party/WebKit/Source/core/inspector/InspectorPageAgent.cpp (left):

https://codereview.chromium.org/2482653002/diff/130001/third_party/WebKit/Sou...
third_party/WebKit/Source/core/inspector/InspectorPageAgent.cpp:62: #include
"core/loader/resource/CSSStyleSheetResource.h"
On 2016/12/07 07:20:20, tyoshino wrote:
> toCSSStyleSheetResource() used in this file is defined in this header using 
the
> DEFINE_RESOURCE_TYPE_CASTS macro.

Oops, there was a merge error... done.

https://codereview.chromium.org/2482653002/diff/130001/third_party/WebKit/Sou...
third_party/WebKit/Source/core/inspector/InspectorPageAgent.cpp:64: #include
"core/loader/resource/ScriptResource.h"
On 2016/12/07 07:20:20, tyoshino wrote:
> ditto about toScriptResource()

Done.

https://codereview.chromium.org/2482653002/diff/130001/third_party/WebKit/Sou...
File third_party/WebKit/Source/platform/network/mime/ContentType.cpp (left):

https://codereview.chromium.org/2482653002/diff/130001/third_party/WebKit/Sou...
third_party/WebKit/Source/platform/network/mime/ContentType.cpp:27: */
On 2016/12/06 13:44:35, Mike West (slow) wrote:
> Nit: If you're just moving the file, don't change the license.

Done.

https://codereview.chromium.org/2482653002/diff/130001/third_party/WebKit/Sou...
File third_party/WebKit/Source/platform/network/mime/MIMETypeRegistry.h (left):

https://codereview.chromium.org/2482653002/diff/130001/third_party/WebKit/Sou...
third_party/WebKit/Source/platform/network/mime/MIMETypeRegistry.h:24: */
On 2016/12/06 13:44:35, Mike West (slow) wrote:
> Nit: If you're just moving the file, don't change the license.

Done.

[kinuko, 2016-12-07 15:33:43]

The CQ bit was checked by kinuko@chromium.org

[kinuko, 2016-12-07 15:33:44]

The patchset sent to the CQ was uploaded after l-g-t-m from
tyoshino@chromium.org, mkwst@chromium.org
Link to the patchset: https://codereview.chromium.org/2482653002/#ps170001
(title: "fix")

[commit-bot: I haz the power, 2016-12-07 15:34:07]

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-12-07 16:35:49]

CQ is committing da patch.

Bot data: {"patchset_id": 170001, "attempt_start_ts": 1481124823319760,
"parent_rev": "ebc6e9a3206afabb4eefd7bcadd48c11b67839c5", "commit_rev":
"a62390df0484d30dbdab075c3ed9dc23f95ce7bb"}

[commit-bot: I haz the power, 2016-12-07 16:36:15]

Description was changed from

==========
Move MIME related platform files under network/mime

* MIMETypeRegistry.cpp
* MIMETypeFromURL.{cpp,h}
* ContentType.{cpp,h}

Follow-up for crrev.com/2444873002

BUG=659467
==========

to

==========
Move MIME related platform files under network/mime

* MIMETypeRegistry.cpp
* MIMETypeFromURL.{cpp,h}
* ContentType.{cpp,h}

Follow-up for crrev.com/2444873002

BUG=659467
==========

[commit-bot: I haz the power, 2016-12-07 16:36:17]

Committed patchset #6 (id:170001)

[commit-bot: I haz the power, 2016-12-07 16:39:14]

Description was changed from

==========
Move MIME related platform files under network/mime

* MIMETypeRegistry.cpp
* MIMETypeFromURL.{cpp,h}
* ContentType.{cpp,h}

Follow-up for crrev.com/2444873002

BUG=659467
==========

to

==========
Move MIME related platform files under network/mime

* MIMETypeRegistry.cpp
* MIMETypeFromURL.{cpp,h}
* ContentType.{cpp,h}

Follow-up for crrev.com/2444873002

BUG=659467

Committed: https://crrev.com/97c9fecf4a50dd69ccf162aa1c75c7f40cdbbf58
Cr-Commit-Position: refs/heads/master@{#436977}
==========

[commit-bot: I haz the power, 2016-12-07 16:39:15]

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