[WebView] Flatten lib/main to lib

[WebView] Flatten lib/main to lib

Moves all files under lib/main to lib/ since two levels
here didn't make sense.

With some git cl format.

BUG=716604
Review-Url: https://codereview.chromium.org/2901333003
Cr-Commit-Position: refs/heads/master@{#474736}
Committed: https://chromium.googlesource.com/chromium/src/+/323e56e432cb28e97150b1b2a36225b751e3c398

[Shimi Zhang, 2017-05-25 01:51:18]

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

[commit-bot: I haz the power, 2017-05-25 01:52:31]

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, 2017-05-25 03:52:01]

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

[commit-bot: I haz the power, 2017-05-25 03:52:02]

Dry run: This issue passed the CQ dry run.

[Shimi Zhang, 2017-05-25 17:00:27]

Description was changed from

==========
[WebView] Flatten lib/main to lib

Moves all files under lib/main to lib/ since two levels
here didn't make sense.

BUG=716604
==========

to

==========
[WebView] Flatten lib/main to lib

Moves all files under lib/main to lib/ since two levels
here didn't make sense.

With some git cl format.

BUG=716604
==========

[Shimi Zhang, 2017-05-25 17:00:51]

ctzsm@chromium.org changed reviewers:
+ torne@chromium.org

[Shimi Zhang, 2017-05-25 17:00:52]

torne@, PTAL, thanks!

[Torne, 2017-05-25 17:04:16]

LGTM with a couple nits

Thanks for doing this, I can cross this off my todo list :)

https://codereview.chromium.org/2901333003/diff/1/android_webview/lib/aw_main...
File android_webview/lib/aw_main_delegate.cc (right):

https://codereview.chromium.org/2901333003/diff/1/android_webview/lib/aw_main...
android_webview/lib/aw_main_delegate.cc:77: // WebRTC hardware decoding is not
supported, internal bug 15075307
It'd look nicer if you just moved the comment inside the #if and left it
indented. (which clang-format should be fine with)

https://codereview.chromium.org/2901333003/diff/1/android_webview/lib/webview...
File android_webview/lib/webview_jni_onload.h (right):

https://codereview.chromium.org/2901333003/diff/1/android_webview/lib/webview...
android_webview/lib/webview_jni_onload.h:5: #ifndef
ANDROID_WEBVIEW_LIB_MAIN_WEBVIEW_JNI_ONLOAD__H_
The guard here should change as well?

[Shimi Zhang, 2017-05-25 17:40:41]

Thanks for catching, I should be more careful though.. :P

https://codereview.chromium.org/2901333003/diff/1/android_webview/lib/aw_main...
File android_webview/lib/aw_main_delegate.cc (right):

https://codereview.chromium.org/2901333003/diff/1/android_webview/lib/aw_main...
android_webview/lib/aw_main_delegate.cc:77: // WebRTC hardware decoding is not
supported, internal bug 15075307
On 2017/05/25 17:04:16, Torne wrote:
> It'd look nicer if you just moved the comment inside the #if and left it
> indented. (which clang-format should be fine with)

Done.

https://codereview.chromium.org/2901333003/diff/1/android_webview/lib/webview...
File android_webview/lib/webview_jni_onload.h (right):

https://codereview.chromium.org/2901333003/diff/1/android_webview/lib/webview...
android_webview/lib/webview_jni_onload.h:5: #ifndef
ANDROID_WEBVIEW_LIB_MAIN_WEBVIEW_JNI_ONLOAD__H_
On 2017/05/25 17:04:16, Torne wrote:
> The guard here should change as well?

Done.

[Shimi Zhang, 2017-05-25 17:42:55]

The CQ bit was checked by ctzsm@chromium.org

[Shimi Zhang, 2017-05-25 17:42:55]

The patchset sent to the CQ was uploaded after l-g-t-m from torne@chromium.org
Link to the patchset: https://codereview.chromium.org/2901333003/#ps20001
(title: "nits")

[commit-bot: I haz the power, 2017-05-25 17:43:51]

CQ is trying da patch.

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

[Shimi Zhang, 2017-05-25 17:51:12]

ctzsm@chromium.org changed reviewers:
+ tedchoc@chromium.org

[Shimi Zhang, 2017-05-25 17:51:13]

tedchoc@, could you PTAL for chrome/browser/android/monochrome_entry_point.cc,
thanks!

[commit-bot: I haz the power, 2017-05-25 17:53:32]

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

[commit-bot: I haz the power, 2017-05-25 17:53:34]

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...)

[Ted C, 2017-05-25 17:57:59]

On 2017/05/25 17:53:34, commit-bot: I haz the power wrote:
> 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...)

monochrome_entry_point.cc - lgtm

[Shimi Zhang, 2017-05-25 17:58:44]

The CQ bit was checked by ctzsm@chromium.org

[commit-bot: I haz the power, 2017-05-25 17:59:12]

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-05-25 19:00:03]

CQ is committing da patch.

Bot data: {"patchset_id": 20001, "attempt_start_ts": 1495735124654390,
"parent_rev": "6e2e0996275dd48dd4497f9c57e6370a423eb3ec", "commit_rev":
"323e56e432cb28e97150b1b2a36225b751e3c398"}

[commit-bot: I haz the power, 2017-05-25 19:00:13]

Description was changed from

==========
[WebView] Flatten lib/main to lib

Moves all files under lib/main to lib/ since two levels
here didn't make sense.

With some git cl format.

BUG=716604
==========

to

==========
[WebView] Flatten lib/main to lib

Moves all files under lib/main to lib/ since two levels
here didn't make sense.

With some git cl format.

BUG=716604

Review-Url: https://codereview.chromium.org/2901333003
Cr-Commit-Position: refs/heads/master@{#474736}
Committed:
https://chromium.googlesource.com/chromium/src/+/323e56e432cb28e97150b1b2a362...
==========

[commit-bot: I haz the power, 2017-05-25 19:00:14]

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