content::ResourceType clean up

content::ResourceType clean up

- Remove unused using statements
- Remove unnecessary content::
- Include resource_type.h where IsResourceTypeFrame() is used
- Use IsResourceTypeFrame() where we can

R=reillyg@chromium.org,mmenke@chromium.org,mattm@chromium.org,treib@chromium.org
BUG=none

Committed: https://crrev.com/430d2f47056ec2691b43239f6390758a3f2e6ccc
Cr-Commit-Position: refs/heads/master@{#438792}

[tyoshino (SeeGerritForStatus), 2016-12-14 08:58:33]

Description was changed from

==========
Clean up ResourceType

BUG=none
==========

to

==========
content::ResourceType clean up

- Remove unused using statements
- Remove unnecessary content::
- Include resource_type.h where IsResourceTypeFrame() is used
- Use IsResourceTypeFrame() where we can

BUG=none
==========

[tyoshino (SeeGerritForStatus), 2016-12-14 08:58:44]

Description was changed from

==========
content::ResourceType clean up

- Remove unused using statements
- Remove unnecessary content::
- Include resource_type.h where IsResourceTypeFrame() is used
- Use IsResourceTypeFrame() where we can

BUG=none
==========

to

==========
content::ResourceType clean up

- Remove unused using statements
- Remove unnecessary content::
- Include resource_type.h where IsResourceTypeFrame() is used
- Use IsResourceTypeFrame() where we can

BUG=none
==========

[tyoshino (SeeGerritForStatus), 2016-12-14 08:59:08]

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

[tyoshino (SeeGerritForStatus), 2016-12-14 08:59:38]

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

[tyoshino (SeeGerritForStatus), 2016-12-14 09:00:09]

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

[tyoshino (SeeGerritForStatus), 2016-12-14 09:00:46]

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

[tyoshino (SeeGerritForStatus), 2016-12-14 09:03:04]

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

[Marc Treib, 2016-12-14 10:28:50]

iframe_source_unittest.cc lgtm

(When adding multiple reviewers, it's considered good form to specify who should
look at what. Many reviewers own various folders...  Thanks!)

[tyoshino (SeeGerritForStatus), 2016-12-14 15:37:56]

On 2016/12/14 10:28:50, Marc Treib wrote:
> iframe_source_unittest.cc lgtm
> 
> (When adding multiple reviewers, it's considered good form to specify who
should
> look at what. Many reviewers own various folders...  Thanks!)

Sorry! I've been doing that but this time I was lazy due to the simpleness of
the change.

Thanks, treib@ for reviewing chrome/browser/search/

reillyg@: PTAL extensions/
mattm@: PTAL components/safe_browsing_db/
mmenke@: PTAL chrome/browser/loader/, chrome/browser/net, content/browser/loader

[tyoshino (SeeGerritForStatus), 2016-12-14 15:39:13]

On 2016/12/14 15:37:56, tyoshino(OOODec16-25) wrote:
> On 2016/12/14 10:28:50, Marc Treib wrote:
> > iframe_source_unittest.cc lgtm
> > 
> > (When adding multiple reviewers, it's considered good form to specify who
> should
> > look at what. Many reviewers own various folders...  Thanks!)
> 
> Sorry! I've been doing that but this time I was lazy due to the simpleness of
> the change.
> 
> Thanks, treib@ for reviewing chrome/browser/search/
> 
> reillyg@: PTAL extensions/
> mattm@: PTAL components/safe_browsing_db/
> mmenke@: PTAL chrome/browser/loader/, chrome/browser/net,
content/browser/loader

Removing jam@. All the files have been covered by the subdirectory OWNERS.
Sorry, John!

[tyoshino (SeeGerritForStatus), 2016-12-14 15:39:22]

tyoshino@chromium.org changed reviewers:
- jam@chromium.org

[tyoshino (SeeGerritForStatus), 2016-12-14 15:39:29]

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

[commit-bot: I haz the power, 2016-12-14 15:39:37]

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

[Reilly Grant (use Gerrit), 2016-12-14 16:10:52]

extensions lgtm

[mmenke, 2016-12-14 16:34:53]

chrome/browser/loader/, chrome/browser/net, content/browser/loader all LGTM!

[commit-bot: I haz the power, 2016-12-14 16:35:56]

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

[commit-bot: I haz the power, 2016-12-14 16:35:57]

Dry run: This issue passed the CQ dry run.

[mattm, 2016-12-14 22:34:24]

safe_browsing_db lgtm

[tyoshino (SeeGerritForStatus), 2016-12-15 08:16:21]

Description was changed from

==========
content::ResourceType clean up

- Remove unused using statements
- Remove unnecessary content::
- Include resource_type.h where IsResourceTypeFrame() is used
- Use IsResourceTypeFrame() where we can

BUG=none
==========

to

==========
content::ResourceType clean up

- Remove unused using statements
- Remove unnecessary content::
- Include resource_type.h where IsResourceTypeFrame() is used
- Use IsResourceTypeFrame() where we can

R=reillyg@chromium.org,mmenke@chromium.org,mattm@chromium.org,treib@chromium.org
BUG=none
==========

[tyoshino (SeeGerritForStatus), 2016-12-15 08:16:31]

The CQ bit was checked by tyoshino@chromium.org

[tyoshino (SeeGerritForStatus), 2016-12-15 08:16:31]

The patchset sent to the CQ was uploaded after l-g-t-m from treib@chromium.org,
reillyg@chromium.org, mmenke@chromium.org, mattm@chromium.org
Link to the patchset: https://codereview.chromium.org/2576663004/#ps20001
(title: "Rebase")

[commit-bot: I haz the power, 2016-12-15 08:16:46]

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-15 09:30:04]

CQ is committing da patch.

Bot data: {"patchset_id": 20001, "attempt_start_ts": 1481789791358810,
"parent_rev": "9b220d080d7b90410e9ffd5256d7bf011e072f15", "commit_rev":
"ac4fdf5574bdadf5b3d6383f4c4efc1841a55c85"}

[commit-bot: I haz the power, 2016-12-15 09:30:29]

Description was changed from

==========
content::ResourceType clean up

- Remove unused using statements
- Remove unnecessary content::
- Include resource_type.h where IsResourceTypeFrame() is used
- Use IsResourceTypeFrame() where we can

R=reillyg@chromium.org,mmenke@chromium.org,mattm@chromium.org,treib@chromium.org
BUG=none
==========

to

==========
content::ResourceType clean up

- Remove unused using statements
- Remove unnecessary content::
- Include resource_type.h where IsResourceTypeFrame() is used
- Use IsResourceTypeFrame() where we can

R=reillyg@chromium.org,mmenke@chromium.org,mattm@chromium.org,treib@chromium.org
BUG=none

Review-Url: https://codereview.chromium.org/2576663004
==========

[commit-bot: I haz the power, 2016-12-15 09:30:31]

Committed patchset #2 (id:20001)

[commit-bot: I haz the power, 2016-12-15 09:37:03]

Description was changed from

==========
content::ResourceType clean up

- Remove unused using statements
- Remove unnecessary content::
- Include resource_type.h where IsResourceTypeFrame() is used
- Use IsResourceTypeFrame() where we can

R=reillyg@chromium.org,mmenke@chromium.org,mattm@chromium.org,treib@chromium.org
BUG=none

Review-Url: https://codereview.chromium.org/2576663004
==========

to

==========
content::ResourceType clean up

- Remove unused using statements
- Remove unnecessary content::
- Include resource_type.h where IsResourceTypeFrame() is used
- Use IsResourceTypeFrame() where we can

R=reillyg@chromium.org,mmenke@chromium.org,mattm@chromium.org,treib@chromium.org
BUG=none

Committed: https://crrev.com/430d2f47056ec2691b43239f6390758a3f2e6ccc
Cr-Commit-Position: refs/heads/master@{#438792}
==========

[commit-bot: I haz the power, 2016-12-15 09:37:04]

Patchset 2 (id:??) landed as
https://crrev.com/430d2f47056ec2691b43239f6390758a3f2e6ccc
Cr-Commit-Position: refs/heads/master@{#438792}