home

Contribute code to Chromium

Apr 09, 2014

준비작업

  • MacBuildInstructions 를 참고하여 소스코드를 받아 놓고, 빌드까지 한다.

  • git 정보 확인

➜  ✗  cd  /chromium/src/chrome/browser

➜  browser git:(master) ✗ pwd
/chromium/src/chrome/browser

➜  browser git:(master) ✗ git remote -v
origin	https://chromium.googlesource.com/chromium/src.git (fetch)
origin	https://chromium.googlesource.com/chromium/src.git (push)

CPP Guide를 참조해서 문제점 찾아보기


➜  base git:(master) ✗ cpplint.py * 

  1 Ignoring DEPS; not a .cc or .h file
  2 Done processing DEPS
  3 Ignoring OWNERS; not a .cc or .h file
  4 Done processing OWNERS
  5 about_flags.cc:44:  "ash/ash_switches.h" already included at about_flags.cc:13  [build/include] [4]
  6 about_flags.cc:103:  Consider using DCHECK_EQ instead of DCHECK(a == b)  [readability/check] [2]
  7 about_flags.cc:460:  { should almost always be at the end of the previous line  [whitespace/braces] [4]
  8 about_flags.cc:467:  { should almost always be at the end of the previous line  [whitespace/braces] [4]
  9 about_flags.cc:479:  { should almost always be at the end of the previous line  [whitespace/braces] [4]
  

5번째 라인
about_flags.cc:44:  "ash/ash_switches.h" already included at about_flags.cc:13  [build/include] [4] 
=> 문제점 리뷰후 내용을 수정

branch 생성

➜  browser git:(master) ✗ git checkout -b browser_bwahn


➜  browser git:(browser_bwahn) ✗ git branch
* browser_bwahn
  master

수정된 내용을 commit

$ git commit -am "cleaning the code"
...

AUTHORS에 자신의 이름/이메일 추가한다.

➜  src git:(master) ✗  pwd
/chromium/src

➜  src git:(master) ✗  vi AUTHORS
...

codereview를 위해 upload


$ git cl upload

=> BUG와 TEST를 안해도 된다면, 모두 NONE 으로..
BUG=NONE
TEST=NONE


// 업로드하면 아래처럼 번호가 나온다.
https://codereview.chromium.org/234023004/

codereviewer 지정하기

// OWNERS 찾아보기 
// 
➜  browser git:(master) ✗ pwd
/chromium/src/chrome/browser

➜  browser git:(master) ✗ cat OWNERS



=> 브라우저 : https://codereview.chromium.org/232973002/ 
// 에디트 후 reviewer 지정

// Publish+Mail Comments 로 메일을 보낸다. 

// reviewer로 부터 
// lgtm 커멘트를 받는다.

Reference :